[CalvinC]

Instead of using a loop that loops through 500 ID's, and check if a player is connected in that ID, you should rather use foreach.
It's more efficient by only looping through connected players.

But in that example you gave, there's no reason to use a loop at all.