[JusTGo]

PHP Code:

public ham_Killed_Post( id, Attacker, shouldgib )
{
    new Float:flLocation[ 3 ];
    pev( id, pev_origin, flLocation );
    
    new players[ 32 ], num, player;
    get_players( players, num );
    for( new i ; i < num ; i ++ )
    {
        player = players[ i ];
        if( get_user_team( player ) != get_user_team( id ) || !is_user_alive( player ) )
        continue;
            
        message_begin( MSG_ONE_UNRELIABLE, g_msgHostageAdd, { 0, 0, 0 }, player );
        write_byte( player );
        write_byte( id )    ;    
        engfunc( EngFunc_WriteCoord, flLocation[ 0 ] );
        engfunc( EngFunc_WriteCoord, flLocation[ 1 ] );
        engfunc( EngFunc_WriteCoord, flLocation[ 2 ] );
        message_end();
                
        message_begin( MSG_ONE_UNRELIABLE, g_msgHostageDel, { 0, 0, 0 }, player );
        write_byte( id );
        message_end();
    }
} 


is it possible for this message to be sent to all players with MSG_BROADCAST ?

[EFFx]

PHP Code:

    for( new i ; i < num ; i ++ ) 
    { 
        player = players[ i ]; 
        if( get_user_team( player ) != get_user_team( id ) || !is_user_alive( player ) ) 
        continue; 
             
        message_begin( MSG_BROADCAST, g_msgHostageAdd); 
        write_byte( player ); 
        write_byte( id )    ;     
        engfunc( EngFunc_WriteCoord, flLocation[ 0 ] ); 
        engfunc( EngFunc_WriteCoord, flLocation[ 1 ] ); 
        engfunc( EngFunc_WriteCoord, flLocation[ 2 ] ); 
        message_end(); 
                 
        message_begin( MSG_ONE_UNRELIABLE, g_msgHostageDel, { 0, 0, 0 }, player ); 
        write_byte( id ); 
        message_end(); 
    } 


[siriusmd99]

just a correction on get user team:
it is more efficient to do
new tmp_team = get_user_team(id) //write this above the loop

then change to this in loop:

if( get_user_team( player ) != tmp_team)


because you are calling get user team for same player multiple times which makes no sense