[boylett]

Quote:

Originally Posted by tomozj This gets a randomly selected connected ID. If none is connected, then in theory, it'll keep trying until someone does. pawn Код: stock GetRandomID() {     new randn = random(MAX_PLAYERS);         if(IsPlayerConnected(randn)) return randn;         else     {         return GetRandomID();     } } Example of use: pawn Код: GivePlayerMoney(GetRandomID(), 5000);

Isnt that a bit dangerous? If no ones connected it will never end, if a few people are connected it will take a long time to complete and pawn is single threaded. I think this might be better:

pawn Код:

stock GetRandomID()
{
    new bool:connected[MAX_PLAYERS] = false, amount = 0;
    for(new i = 0; i < MAX_PLAYERS; i++)
      if(IsPlayerConnected(i))
      {
        connected[i] = true;
        amount++;
      }
    if(amount = 0) return -1;
    new rand = random(amount), done = 0;
    for(new i = 0; i < MAX_PLAYERS; i++)
      if(connected[i])
      {
        if(done == rand) return i;
        done++;
        }
    return -1;
}

returns -1 if no1s connected.