[pmk1]

Hi guys! What's up? I've made a fix car command so when i type /fix , my car gets fixed for 1000 $ for no problem. But, When i Get Under 1000$ , I Added A Line which should not let the player fix the car, but it just don't work. It only send a Clientmessage i made to say that you're too poor to fix it, but the car gets fixed anyway, and the money goes under 0. I'd like to Make it work. Can Someone Tell me What's Wrong? here's The Script:

if (!strcmp("/fix", cmdtext))
{
if(!IsPlayerInAnyVehicle(playerid)) return SendClientMessage(playerid, COLOR_BRIGHTRED, "You're Not In A Vehicle! What Do You Want To Repair?");
RepairVehicle(GetPlayerVehicleID(playerid));
GivePlayerMoney(playerid, -1000);
SendClientMessage(playerid, COLOR_SKYBLUE, "You Fixed Your Car For 1000$");
if(GetPlayerMoney(playerid) < 999) return SendClientMessage(playerid, COLOR_YELLOW, "Your Car Can't Be Fixed Because You're Too Poor. R.I.P If It's In Fire.");
return 1;
}



Thanks!

[Torran]

Try,

pawn Код:

if (!strcmp("/fix", cmdtext))
  {
      if(GetPlayerMoney(playerid) < 999) return SendClientMessage(playerid, COLOR_YELLOW, "Your Car Can't Be Fixed Because You're Too Poor. R.I.P If It's In Fire.");
    elseif(!IsPlayerInAnyVehicle(playerid)) return SendClientMessage(playerid, COLOR_BRIGHTRED, "You're Not In A Vehicle! What Do You Want To Repair?");
else
{
    RepairVehicle(GetPlayerVehicleID(playerid));
    GivePlayerMoney(playerid, -1000);
    SendClientMessage(playerid, COLOR_SKYBLUE, "You Fixed Your Car For 1000$");
    }
  return 1;
  }

[aircombat]

Код:

if (!strcmp("/fix", cmdtext))
  {
    if(GetPlayerMoney(playerid) < 999) return SendClientMessage(playerid, COLOR_YELLOW, "Your Car Can't Be Fixed Because You're Too Poor. R.I.P If It's In Fire.");
    if(!IsPlayerInAnyVehicle(playerid)) return SendClientMessage(playerid, COLOR_BRIGHTRED, "You're Not In A Vehicle! What Do You Want To Repair?");
    else {
    RepairVehicle(GetPlayerVehicleID(playerid));
    GivePlayerMoney(playerid, -1000);
    SendClientMessage(playerid, COLOR_SKYBLUE, "You Fixed Your Car For 1000$");
    }
    return 1;
  }

Edit : Torren Stop Posting Such Fast :P
________
Washington dispensaries

[pmk1]

Ok thanks it worked! But can you help me with another thing plz? how can i make that if the car has already 100 health, it can't be fixed? To not waste money for a already fixed car. Thanks!

[aircombat]

try that :

Код:

if (!strcmp("/fix", cmdtext))
{
      new Float:health;
	 new veh;
	 veh = GetPlayerVehicleID(playerid);
	 GetVehicleHealth(veh, health);
	 if(GetPlayerMoney(playerid) < 999) return SendClientMessage(playerid, COLOR_YELLOW, "Your Car Can't Be Fixed Because You're Too Poor. R.I.P If It's In Fire.");
      else if(!IsPlayerInAnyVehicle(playerid)) return SendClientMessage(playerid, COLOR_BRIGHTRED, "You're Not In A Vehicle! What Do You Want To Repair?");
      else if(health > 1000) return SendClientMessage(playerid,COLOR_BRIGHTRED,"Doesn't need repairing!");
	 else
  	 {
  	 RepairVehicle(GetPlayerVehicleID(playerid));
    	 GivePlayerMoney(playerid, -1000);
    	 SendClientMessage(playerid, COLOR_SKYBLUE, "You Fixed Your Car For 1000$");
	 } 
      return 1;
}

________
DEPRESSION FORUMS

[pmk1]

Hmm. Nah it didnt work :\ Still having the same problem

[aircombat]

try replacing else if(health > 1000) return SendClientMessage(playerid,COLOR_BRIGHTRED,"Doesn' t need repairing!");
with
else if(health >= 1000) return SendClientMessage(playerid,COLOR_BRIGHTRED,"Doesn' t need repairing!");
________
Nevada medical marijuana dispensary

[pmk1]

Yea that's great! It worked! Thanks to you

[aircombat]

Quote:

Originally Posted by pmk1 Yea that's great! It worked! Thanks to you

anytime
________
NEW JERSEY MARIJUANA DISPENSARIES

[tuuler]

easy way

if (strcmp("/r", cmdtext, true, 10) == 0)
{
RepairVehicle(GetPlayerVehicleID(playerid));
return 1;
}