[Snitch]

L 08/30/2016 - 19:06:28: [CSTRIKE] Non-player entity 3 out of range
L 08/30/2016 - 19:06:28: [AMXX] Displaying debug trace (plugin "HumansGuns.amxx")
L 08/30/2016 - 19:06:28: [AMXX] Run time error 10: native error (native "cs_get_weapon_id")
L 08/30/2016 - 19:06:28: [AMXX]    [0] HumansGuns.sma::fHam_TakeDmgPost

PHP Code:

RegisterHam( Ham_TakeDamage, "player", "fHam_TakeDmgPost", .Post = true ); 


public fHam_TakeDmgPost(iVictim, iInflictor, iAttacker, Float:fDamage, iDamageBits )
{
    if(!is_user_alive(iAttacker) || !is_user_alive(iVictim) || iVictim == iAttacker) 
        return HAM_IGNORED;
    
    static iWeaponInf, WeaponPrim, WeaponSEC;
    iWeaponInf = cs_get_weapon_id( iAttacker );
    
    WeaponPrim = g_szWeaponsPrim[ g_szWeaponSelected[ iAttacker ][ PRIM ] ][ WEAPON_CSW ];
    WeaponSEC = g_szWeaponsSec[ g_szWeaponSelected[ iAttacker ][ SEC ] ][ WEAPON_CSW ];
    
    if( iWeaponInf == WeaponPrim )
    {
        SetHamParamFloat( 4, fDamage * g_szWeaponsPrim[ g_szWeaponSelected[ iAttacker ][ PRIM ] ][ fDMG ] );
    }
    if( iWeaponInf == WeaponSEC )
    {
        SetHamParamFloat( 4, fDamage * g_szWeaponsSec[ g_szWeaponSelected[ iAttacker ][ SEC ] ][ fDMG ] );
    }
    
    return HAM_IGNORED;
}

[HamletEagle]

http://amxmodx.org/api/cstrike/cs_get_weapon_id

[Snitch]

Originally Posted by HamletEagle http://amxmodx.org/api/cstrike/cs_get_weapon_id

and? i did everything good

[SpannerSpammer]

/Sigh

The function cs_get_weapon_id( ) takes the entity index of a WEAPON, Not a player index.

In your code iInflictor is the WEAPON index. Use it.

[Snitch]

Originally Posted by SpannerSpammer /Sigh The function cs_get_weapon_id( ) takes the entity index of a WEAPON, Not a player index. In your code iInflictor is the WEAPON index. Use it.

alr, so i just have to change this line:

iAttacker > iInflictor

iWeaponInf = cs_get_weapon_id( iInflictor );

[Snitch]

bump?

[Craxor]

This works?

public fHam_TakeDmgPost(iVictim, iInflictor, iAttacker, Float:fDamage, iDamageBits )
{
    if(!is_user_alive(iAttacker) || iVictim == iAttacker) 
        return HAM_SUPERCEDE;
    
    static iWeaponInf, WeaponPrim, WeaponSEC;
    iWeaponInf = cs_get_weapon_id( iInflictor );
    
    WeaponPrim = g_szWeaponsPrim[ g_szWeaponSelected[ iAttacker ][ PRIM ] ][ WEAPON_CSW ];
    WeaponSEC = g_szWeaponsSec[ g_szWeaponSelected[ iAttacker ][ SEC ] ][ WEAPON_CSW ];
    
    if( iWeaponInf == WeaponPrim )
    {
        SetHamParamFloat( 4, fDamage * g_szWeaponsPrim[ g_szWeaponSelected[ iAttacker ][ PRIM ] ][ fDMG ] );
    }
    else if( iWeaponInf == WeaponSEC )
    {
        SetHamParamFloat( 4, fDamage * g_szWeaponsSec[ g_szWeaponSelected[ iAttacker ][ SEC ] ][ fDMG ] );
    }
    else
        return HAM_IGNORED;
 
    
    return HAM_IGNORED;
}

[PRoSToTeM@]

Originally Posted by SpannerSpammer In your code iInflictor is the WEAPON index. Use it.

In most cases inflictor is equal attacker.

[Craxor]

If he just use get_user_weapon( attacker .. is not better?

[PRoSToTeM@]

Originally Posted by Craxor If he just use get_user_weapon( attacker .. is not better?

He also need check for grenade.