[Krtola]

When player reload specific sound is played. That sound can hear all players,from TS and CT.
How can I make this sound heard only players on my team?

PHP Code:

public Event_OnReload ( const i_Ent )
{
        static id; id = pev ( i_Ent, pev_owner );
        
        if ( UTIL_IsWeaponReloading ( id, i_Ent ) )
        {
            emit_sound ( id, CHAN_AUTO, SoundList[ random_num( 0, sizeof SoundList - 1 ) ], 1.0, ATTN_NORM, 0, PITCH_NORM );
        }
} 


[edon1337]

Create a variable that will hold player's team, loop through all players and check if looped player's team isn't equal to the player who is reloading, if true use continue, else emit_sound with the variable that you looped.

new g_iMaxPlayers ;

public plugin_init( ) {

    g_iMaxPlayers = get_maxplayers( ) ;
}

public Event_OnReload ( const i_Ent )
{
    static id; id = pev ( i_Ent, pev_owner );
    
    new iTeam = get_user_team( id ) ;
    
    if ( UTIL_IsWeaponReloading ( id, i_Ent ) )
    {   
        for( new i = 1 ; i <= g_iMaxPlayers ; i++ ) {
            
            if( ! is_user_connected( i ) )
            continue ;

            if( iTeam != get_user_team( i ) )
            continue ;
            
            emit_sound ( i, CHAN_AUTO, SoundList[ random_num( 0, sizeof SoundList - 1 ) ], 1.0, ATTN_NORM, 0, PITCH_NORM );
        }
    }
    
    return PLUGIN_CONTINUE ;
}

[Depresie]

I don't think you can do that, even the name suggests it, "emit a sound ( from a location )"
The only official way to do something like that is using the spk command

But, there may be a workaround, but i'm not sure if it would work..

One thing i was thinking was blocking the sound by hooking the emit sound forward ( if i remember right there should be something like that )

Another thing would be to use a specific channel, then for the specific players emit a mute sound on the same channel it should replace the previous sound...

[HamletEagle]

I'm tired of repeating it, but edon, that's not how you loop through all players. Your code will throw errors in 99,99% of cases.

[edon1337]

Originally Posted by HamletEagle I'm tired of repeating it, but edon, that's not how you loop through all players. Your code will throw errors in 99,99% of cases.

Huh?

[OciXCrom]

PHP Code:

for(new i; i > 0; i++)
{
    if("edon1337" uses wrong loop)
        message("edon1337", "USE get_players() function")
} 


[Krtola]

edon1337 tested and dont work.

[HamletEagle]

Originally Posted by edon1337 Huh?

How do you know that all slots from 1 to max players are filled? Using get_players is the proper way. Before asking why, search, you will find one million threads about that.

[edon1337]

Originally Posted by HamletEagle How do you know that all slots from 1 to max players are filled? Using get_players is the proper way. Before asking why, search, you will find one million threads about that.

is_user_connected .. ? Get_players has became an obsession to you, it's not like he's trying to make the fastest plugin ever. Module or not, they both do the same thing.

[HamletEagle]

Originally Posted by edon1337 is_user_connected .. ? Get_players has became an obsession to you, it's not like he's trying to make the fastest plugin ever. Module or not, they both do the same thing.

I simply don't understand why people want to use bad ways and even more, defend them. Why would you teach yourself bad practices? Why would you teach others bad practices?
That's not about speed after all, it's about keeping good practices while coding. If not, you'll just keep using poor ways over and over again, because who cares, right? At some points all this small things will start to matter. You already messed it by not adding the is_user_connected in the first place. If you would simply go with the right way you would not had any issue.

Do whatever you want, if you can't see that I'm trying to help, fine by me. Just don't tell other people to do the same.