if (strcmp("/ak47", cmdtext, true, 10) == 0)
{
if((GetPlayerMoney(playerid) >= 100)
{ // this is line 164
SendClientMessage(playerid, 0xFF0000AA,"You bought an Ak47");
GivePlayerWeapon(playerid, 30, 1000);
GivePlayerMoney(playerid, -100);
}
else
{
if(GetPlayerMoney(playerid) < 100)
{
SendClientMessage(playerid, 0xFF0000AA,"You don't have enough money");
}
return 1;
}
if (strcmp("/ak47", cmdtext, true, 10) == 0)
{
if( GetPlayerMoney(playerid) < 100) return SendClientMessage(playerid, COLOR_RED, "You don't have enough money to buy an AK-47!");
GivePlayerMoney(playerid, -100);
GivePlayerWeapon(playerid, 30, 1000);
SendClientMessage(playerid, COLOR_GREEN, "WEAPON SHOP : You have succesfully bought an AK-47!");
return 1;
}
if (strcmp("/ak47", cmdtext, true, 10) == 0) { if(GetPlayerMoney(playerid) < 100) return SendClientMessage(playerid, 0xFF0000AA,"You don't have enough money"); SendClientMessage(playerid, 0xFF0000AA,"You bought an Ak47"); GivePlayerWeapon(playerid, 30, 1000); GivePlayerMoney(playerid, -100); return 1; } |
if((GetPlayerMoney(playerid) >= 100) //wrong a extra ( bracket here
//this should be
if(GetPlayerMoney(playerid) >= 100)
if (strcmp("/ak47", cmdtext, true, 10) == 0)
{
if(GetPlayerMoney(playerid) >= 100)//a extra ( bracket here
{
SendClientMessage(playerid, 0xFF0000AA,"You bought an Ak47");
GivePlayerWeapon(playerid, 30, 1000);
GivePlayerMoney(playerid, -100);
}
else
{
if(GetPlayerMoney(playerid) < 100)
{
SendClientMessage(playerid, 0xFF0000AA,"You don't have enough money");
}
}
if (strcmp("/ak47", cmdtext, true, 10) == 0) { if(GetPlayerMoney(playerid) >= 100)//a extra ( bracket here { // this is line 164 SendClientMessage(playerid, 0xFF0000AA,"You bought an Ak47"); GivePlayerWeapon(playerid, 30, 1000); GivePlayerMoney(playerid, -100); } else { if(GetPlayerMoney(playerid) < 100) { SendClientMessage(playerid, 0xFF0000AA,"You don't have enough money"); } }
if (strcmp("/ak47", cmdtext, true, 10) == 0) { if(GetPlayerMoney(playerid) < 100) return SendClientMessage(playerid, 0xFF0000AA,"You don't have enough money"); SendClientMessage(playerid, 0xFF0000AA,"You bought an Ak47"); GivePlayerWeapon(playerid, 30, 1000); GivePlayerMoney(playerid, -100); } }
if(GetPlayerMoney(playerid) < 100) return SendClientMessage(playerid, 0xFF0000AA,"You don't have enough money"); |
Sorry for offtopic , but can i learn you something shabi?
why would you do it like this Код:
if (strcmp("/ak47", cmdtext, true, 10) == 0) { if(GetPlayerMoney(playerid) >= 100)//a extra ( bracket here { // this is line 164 SendClientMessage(playerid, 0xFF0000AA,"You bought an Ak47"); GivePlayerWeapon(playerid, 30, 1000); GivePlayerMoney(playerid, -100); } else { if(GetPlayerMoney(playerid) < 100) { SendClientMessage(playerid, 0xFF0000AA,"You don't have enough money"); } } Код:
if (strcmp("/ak47", cmdtext, true, 10) == 0) { if(GetPlayerMoney(playerid) < 100) return SendClientMessage(playerid, 0xFF0000AA,"You don't have enough money"); SendClientMessage(playerid, 0xFF0000AA,"You bought an Ak47"); GivePlayerWeapon(playerid, 30, 1000); GivePlayerMoney(playerid, -100); } } 3rd line : If playermoney is less than 100$ , return send that message to player.. else , code send's him message that he bought ak47 , gives him weapon and charge him.. OT: if((GetPlayerMoney(playerid) >= 100) you don't need 2 bracket here, just one, and the second thing that i bold , it says if playermoney is higher or equal to 100 , so you do it like this if(GetPlayerMoney(playerid) < 100) return SendClientMessage(playerid,-1,"You don't have enought money"); just read the code, if player money is less than 100 , code will return him message "you don't have enought money.." |
But you didn't type else (cough im embarresed right now)
[EDIT]: Your script did work, but when i typed /ak47, and i had 0 money it said "You don't have enough money!" "You just bought an ak400" And i had an ak400 with -100 money, so basicly your script doesn't work :P |
if (strcmp("/ak47", cmdtext, true, 5) == 0)
{
if(GetPlayerMoney(playerid) < 100) SendClientMessage(playerid, 0xFF0000AA,"You don't have enough money to buy an AK47!");
else
{
GivePlayerWeapon(playerid, 30, 1000);
GivePlayerMoney(playerid, -100);
SendClientMessage(playerid, 0xFF0000AA,"You've bought an AK47!");
}
}