[TheKidz]

what is the best way ?

PHP Code:

 is_user_connected( index ) 


or

PHP Code:

new bool:g_Connected[ 33 ];

public client_connected( id ) g_Connected[ id ] = true

public client_disconnected( id ) g_Connected[ id ] = false

stock bool:isConnected( index ) return g_Connected[ index ] ? true : false 


[hleV]

How often are you checking it?

[TheKidz]

what u mean with often ?

[bibu]

How much do you check this function in your plugin.....

[Sylwester]

If you are checking if user is connected for example when you execute command where player is a target then using is_user_connected(id) is fine, but if you loop through all players in prethink then caching is_user_connected is better however your method is not the best. You should use something like this:

PHP Code:

new _pg_is_ced
#define _IsUserConnected(%1) ( _pg_is_ced & 1<<(%1) )
#define _SetUserConnected(%1) _pg_is_ced |= 1<<(%1)
#define _SetUserNotConnected(%1) _pg_is_ced &= ~( 1<<(%1) )

public client_connected( id ) _SetUserConnected(id)

public client_disconnected( id ) _SetUserNotConnected(id)

public some_function(id){
    if(_IsUserConnected(id)){
        //code...
    }
} 


[TheKidz]

i check if is user connected like a billion of times in my code, everything is checked :/
bcoz i got too many players count, tasks, etcetera
and btw, there is another way without bitsum ?

[hleV]

Originally Posted by TheKidz i check if is user connected like a billion of times in my code, everything is checked :/ bcoz i got too many players count, tasks, etcetera and btw, there is another way without bitsum ?

Use bitsum.

[TheKidz]

but it really worth change is_user_connected( index ) to bitsums ? just wondering

[Xellath]

You could use an array, but a bitsum would be more efficient.

[TheKidz]

Thanks. Solved ;*