[extremeg]

Hello guys,

I have the question to you if someone of you maybe know how we can prevent that connected players which are alive that we become sometimes the error:

Exception reported: Client 3 is not connected

My problem is that i dont understand it why. The player is connected successfully and is already a time on the server. I have already a check for every command that i check with IsValidClient(client) that already includes a IsClientConnected check for the client. This tells me then that this function (IsClientConnected) maybe dont works? When i check before i access to the command with IsValidClient(client) and it returns true it cant be right.

Code of IsValidClient:

stock bool IsValidClient(int client)
{
	if (client >= 1 && client <= MaxClients && IsValidEntity(client) && IsClientConnected(client) && IsClientInGame(client))
	{
		return true;
	}
	
	return false;
}

good Regards,
cRUSHER [#HLG]

[Fyren]

Show us the actual code that causes the run-time error.

[Bara]

Remove IsValidEntity and IsClientConnected (from your IsValidClient stock), your error should be fixed now with the "same" valid check.

[pride95]

PHP Code:


RegConsoleCmd("sm_spec", Client_Spec);

public Action Client_Spec(int Client, int Args)
{
    if(Client != 0) // Client 0 means the server itself.
    {
        SpecPlayer(Client, Args);
    }    
        
    return Plugin_Handled;
} 


show SpecPlayer function.

in post commands hook client is always connected and in game (like regadmincmd, regconsolecmd or OnClientSayCommand_Post).

if you hook a command with OnClientSayCommand or addcommandlistener you should chech after Client != 0, if IsClientInGame(Client).

[Psyk0tik]

I know you solved your problem but I just want to share my IsValidClient stock with you since it looks different from yours.

PHP Code:

stock bool bIsValidClient(int client)
{
    return (client > 0 && client <= MaxClients && IsClientInGame(client) && !IsClientInKickQueue(client) && !IsFakeClient(client));
} 


I also noticed that your stock originally had IsClientConnected() && IsClientInGame in it, which is redundant.

This post explains why: https://forums.alliedmods.net/showthread.php?t=132438