[FiveCentz]

So I am making just a little repair car FS and there is some problems.

1. When you are outside of a car it says "The vehicle is too damage to repair" instead of "You must be inside a vehicle"
2. When the vehicle is under 900DL it sends the clientmessage but never repairs it.
Thanks for any help.

Код:

CMD:fixveh(playerid, params[])
{
    new Float:health;
    new veh = GetPlayerVehicleID(playerid);
    GetVehicleHealth(veh, health);
    {
		if(health > 900) return SendClientMessage(playerid, COLOR_RED, "Vehicle doesn't need repairing!");
		else if(health < 500) return SendClientMessage(playerid, COLOR_RED, "Vehicle is too damaged! Get a mechanic!");
		if(GetPlayerMoney(playerid) < 500) return SendClientMessage(playerid, COLOR_RED, "You don't have enough money! You need $500!");
		else if(GetPlayerMoney(playerid) > 500) return SendClientMessage(playerid, COLOR_GREEN, "The vehicle is now repaired."); SetVehicleHealth(veh, 1000.0); GetPlayerMoney(playerid, -500);
		return 0
	}
	IsPlayerInAnyVehicle(playerid);
	{
		if(false) return SendClientMessage(playerid, COLOR_RED, "You must be in a vehicle");
		return 0
	}
	return 1;

[Vince]

Statements are evaluated top to bottom, not in some random order. You're already checking the health before you have verified that the player is actually in a vehicle.

[FiveCentz]

What about it not fixing itself? And I will the try putting it at the top/

E:

Код:

CMD:fixveh(playerid, params[])
{
    new Float:health;
    new veh = GetPlayerVehicleID(playerid);
    IsPlayerInAnyVehicle(playerid);
	{
		if(false) return SendClientMessage(playerid, COLOR_RED, "You must be in a vehicle");
	}
    GetVehicleHealth(veh, health);
    {
		if(health > 900) return SendClientMessage(playerid, COLOR_RED, "Vehicle doesn't need repairing!");
		else if(health < 500) return SendClientMessage(playerid, COLOR_RED, "Vehicle is too damaged! Get a mechanic!");
		if(GetPlayerMoney(playerid) < 500) return SendClientMessage(playerid, COLOR_RED, "You don't have enough money! You need $500!");
		else if(GetPlayerMoney(playerid) > 500) return SendClientMessage(playerid, COLOR_GREEN, "The vehicle is now repaired."); SetVehicleHealth(veh, 1000.0); GetPlayerMoney(playerid, -500);
	}
	return 1;
}

The script is like that now, but it still didnt work.

[UltraScripter]

pawn Код:

CMD:fixveh(playerid, params[])
{
    new Float:health;
    new veh = GetPlayerVehicleID(playerid);
    GetVehicleHealth(veh, health);
    {
        if(health > 900) return SendClientMessage(playerid, COLOR_RED, "Vehicle doesn't need repairing!");
        else if(health < 500) return SendClientMessage(playerid, COLOR_RED, "Vehicle is too damaged! Get a mechanic!");
        if(GetPlayerMoney(playerid) < 500) return SendClientMessage(playerid, COLOR_RED, "You don't have enough money! You need $500!");
        else if(GetPlayerMoney(playerid) > 500) return SendClientMessage(playerid, COLOR_GREEN, "The vehicle is now repaired."); RepairVehicle(veh) GetPlayerMoney(playerid, -500);
        return 0
    }
    IsPlayerInAnyVehicle(playerid);
    {
        if(false) return SendClientMessage(playerid, COLOR_RED, "You must be in a vehicle");
        return 0
    }
    return 1;

[FiveCentz]

http://prntscr.com/59hkj0 I got errors when I did that.

[UltraScripter]

Ok Try This :

pawn Код:

CMD:fixveh(playerid, params[])
{
    new veh, Float:health;
    veh = GetPlayerVehicleID(playerid);
    GetVehicleHealth(veh, health);
    {
        if(!IsPlayerInAnyVehicle(playerid)) return SendClientMessage(playerid, 0xFF0000FF, "You must be in vehicle");
        if(health >= 900) return SendClientMessage(playerid, 0xFF0000FF, "Vehicle doesn't need repairing!");
        else if(health <= 500) return SendClientMessage(playerid, 0xFF0000FF, "Vehicle is too damaged! Get a mechanic!");
        if(GetPlayerMoney(playerid) < 500) return SendClientMessage(playerid, 0xFF0000FF, "You don't have enough money! You need $500!");
        SendClientMessage(playerid, 0x00FF00FF, "The vehicle is now repaired.");
        RepairVehicle(veh);
        GivePlayerMoney(playerid, -500);
    }
    return 1;
}

[FiveCentz]

It works! THANKS!

[UltraScripter]

No Prob Bro !.