[Ananisiki]

^^^^^^^^

[jamesbond007]

pawn Код:

stock SelectRandomPlayer()
{
    new Prandom = random(MAX_PLAYERS);
    if(IsPlayerConnected(Prandom))
    {
       return Prandom;
    }
    else
    {
       return SelectRandomPlayer();
    }
}

[MP2]

Use Iter_Random in foreach. Also, that code could still return a non-connected player ID.

[jamesbond007]

Quote:

Originally Posted by MP2 Also, that code could still return a non-connected player ID.

how..?

[MP2]

Okay I failed and didn't see the recursion. But still, you should use Iter_Random in foreach(). Also, a while() loop would be better than recursion.

[jamesbond007]

Quote:

They aren't connected either, but i'll still return the ID..

wtf? wrong.. it will keep looping until it finds connected... possibly going infinite if server is empty..
why would it stop there.. lol

[MP2]

Re-read please - I fucked up :P