[OciXCrom]

I registered multiple primary attacks into one function like this:

PHP Code:

RegisterHam(Ham_Weapon_PrimaryAttack, "weapon_famas", "eventPrimaryAttack", 1)
RegisterHam(Ham_Weapon_PrimaryAttack, "weapon_ak47", "eventPrimaryAttack", 1)
RegisterHam(Ham_Weapon_PrimaryAttack, "weapon_m4a1", "eventPrimaryAttack", 1) 


And in the function I'm trying to play a different sound for each one:

PHP Code:

public eventPrimaryAttack(iWeapon)
{
    new szSound[128], iClip, iAmmo, id = pev(iWeapon, pev_owner)
    get_user_weapon(id, iClip, iAmmo)
    
    if(iClip)
    {
        switch(iWeapon)
        {
            case CSW_FAMAS: formatex(szSound, charsmax(szSound), "%s", SOUND_FAMAS)
        }
    }
    
    emit_sound(id, CHAN_WEAPON, szSound, 1.0, ATTN_NORM, 0, PITCH_HIGH)
} 


The switch code doesn't work. iWeapon doesn't seem to respond with CSW_ constants. How do I check the weapon from this code and is this a good way to do it? I know there isn't a 100% efficient way to do this.

[xxxperts]

Are you trying to play some different sound with different guns ?

EDIT: If yes then please check out this post and if you got time then see whole theard you will find something useful

[HamletEagle]

What you get from the forward is the weapon entity index, not the CSW_* index. You have to use cs_get_weapon_id to get what you want.

[siriusmd99]

Originally Posted by HamletEagle What you get from the forward is the weapon entity index, not the CSW_* index. You have to use cs_get_weapon_id to get what you want.

Yes, but in his case (he is using get_user_weapon to get clip and ammo) he doesn't need to use cs_get_weapon_id because cs_get_user_weapon returns weapon id in CSW_* format.
And you would better use copy instead of formatex because you are not formating string , just copying.

But i don't understand why you are creating new string, then copying the sound to it and then using emit sound.

Just use emit sound directly with your sound.

Here's the code:

PHP Code:

public eventPrimaryAttack(iWeapon)
{
    new iClip, iAmmo, id = pev(iWeapon, pev_owner)
    // You can add here:  if(!is_user_connected(id) || !is_user_alive(id)) return HAM_IGNORED;
   // But i don't know if it's necessary.
  
    iWeapon = get_user_weapon(id, iClip, iAmmo)
    
    if(iClip)
    {
        switch(iWeapon)
        {
            case CSW_FAMAS: emit_sound(id, CHAN_WEAPON, SOUND_FAMAS, 1.0, ATTN_NORM, 0, PITCH_HIGH)
            case CSW_M4A1: emit_sound(id, CHAN_WEAPON, SOUND_M4A1, 1.0, ATTN_NORM, 0, PITCH_HIGH)
            case CSW_AK47: emit_sound(id, CHAN_WEAPON, SOUND_M4A1, 1.0, ATTN_NORM, 0, PITCH_HIGH)
        }
    }
    
    
} 


[OciXCrom]

@xxxperts: I went over a bunch of threads like that, thank you for that one as well.
@siriusmd99: Works like a charm. Thank you!

[SpeeDeeR]

Ham_Weapon_PrimaryAttack is called by the entity, not weaponID as stated above.

[OciXCrom]

I'm using siriusmd99's code and I have a little issue with it. When I hold down the attack button on a pistol (deagle, fiveseven etc), it spams the sound repeatedly. The other weapons work fine. How to I prevent this from happening?

[OciXCrom]

Up. I'm still haven't found a solution for the problem in my last post.