[Siveroo]

As the title said, i want to run some piece of code if a player is shooting, (not just left clicking, +attack, or whatever). I want it to trigger if the players actually shoots a bullet. I've search something related to this, and someone said that its possible to use playbackevent, but i dont know what that is, let alone using it. I tried to search how to use it tho, but still have zero clue..

[Shadows Adi]

Hamsandwich functions:

PHP Code:

Ham_Weapon_PrimaryAttack(iEnt) - For primary attack hook 


PHP Code:

Ham_Weapon_SecondaryAttack(iEnt) - For secondary attack hook 


[gabuch2]

I think the closest thing you can do is hooking FireBulletsPlayer() with Orpheu, but that is more complex since you need signatures. Otherwise you'll need to rely on Ham_Weapon_PrimaryAttack and Ham_Weapon_SecondaryAttack but that won't give you the exact moment when they shoot, but rather the moment the attack function is processed. (Example of PrimaryAttack() in weapon_9mmar).

[Natsheh]

here's the proper way its rather than using primary attack.

https://forums.alliedmods.net/showthread.php?t=54887

you can also catch all fired weapons by hooking FM_TraceLine or Ham_TraceAttack if you're interested to hook a fire for a specific weapon.

[HamletEagle]

PrimaryAttack is not a good idea, if you are out of ammo and hold m1 it will still trigger. As Natsheh said, PlayBackEvent is a better option.

[OciXCrom]

Originally Posted by HamletEagle if you are out of ammo and hold m1 it will still trigger

You can easily prevent that:

const XO_WEAPON = 4
const m_iClientClip = 52

if(cs_get_weapon_ammo(iEnt) == get_pdata_int(iEnt, m_iClientClip, XO_WEAPON))
{
    return
}

[HamletEagle]

Originally Posted by OciXCrom You can easily prevent that: const XO_WEAPON = 4 const m_iClientClip = 52 if(cs_get_weapon_ammo(iEnt) == get_pdata_int(iEnt, m_iClientClip, XO_WEAPON)) {     return }

Which is useless when you already have PlaybackEvent. It is called from PrimaryAttack(indirectly, PrimaryAttack -> WeaponFire -> PlaybackEvent) after checking ammo.
Just because you can do something it does not mean you should.

[Natsheh]

Basically its like going to a destination C, starting from point A going through Point B, knowing you can go to point C directly without needing to pass through point B.

[Siveroo]

Originally Posted by Natsheh here's the proper way its rather than using primary attack. https://forums.alliedmods.net/showthread.php?t=54887 you can also catch all fired weapons by hooking FM_TraceLine or Ham_TraceAttack if you're interested to hook a fire for a specific weapon.

i think im gonna stick with this for now, also im wondering, can i hook knife left click and right click attack too? (still gonna apply the same rule where you need to actually attack, not just detecting if +attack/+attack2 is pressed)

[OciXCrom]

Originally Posted by HamletEagle Which is useless when you already have PlaybackEvent. It is called from PrimaryAttack(indirectly, PrimaryAttack -> WeaponFire -> PlaybackEvent) after checking ammo. Just because you can do something it does not mean you should.

True, but PlaybackEvent requires manually typing in the .sc files and handling precache, while with Ham you can directly use get_weaponname(). If one simple check can prevent all of this, I'd go for that even though the other method is more direct.