[TheSpaniard]

Originally Posted by siriusmd99 Your way of thinking is really strange... You cannot call it from other function and multiply damage outside the hook function. If you want to do this just use standard ham Take damage and check some bools set from other functions. For example I make a plugin which gives me possibility to have 2x more damage by activating some kind of extra energy potion (out of imagination, sorry). For example you want it to last like for 10 seconds. So you hook impulse (flashlight, standard when you press f) and when you press f function activate_damage will be called. In this plugin you just set the time when this will end like g_endtime[player] =get systime + how_many_seconds Then in ham take damage you check If get systime - g_endtime[player] >= 0 //then multiply damage or other stuff you want Try to think more abstract instead of inventing weird things..

I am not calling anything outside the hook see this :-

PHP Code:

public fw_takedamage(iClient, iInflictor, iAttacker, Float:fDamage, m_Damagebits)
{
    if(is_user_alive(iClient) && is_user_alive(iAttacker) && iInflictor == CSW_AK47||CSW_M4A1||CSW_AWP)
    {
        SetHamParamFloat(4, fDamage*get_pcvar_float(CVAR_DAMAGE_MULTI))
    }
    return HAM_IGNORED
} 


As the code suggests i want the damage multiplied in specific case where the inflictor is m4a1 and ak47 and awp
The Catch:-
These are not the only conditions i want to apply i want the ham to be executed only when it is called in a specific case, in switch condition or the above ham would be executed on all ak47 or awp or m4a1 skins i am putting in the plugin.

EDIT: The above code is done immediately after crazy's answer, this for people who can say that i am a copycat not a programmer.

[DjSoftero]

Originally Posted by TheSpaniard I am not calling anything outside the hook see this :- PHP Code: public fw_takedamage(iClient, iInflictor, iAttacker, Float:fDamage, m_Damagebits) {     if(is_user_alive(iClient) && is_user_alive(iAttacker) && iInflictor == CSW_AK47||CSW_M4A1||CSW_AWP)     {         SetHamParamFloat(4, fDamage*get_pcvar_float(CVAR_DAMAGE_MULTI))     }     return HAM_IGNORED }  As the code suggests i want the damage multiplied in specific case where the inflictor is m4a1 and ak47 and awp The Catch:- These are not the only conditions i want to apply i want the ham to be executed only when it is called in a specific case, in switch condition or the above ham would be executed on all ak47 or awp or m4a1 skins i am putting in the plugin. EDIT: The above code is done immediately after crazy's answer, this for people who can say that i am a copycat not a programmer.

inflictor is usualy the attacker, unless it`s the grenade so you can`t compare it that way. also if inflictor was gun it should be inflictor == csw_awp || incfilctor == csw_ak47 || inflicotr ==... etc. but again. inflicotr is not gun entity. so dont do it

[TheSpaniard]

Originally Posted by DjSoftero inflictor is usualy the attacker, unless it`s the grenade so you can`t compare it that way. also if inflictor was gun it should be inflictor == csw_awp || incfilctor == csw_ak47 || inflicotr ==... etc. but again. inflicotr is not gun entity. so dont do it

it says victim is the person on which the damage is executed, attacker is the owner of the inflictor and inflictor is the entity by which the direct damage is inflicted for example knife.

[DjSoftero]

accidently deleted message. amazing...

look at this: https://forums.alliedmods.net/showthread.php?t=82334

Inflictor is the entity that caused the damage directly to the victim. For knife/guns attacker is the inflictor.

If you don`t believe me, then do the research/testing part on your won.

[aron9forever]

Originally Posted by TheSpaniard it says victim is the person on which the damage is executed, attacker is the owner of the inflictor and inflictor is the entity by which the direct damage is inflicted for example knife.

That's incorrect, when it's a player directly inflicting the damage(using a weapon) inflictor will always be the same as attacker.

Inflictor will contain non-player entity id for things such as nade explosions, c4 explodes or other sorts of custom damages such as spawning an env_explosion


Here's how to do damage bonuses on different weapons

//Inside ham takedamage to double knife damage
if(is_user_connected(attacker) && attacker == inflictor && get_user_weapon(attacker) == CSW_KNIFE) {
    SetHamParamFloat(4, damage * 2);
  }

code can be replicated for other or multiple weapons
if you have more than one then I would strongly suggest you do a switch for get_user_weapon()

[DeMNiX]

Originally Posted by TheSpaniard public fw_takedamage(iClient, iInflictor, iAttacker, Float:fDamage, m_Damagebits) { if(is_user_alive(iClient) && is_user_alive(iAttacker) && iInflictor == CSW_AK47||CSW_M4A1||CSW_AWP) { SetHamParamFloat(4, fDamage*get_pcvar_float(CVAR_DAMAGE_MULTI)) } return HAM_IGNORED }

PHP Code:

 iInflictor == CSW_AK47||CSW_M4A1||CSW_AWP 


did u test it? i think not

PHP Code:

 (is_user_alive(iClient) 


it's unnecessary code. Player that tooks damage can't be invalid

PHP Code:

 //checks knife damage
if (m_Damagebits & (DMG_BULLET | DMG_NEVERGIB )) 


[Natsheh]

This check is incorrect

iInflictor == CSW_AK47||CSW_M4A1||CSW_AWP

[DeMNiX]

PHP Code:

public fw_takedamage(iClient, iInflictor, iAttacker, Float:fDamage, m_Damagebits)
{
     if(~m_Damagebits & (DMG_BULLET | DMG_NEVERGIB )) return HAM_IGNORED; // if not knife attack return;
     //some yours conditions here
     // no needs to checking on attacker becouse it's knife attack. No needs to check victim
     if (!you_condition) return HAM_IGNORED;

     SetHamParamFloat(4, fDamage*get_pcvar_float(CVAR_DAMAGE_MULTI))
     return HAM_IGNORED;
} 


[TheSpaniard]

Originally Posted by DeMNiX PHP Code: public fw_takedamage(iClient, iInflictor, iAttacker, Float:fDamage, m_Damagebits) {      if(~m_Damagebits & (DMG_BULLET | DMG_NEVERGIB )) return HAM_IGNORED; // if not knife attack return;      //some yours conditions here      // no needs to checking on attacker becouse it's knife attack. No needs to check victim      if (!you_condition) return HAM_IGNORED;      SetHamParamFloat(4, fDamage*get_pcvar_float(CVAR_DAMAGE_MULTI))      return HAM_IGNORED; }

Doesn't anybody think for my case event_damage makes more sense. Because i want it for multiple weapons??

[DeMNiX]

Originally Posted by TheSpaniard Doesn't anybody think for my case event_damage makes more sense. Because i want it for multiple weapons??

PHP Code:

public fw_takedamage(iClient, iInflictor, iAttacker, Float:fDamage, m_Damagebits)
{
     if(m_Damagebits & (DMG_BULLET | DMG_NEVERGIB )) { //if knife attack;
     //some yours conditions here
     // no needs to checking on attacker becouse it's knife attack. No needs to check victim
     if (!you_condition) return HAM_IGNORED;

     SetHamParamFloat(4, fDamage*get_pcvar_float(CVAR_DAMAGE_MULTI))
     }
     else if(m_Damagebits & DMG_BULLET) {
            // Check active item of attacker here and set the damage

      
     } 

    return HAM_IGNORED;
}