[fysiks]

I've been told is much more efficient to use get_players() in most all cases even if you do need to loop through them. One thing that it does is that it optimizes how many players you need to loop through.

I'm not entirely sure if your method would be any better or worse than having to call multiple calls to get_players() to get the same resulting data. With modern computers it might not be all that significant either way, especially since you're not calling it all that often.

P.S. Whitespace is not your enemy, it is your friend when used correctly so you should use it. Nobody wants to have to decipher your lack of whitespace when trying to help, it's just annoying. With such a small community, it's probably a good idea to make code that you post readable to make it easier to help you.