[SOLVED] Fix command problems :\

[SOLVED] Fix command problems :\ - Printable Version

+- SA-MP Forums Archive (https://sampforum.blast.hk)
+-- Forum: SA-MP Scripting and Plugins (https://sampforum.blast.hk/forumdisplay.php?fid=8)
+--- Forum: Scripting Help (https://sampforum.blast.hk/forumdisplay.php?fid=12)
+---- Forum: Help Archive (https://sampforum.blast.hk/forumdisplay.php?fid=89)
+---- Thread: [SOLVED] Fix command problems :\ (/showthread.php?tid=132262)

[SOLVED] Fix command problems :\ - pmk1 - 07.03.2010

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!

Re: I'd need some help with a problem :\ - Torran - 07.03.2010

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;
  }

Re: I'd need some help with a problem :\ - aircombat - 07.03.2010

Код:

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

Re: I'd need some help with a problem :\ - pmk1 - 07.03.2010

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!

Re: I'd need some help with a problem :\ - aircombat - 07.03.2010

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

Re: I'd need some help with a problem :\ - pmk1 - 07.03.2010

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

Re: I'd need some help with a problem :\ - aircombat - 07.03.2010

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

Re: I'd need some help with a problem :\ - pmk1 - 07.03.2010

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

Re: [SOLVED] Fix command problems :\ - aircombat - 07.03.2010

Quote:

Originally Posted by pmk1

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

anytime

________
NEW JERSEY MARIJUANA DISPENSARIES

Re: [SOLVED] Fix command problems :\ - tuuler - 23.08.2011

easy way

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