[greentarch]

Wrong usage
Why do you even loop?

Use it like

pawn Код:

new
    x = GetClosestPlayer( playerid, 5.0 );
if ( id != -1 ) // ID is valid
{
    // Rest of code
}