[pentakill]

So, I have a weapons menu, and the player can choose the normal menu, gold menu and super menu

Gold menu weapons take double damage, and super take 3x damage

PHP Code:

public Damage(victim, inflictor, attacker, Float:damage, damagebits) 
{ 
    if(is_user_connected(victim) || is_user_alive(victim))  
    { 
        if( get_user_weapon(attacker) == CSW_AK47 )
        {
            if (AreGoldenAk[attacker])
            {
                SetHamParamFloat( 4, damage * 2 )
                return HAM_HANDLED
            }
            if (AreSuperAk[attacker])
            {
                SetHamParamFloat( 4, damage * 3)
                return HAM_HANDLED
            }
        }
    }
    return HAM_IGNORED
} 


But this isn't working and I don't know why...

I use the same bools (AreGoldenAk and AreSuperAk) to change the skins and works!

Can someone help me?

[HamletEagle]

Give us full code or explain better what you mean by not working, because it's not something wrong with the code you posted, just: is_user_connected(victim) || is_user_alive(victim) is should be &&

[pentakill]

Originally Posted by HamletEagle Give us full code or explain better what you mean by not working, because it's not something wrong with the code you posted, just: is_user_connected(victim) || is_user_alive(victim) is should be &&

I gave all the informations, what you want to know more?

&& and || have the same function I think, for me works fine using ||

[Balck]

Originally Posted by pentakill && and || have the same function I think, for me works fine using ||

|| == or
&& == and
so it's not the same

[aron9forever]

oh I see you guys didn't notice he's returning HAM_HANDLED instead of HAM_SUPERCEDE and not overwriting the normal damage inflicted

gotta pay more attention next time

[jimaway]

Originally Posted by aron9forever oh I see you guys didn't notice he's returning HAM_HANDLED instead of HAM_SUPERCEDE and not overwriting the normal damage inflicted gotta pay more attention next time

HAM_SUPERCEDE blocks the function, definitely not what he wants.

you don't have to check if user is connected when you use is_user_alive(), it already checks that.
post how you are hooking ham_takedamage

[pentakill]

Originally Posted by jimaway you don't have to check if user is connected when you use is_user_alive(), it already checks that.

Big mistake, edited to only check is_user_alive

Originally Posted by jimaway post how you are hooking ham_takedamage

RegisterHam(Ham_TakeDamage, "player", "Damage")

EDITED: I don't know what I've done, but know the plugin works! Thanks anyway

[Decak]

#include <amxmodx>
#include <amxmisc>
#include <hamsandwich>

new bool:AreGoldenAk[33], bool:AreSuperAk[33];

public plugin_init() {
    register_plugin("Register Name", "1.0", "Author")
    RegisterHam(Ham_TakeDamage, "player", "Damage")
}

public Damage(victim, inflictor, attacker, Float:damage, damagebits) 
{ 
    if(is_user_connected(victim) || is_user_alive(victim))  
    { 
        if( get_user_weapon(attacker) == CSW_AK47 )
        {
            if (AreGoldenAk[attacker])
            {
                SetHamParamFloat( 4, damage * 2 )
                return HAM_HANDLED
            }
            if (AreSuperAk[attacker])
            {
                SetHamParamFloat( 4, damage * 3)
                return HAM_HANDLED
            }
        }
    }
    return HAM_IGNORED
}

... add code with bool AreGoldenAk and AreSuperAk.

[pentakill]

Originally Posted by Decak #include <amxmodx> #include <amxmisc> #include <hamsandwich> new bool:AreGoldenAk[33], bool:AreSuperAk[33]; public plugin_init() { register_plugin("Register Name", "1.0", "Author") RegisterHam(Ham_TakeDamage, "player", "Damage") } public Damage(victim, inflictor, attacker, Float:damage, damagebits) { if(is_user_connected(victim) || is_user_alive(victim)) { if( get_user_weapon(attacker) == CSW_AK47 ) { if (AreGoldenAk[attacker]) { SetHamParamFloat( 4, damage * 2 ) return HAM_HANDLED } if (AreSuperAk[attacker]) { SetHamParamFloat( 4, damage * 3) return HAM_HANDLED } } } return HAM_IGNORED } ... add code with bool AreGoldenAk and AreSuperAk.

My plugin already works but thanks anyway

[aron9forever]

Originally Posted by jimaway HAM_SUPERCEDE blocks the function, definitely not what he wants. you don't have to check if user is connected when you use is_user_alive(), it already checks that. post how you are hooking ham_takedamage

woops, sorry, I was high as fuck , that's when I start shitposting
he still needs return HAM_OVERRIDE instead of supercede