Hah, this forum never changes, discussing minor details about code that doesn't even meet the requirements.
padilha007: you've been told what's wrong with your code. You're using set_task() as a control structure and that's simply not what it's for. Your code also includes a redundant test.
Exolent: whether or not to use get_players() once in a single round is a matter of style. The speed difference, if measurable, is negligible. Not only that, but you need to determine if the players are alive (or on the correct team), which is "unsupported" using get_players() flags for whatever reason.
Dores: that code will result in a infinite loop if there's no valid players connected. You also include a similar bug as hleV (by assuming player indexes are consecutive and begin at 1) and so could be stuck in an infinite loop even if there are valid players connected.
micke1101: you need to follow
Exolent's tutorial. As has been stated multiple times, none of the code here meets your request. Post again if you need help with it.