[Joe Staff]

Don't place variable calls inside of loops, it's very redundant (like creating a new variable for every connected player, when you could just create one variable and then use it for each.)

pawn Код:

forward HealthCheck();
public HealthCheck()
{
    new Float:fHealth;
    for( new i = 0; i < MAX_PLAYERS; i ++ ) if( IsPlayerConnected( i ) )
    {
        
        GetPlayerHealth(i, fHealth);
        if(fHealth > 100.0) SetPlayerHealth(i, 100.0);
    }
    return 1;
}

There's no reason why that code shouldn't be working for you, OP.