[Halt]

Any idea why this would return a -1 client index?

PHP Code:

GetRandomPlayer()
{
    new players[MaxClients + 1];
    new ClientCount;
    for    (new i = 1; i <= MaxClients; i++)
        if(IsClientInGame(i))
            if    (PlayerTeam[i] == TFTeam_Blue && PlayerClass[i] >= 1)
                players[ClientCount++] = i;
    return (ClientCount == 0) ? -1 : players[GetRandomInt(0, ClientCount - 1)];
} 


[headline]

When no players match the following check

PHP Code:

if (PlayerTeam[i] == TFTeam_Blue && PlayerClass[i] >= 1) 


[Halt]

Wow.... I'm a moron, I never had a check if players are even in the server... my mod was just calling this on an empty server. Thanks man.

[Bacardi]

...why you not use GetClientTeam(client) to get client team index ?

[Halt]

I do, but I get it when I client spawns and store it in a global variable (PlayerTeam)

[Bacardi]

Originally Posted by Halt I do, but I get it when I client spawns and store it in a global variable (PlayerTeam)

...roger that. I don't want to start preach

[Halt]

Originally Posted by Bacardi ...roger that. I don't want to start preach

No please, I'm in the business of learning. If I'm doing something improperly please tell me why.