[Dr.Mohammad]

hi guys!

this function is true ??

PHP Code:

#pragma newdecls required
#pragma semicolon 1

if (IsValidClient(client))
{
    //Cod Here
}

/*********************************
 *  IsValidClient
 *********************************/
stock bool IsValidClient(int client)
{
    if(client <= 0)
        return false;
    if(client > MaxClients)
        return false;
    if(!IsClientInGame(client))
        return false;
    if(IsFakeClient(client))
        return false;
    return true;
} 


[Bacardi]

Well yes.

You can skip IsClientConnected part, because IsClientInGame is enough before IsFakeClient.

[Dr.Mohammad]

Originally Posted by Bacardi Well yes. You can skip IsClientConnected part, because IsClientInGame is enough before IsFakeClient.

Thank you

[eyal282]

Take an important note. I might be wrong but generally speaking, you never really need IsValidClient.

It's either you need IsClientInGame or IsEntityPlayer ( entity >= 1 && entity <= MaxClients )

[Ilusion9]

Originally Posted by eyal282 Take an important note. I might be wrong but generally speaking, you never really need IsValidClient. It's either you need IsClientInGame or IsEntityPlayer ( entity >= 1 && entity <= MaxClients )

Yes, he doesn't need at all this function. Also, he can skip IsClientInGame for some cases.

[Dr.Mohammad]

Originally Posted by eyal282 Take an important note. I might be wrong but generally speaking, you never really need IsValidClient. It's either you need IsClientInGame or IsEntityPlayer ( entity >= 1 && entity <= MaxClients )

i must use then IsClientInGame and enough this ??