[Sjn]

The way you have made this command is really bad. That's not how you check if a vehicle's health is equal to 1000 either. Plus, gettime() is a better replacement for GetTickCount().

Anyway, I have optimized your code a little. Try it out and let me know how it turns out.

PHP код:

// You can replace this array with your current tick holder or use this one directly
new g_RepairTime[MAX_PLAYERS]; // Global array, put this on top of your script
// Inside OnPlayerConnect
    g_RepairTime[playerid] = 0;
    
CMD:vfixveh(playerid, params[])
{
    if(Player[playerid][pVip] < 2)
        return SCM(playerid, ADMIN_COLOR, ADMIN_MESSAGE);
    if(!IsPlayerInAnyVehicle(playerid))
        return SCM(playerid, ADMIN_COLOR, " You are not in any vehicle.");
        
    if (g_RepairTime[playerid] > gettime())
        return SCM(playerid, ADMIN_COLOR, "You need to wait 3 minutes before using this command.");
    new vehicleid, Float: health;
    
    vehicleid = GetPlayerVehicleID(playerid);
    GetVehicleHealth(vehicleid, health);
    if(health == 1000.0)
        return SCM(playerid, ADMIN_COLOR, " Your vehicle is already at full hp!");
    new vip[MAX_PLAYER_NAME], string[128],
    GetPlayerName(playerid, vip, sizeof(vip));
    
    RepairVehicle(vehicleid);
    SCM(playerid, -1, " {74FF5C}You have fixed your vehicle! You'll be able to use this command again after 3 minutes.");
    format(string, sizeof(string), "{EB4255}[VIP CMD] VIP Player %s has repaired his vehicle.", vip);
    SendManagerMessage(-1, string);
    g_RepairTime[playerid] = (gettime() + 180);
    return 1;
} 


And btw, why are you using that "pid" inside this command? There's no pid here.