[(SF)Noobanatior]

untested but i would go like this

pawn Code:

public GetRandomPlayer(){          //returns random playerid or -1 if no players
    new test,test2,players[MAX_PLAYERS];
    for(new i=0;i<MAX_PLAYERS;i++){
        if(IsPlayerConnected(i)){
            players[test] = i;
            test++;
        }
    }
    if(test == 0)return -1;
    else test2 = random(test);
    return players[test2];
}

fine dont think it makes much difference however -1 it is