Formulas may not be the efficient in the aspects of speed if it is used in a short loop. However, 'elseifs' are only efficient at the first condition. The longer it becomes the less efficient it is. In this case a for loop is actually faster since it will not have to check through any ifs. Also, it avoids leaking at all. Players are static and will have a total of X in every game. Player Groups are not. Also, like GhostWolf said, destroying "All Players" is a bad idea since it has a great risk of screwing things up and create complications. (I mean, you will not be able to access the 'All Players' again since it is destroyed.)
Oh and, for-loops also makes the code shorter...
Edit: I just realised he didnt use a for loop...