[UnknownGamer]

pawn Код:

if(strcmp(cmd, "/vhealth", true) == 0)
    {
        if(IsPlayerConnected(playerid))
        {
            
            new carid;
            carid = GetPlayerVehicleID(playerid);

            if(IsPlayerInAnyVehicle(playerid))
            {
            if(IsAnOwnableCar(carid))
            {
            new Float:health;
            GetVehicleHealth(carid, health);
            if(health <350) return SendClientMessage(playerid,COLOR_LIGHTRED,"Vehicle Health: Damaged!!");
            if(health >351) return SendClientMessage(playerid,COLOR_GREEN,"Vehicle Health: Good!!");
            }
            else
            {
            SendClientMessage(playerid, COLOR_GREY, "* You are not in a vehicle!");
            return 1;
        }
        return 1;
    }
}
}

Problem is, It doesn't say You are not in a vehicle.

Please could someone check this, and fix for me?

Thankyou.

[Cypress]

pawn Код:

if(strcmp(cmd, "/vhealth", true) == 0)
{
        if(IsPlayerConnected(playerid))
        {
            new carid;
            carid = GetPlayerVehicleID(playerid);

            if(!IsPlayerInAnyVehicle(playerid)) return SendClientMessage(playerid, COLOR_GREY, "* You are not in a vehicle!");
           
            if(IsAnOwnableCar(carid))
            {
               new Float:health;
               GetVehicleHealth(carid, health);
               if(health <350) return SendClientMessage(playerid,COLOR_LIGHTRED,"Vehicle Health: Damaged!!");
               if(health >351) return SendClientMessage(playerid,COLOR_GREEN,"Vehicle Health: Good!!");
            }
            return 1;
        }
        return 1;
}

[new121]

You are trying to replicate another servers script but you can't even grasp proper indentation?

[UnknownGamer]

Quote:

Originally Posted by new121 You are trying to replicate another servers script but you can't even grasp proper indentation?

Why do you think this is a copy of HBG again,

Nothing of it, I made a command to tell users if the damage of the vehicle.

(HBG have on-screen thing)

[mamorunl]

Quote:

Originally Posted by Cypress pawn Код: if(strcmp(cmd, "/vhealth", true) == 0) {         if(IsPlayerConnected(playerid))         {             new carid;             carid = GetPlayerVehicleID(playerid);             if(!IsPlayerInAnyVehicle(playerid)) return SendClientMessage(playerid, COLOR_GREY, "* You are not in a vehicle!");                         if(IsAnOwnableCar(carid))             {                new Float:health;                GetVehicleHealth(carid, health);                if(health <350) return SendClientMessage(playerid,COLOR_LIGHTRED,"Vehicle Health: Damaged!!");                if(health >351) return SendClientMessage(playerid,COLOR_GREEN,"Vehicle Health: Good!!");             }             return 1;         }         return 1; }

Even though this code is good, I would like to point out a few things. The GetVehicleID stuff, should also be inside the isPlayerinAnyVehicle function call. Also, I'd check if the player is NOT connected. It saves you a lot of code and thus making your code more maintainable.

[Seven_of_Nine]

Problem should be here:

Код:

IsAnOwnableCar(carid)

Also, I don't know why'd you check for IsPlayerConnected..

[Dark_Kostas]

pawn Код:

if(strcmp(cmd, "/vhealth", true) == 0)
{
    new carid = GetPlayerVehicleID(playerid);
    if(carid == 0) return SendClientMessage(playerid, COLOR_GREY, "* You are not in a vehicle!");
    if(!IsAnOwnableCar(carid)) return 1;

    new Float:health;
    GetVehicleHealth(carid, health);

    if(health <= 350) return SendClientMessage(playerid,COLOR_LIGHTRED,"Vehicle Health: Damaged!!");
    else return SendClientMessage(playerid,COLOR_GREEN,"Vehicle Health: Good!!");
}

Also at your previous code if health was 350 or 351 it wouldn't say anything.
Smaller and organised.

[clarencecuzz]

pawn Код:

if(strcmp(cmd, "/vhealth", true) == 0)
{
    if(!IsPlayerInAnyVehicle(playerid)) return SendClientMessage(playerid, COLOR_GREY, "* You are not in a vehicle!");
    if(!IsAnOwnableCar(carid)) return 1;

    new carid = GetPlayerVehicleID(playerid);
    new Float:health;
    GetVehicleHealth(carid, health);

    if(health <= 350)
    {
        SendClientMessage(playerid,0xFF0000FF,"Vehicle Health: Damaged!!");
    }
    else return SendClientMessage(playerid,0x00FF00FF,"Vehicle Health: Good!!");
    return 1;
}