1. SA-MP Forums Archive
  2. SA-MP Scripting and Plugins
  3. Scripting Help
  4. Help Archive

Threaded Mode
[Solved]4 Errors!!!
Tigerbeast11
Gangsta
*****
Posts: 771
Threads: 135
Joined: Aug 2009
Reputation: 0
#1

12.10.2009, 20:27
Here is the script...
Код:
public OnPlayerDeath(playerid, killerid, reason)
{
	SendDeathMessage(killerid, playerid, reason);
	
	if(!killerid = INVALID_PLAYER_ID); //Line 298 where the errors are...
	{
	  if(PlayerKills(playerid) >= 3 && PlayerKills[playerid] <= 14)  )
      	new PlayerName[MAX_PLAYER_NAME],
				string[128];
		 		GetPlayerName(playerid, PlayerName, sizeof(PlayerName));
		 		format(string, sizeof(string), "[Spree]%s killing spree is now over", PlayerName);
		 		SendClientMessageToAll(COLOR_YELLOW, string);
		 		GivePlayerMoney(killerid, 750);
      }
      if(PlayerKills[playerid] >= 15 && PlayerKills[playerid] <= 44) {
      	new PlayerName[MAX_PLAYER_NAME],
				string[128];
		 		GetPlayerName(playerid, PlayerName, sizeof(PlayerName));
		 		format(string, sizeof(string), "[Spree]%s killing spree is now over", PlayerName);
		 		SendClientMessageToAll(COLOR_YELLOW, string);
		 		GivePlayerMoney(killerid, 1500);
      }
      if(PlayerKills[playerid] >= 45) {
      	new PlayerName[MAX_PLAYER_NAME],
				string[128];
		 		GetPlayerName(playerid, PlayerName, sizeof(PlayerName));
		 		format(string, sizeof(string), "[Spree]%s killing spree is now over", PlayerName);
		 		SendClientMessageToAll(COLOR_YELLOW, string);
		 		GivePlayerMoney(killerid, 2500);
      }

        PlayerKills[playerid] = 0; //Set the murdered's kill count to 0
        PlayerKills[killerid]++; //Set the killer's kill count +1

        if(PlayerKills[killerid] == 3) {
        SendClientMessage(playerid, COLOR_YELLOW,"You are now on a killing spree!");
        new PlayerName[MAX_PLAYER_NAME],
			 	string[128];
			 	GetPlayerName(playerid, PlayerName, sizeof(PlayerName));
			 	format(string, sizeof(string), "[Spree]%s is now on a killing spree", PlayerName);
			 	SendClientMessageToAll(COLOR_YELLOW, string);
			 	GivePlayerMoney(playerid, 450);
        }
        if(PlayerKills[killerid] == 15) {
        SendClientMessage(playerid, COLOR_YELLOW,"You have gotten 15 kills! You have received a Minigun!");
        new PlayerName[MAX_PLAYER_NAME],
			 	string[128];
			 	GetPlayerName(playerid, PlayerName, sizeof(PlayerName));
			 	format(string, sizeof(string), "[Spree]%s has been awarded a Minigun for getting 15 kills!", PlayerName);
			 	SendClientMessageToAll(COLOR_YELLOW, string);
			 	GivePlayerWeapon(playerid, 38, 3000);
			 	GivePlayerMoney(playerid, 1500);
        }
        if(PlayerKills[killerid] == 45) {
				SendClientMessage(playerid, COLOR_YELLOW,"You have gotten 45 kills! You can now use /shop and /godcar!");
        new PlayerName[MAX_PLAYER_NAME],
			 	string[128];
			 	GetPlayerName(playerid, PlayerName, sizeof(PlayerName));
			 	format(string, sizeof(string), "[Spree]%s has gotten 45 kills and is now able to access /godcar and /shop", PlayerName);
			 	SendClientMessageToAll(COLOR_YELLOW, string);
			 	GivePlayerMoney(playerid, 2500);
        }
	return 1;
}
Here are the 4 errors...
Код:
\DEATHM~1.PWN(298) : warning 211: possibly unintended assignment
\DEATHM~1.PWN(298) : error 022: must be lvalue (non-constant)
\DEATHM~1.PWN(298) : warning 215: expression has no effect
\DEATHM~1.PWN(298) : error 001: expected token: ";", but found ")"
\DEATHM~1.PWN(298) : error 029: invalid expression, assumed zero
\DEATHM~1.PWN(298) : fatal error 107: too many error messages on one line
Im a newbie so I dont know what the problem is... Plz if you know what I have done wrong, plz tell me to correct it!
Find
Reply
Imran.Abbas
Huge Clucker
****
Posts: 342
Threads: 9
Joined: Aug 2009
Reputation: 0
#2

12.10.2009, 20:30
try

Код:
public OnPlayerDeath(playerid, killerid, reason)
{
	SendDeathMessage(killerid, playerid, reason);
	
	if(!killerid = INVALID_PLAYER_ID);
	{
	  if(PlayerKills(playerid) >= 3 && PlayerKills[playerid] <= 14)  )
      	new PlayerName[MAX_PLAYER_NAME];
				string[128];
		 		GetPlayerName(playerid, PlayerName, sizeof(PlayerName));
		 		format(string, sizeof(string), "[Spree]%s killing spree is now over", PlayerName);
		 		SendClientMessageToAll(COLOR_YELLOW, string);
		 		GivePlayerMoney(killerid, 750);
      }
      if(PlayerKills[playerid] >= 15 && PlayerKills[playerid] <= 44) {
      	new PlayerName[MAX_PLAYER_NAME];
				string[128];
		 		GetPlayerName(playerid, PlayerName, sizeof(PlayerName));
		 		format(string, sizeof(string), "[Spree]%s killing spree is now over", PlayerName);
		 		SendClientMessageToAll(COLOR_YELLOW, string);
		 		GivePlayerMoney(killerid, 1500);
      }
      if(PlayerKills[playerid] >= 45) {
      	new PlayerName[MAX_PLAYER_NAME];
				string[128];
		 		GetPlayerName(playerid, PlayerName, sizeof(PlayerName));
		 		format(string, sizeof(string), "[Spree]%s killing spree is now over", PlayerName);
		 		SendClientMessageToAll(COLOR_YELLOW, string);
		 		GivePlayerMoney(killerid, 2500);
      }

        PlayerKills[playerid] = 0; //Set the murdered's kill count to 0
        PlayerKills[killerid]++; //Set the killer's kill count +1

        if(PlayerKills[killerid] == 3) {
        SendClientMessage(playerid, COLOR_YELLOW,"You are now on a killing spree!");
        new PlayerName[MAX_PLAYER_NAME];
			 	string[128];
			 	GetPlayerName(playerid, PlayerName, sizeof(PlayerName));
			 	format(string, sizeof(string), "[Spree]%s is now on a killing spree", PlayerName);
			 	SendClientMessageToAll(COLOR_YELLOW, string);
			 	GivePlayerMoney(playerid, 450);
        }
        if(PlayerKills[killerid] == 15) {
        SendClientMessage(playerid, COLOR_YELLOW,"You have gotten 15 kills! You have received a Minigun!");
        new PlayerName[MAX_PLAYER_NAME];
			 	string[128];
			 	GetPlayerName(playerid, PlayerName, sizeof(PlayerName));
			 	format(string, sizeof(string), "[Spree]%s has been awarded a Minigun for getting 15 kills!", PlayerName);
			 	SendClientMessageToAll(COLOR_YELLOW, string);
			 	GivePlayerWeapon(playerid, 38, 3000);
			 	GivePlayerMoney(playerid, 1500);
        }
        if(PlayerKills[killerid] == 45) {
				SendClientMessage(playerid, COLOR_YELLOW,"You have gotten 45 kills! You can now use /shop and /godcar!");
        new PlayerName[MAX_PLAYER_NAME];
			 	string[128];
			 	GetPlayerName(playerid, PlayerName, sizeof(PlayerName));
			 	format(string, sizeof(string), "[Spree]%s has gotten 45 kills and is now able to access /godcar and /shop", PlayerName);
			 	SendClientMessageToAll(COLOR_YELLOW, string);
			 	GivePlayerMoney(playerid, 2500);
        }
	return 1;
}
Find
Reply
Tigerbeast11
Gangsta
*****
Posts: 771
Threads: 135
Joined: Aug 2009
Reputation: 0
#3

12.10.2009, 20:33
Quote:
Originally Posted by Imran.Abbas[Agent
]
try

Код:
public OnPlayerDeath(playerid, killerid, reason)
{
	SendDeathMessage(killerid, playerid, reason);
	
	if(!killerid = INVALID_PLAYER_ID);
	{
	  if(PlayerKills(playerid) >= 3 && PlayerKills[playerid] <= 14) )
      	new PlayerName[MAX_PLAYER_NAME];
				string[128];
		 		GetPlayerName(playerid, PlayerName, sizeof(PlayerName));
		 		format(string, sizeof(string), "[Spree]%s killing spree is now over", PlayerName);
		 		SendClientMessageToAll(COLOR_YELLOW, string);
		 		GivePlayerMoney(killerid, 750);
      }
      if(PlayerKills[playerid] >= 15 && PlayerKills[playerid] <= 44) {
      	new PlayerName[MAX_PLAYER_NAME];
				string[128];
		 		GetPlayerName(playerid, PlayerName, sizeof(PlayerName));
		 		format(string, sizeof(string), "[Spree]%s killing spree is now over", PlayerName);
		 		SendClientMessageToAll(COLOR_YELLOW, string);
		 		GivePlayerMoney(killerid, 1500);
      }
      if(PlayerKills[playerid] >= 45) {
      	new PlayerName[MAX_PLAYER_NAME];
				string[128];
		 		GetPlayerName(playerid, PlayerName, sizeof(PlayerName));
		 		format(string, sizeof(string), "[Spree]%s killing spree is now over", PlayerName);
		 		SendClientMessageToAll(COLOR_YELLOW, string);
		 		GivePlayerMoney(killerid, 2500);
      }

        PlayerKills[playerid] = 0; //Set the murdered's kill count to 0
        PlayerKills[killerid]++; //Set the killer's kill count +1

        if(PlayerKills[killerid] == 3) {
        SendClientMessage(playerid, COLOR_YELLOW,"You are now on a killing spree!");
        new PlayerName[MAX_PLAYER_NAME];
			 	string[128];
			 	GetPlayerName(playerid, PlayerName, sizeof(PlayerName));
			 	format(string, sizeof(string), "[Spree]%s is now on a killing spree", PlayerName);
			 	SendClientMessageToAll(COLOR_YELLOW, string);
			 	GivePlayerMoney(playerid, 450);
        }
        if(PlayerKills[killerid] == 15) {
        SendClientMessage(playerid, COLOR_YELLOW,"You have gotten 15 kills! You have received a Minigun!");
        new PlayerName[MAX_PLAYER_NAME];
			 	string[128];
			 	GetPlayerName(playerid, PlayerName, sizeof(PlayerName));
			 	format(string, sizeof(string), "[Spree]%s has been awarded a Minigun for getting 15 kills!", PlayerName);
			 	SendClientMessageToAll(COLOR_YELLOW, string);
			 	GivePlayerWeapon(playerid, 38, 3000);
			 	GivePlayerMoney(playerid, 1500);
        }
        if(PlayerKills[killerid] == 45) {
				SendClientMessage(playerid, COLOR_YELLOW,"You have gotten 45 kills! You can now use /shop and /godcar!");
        new PlayerName[MAX_PLAYER_NAME];
			 	string[128];
			 	GetPlayerName(playerid, PlayerName, sizeof(PlayerName));
			 	format(string, sizeof(string), "[Spree]%s has gotten 45 kills and is now able to access /godcar and /shop", PlayerName);
			 	SendClientMessageToAll(COLOR_YELLOW, string);
			 	GivePlayerMoney(playerid, 2500);
        }
	return 1;
}
Same 4 errors Really need help!
Find
Reply
Eazy_Efolife
High-roller
*****
Posts: 1,141
Threads: 56
Joined: Aug 2009
Reputation: 0
#4

12.10.2009, 20:33
Whats on line "298"?
Find
Reply
Jose 510
Huge Clucker
***
Posts: 212
Threads: 11
Joined: Jun 2008
Reputation: 0
#5

12.10.2009, 20:35
Quote:
Originally Posted by Tigerbeast11
Quote:
Originally Posted by Imran.Abbas[Agent
]
try

Код:
public OnPlayerDeath(playerid, killerid, reason)
{
	SendDeathMessage(killerid, playerid, reason);
	
	if(!killerid = INVALID_PLAYER_ID);
	{
	  if(PlayerKills(playerid) >= 3 && PlayerKills[playerid] <= 14)  )
      	new PlayerName[MAX_PLAYER_NAME];
				string[128];
		 		GetPlayerName(playerid, PlayerName, sizeof(PlayerName));
		 		format(string, sizeof(string), "[Spree]%s killing spree is now over", PlayerName);
		 		SendClientMessageToAll(COLOR_YELLOW, string);
		 		GivePlayerMoney(killerid, 750);
      }
      if(PlayerKills[playerid] >= 15 && PlayerKills[playerid] <= 44) {
      	new PlayerName[MAX_PLAYER_NAME];
				string[128];
		 		GetPlayerName(playerid, PlayerName, sizeof(PlayerName));
		 		format(string, sizeof(string), "[Spree]%s killing spree is now over", PlayerName);
		 		SendClientMessageToAll(COLOR_YELLOW, string);
		 		GivePlayerMoney(killerid, 1500);
      }
      if(PlayerKills[playerid] >= 45) {
      	new PlayerName[MAX_PLAYER_NAME];
				string[128];
		 		GetPlayerName(playerid, PlayerName, sizeof(PlayerName));
		 		format(string, sizeof(string), "[Spree]%s killing spree is now over", PlayerName);
		 		SendClientMessageToAll(COLOR_YELLOW, string);
		 		GivePlayerMoney(killerid, 2500);
      }

        PlayerKills[playerid] = 0; //Set the murdered's kill count to 0
        PlayerKills[killerid]++; //Set the killer's kill count +1

        if(PlayerKills[killerid] == 3) {
        SendClientMessage(playerid, COLOR_YELLOW,"You are now on a killing spree!");
        new PlayerName[MAX_PLAYER_NAME];
			 	string[128];
			 	GetPlayerName(playerid, PlayerName, sizeof(PlayerName));
			 	format(string, sizeof(string), "[Spree]%s is now on a killing spree", PlayerName);
			 	SendClientMessageToAll(COLOR_YELLOW, string);
			 	GivePlayerMoney(playerid, 450);
        }
        if(PlayerKills[killerid] == 15) {
        SendClientMessage(playerid, COLOR_YELLOW,"You have gotten 15 kills! You have received a Minigun!");
        new PlayerName[MAX_PLAYER_NAME];
			 	string[128];
			 	GetPlayerName(playerid, PlayerName, sizeof(PlayerName));
			 	format(string, sizeof(string), "[Spree]%s has been awarded a Minigun for getting 15 kills!", PlayerName);
			 	SendClientMessageToAll(COLOR_YELLOW, string);
			 	GivePlayerWeapon(playerid, 38, 3000);
			 	GivePlayerMoney(playerid, 1500);
        }
        if(PlayerKills[killerid] == 45) {
				SendClientMessage(playerid, COLOR_YELLOW,"You have gotten 45 kills! You can now use /shop and /godcar!");
        new PlayerName[MAX_PLAYER_NAME];
			 	string[128];
			 	GetPlayerName(playerid, PlayerName, sizeof(PlayerName));
			 	format(string, sizeof(string), "[Spree]%s has gotten 45 kills and is now able to access /godcar and /shop", PlayerName);
			 	SendClientMessageToAll(COLOR_YELLOW, string);
			 	GivePlayerMoney(playerid, 2500);
        }
	return 1;
}
Same 4 errors Really need help!
Can you just post the script on pastebin and then remove it when it is fixed because you really aren't showing what's on line 298
Find
Reply
Tigerbeast11
Gangsta
*****
Posts: 771
Threads: 135
Joined: Aug 2009
Reputation: 0
#6

12.10.2009, 20:35
Quote:
Originally Posted by Compton's Eazy E
Whats on line "298"?
I dont know... Im a newbie

Its right in front of u
And by the way, i have defined the PlayerKills...
Find
Reply
Peter_Corneile
Gangsta
*****
Posts: 870
Threads: 26
Joined: Mar 2009
Reputation: 0
#7

12.10.2009, 20:36
Quote:
Originally Posted by Tigerbeast11
Quote:
Originally Posted by Compton's Eazy E
Whats on line "298"?
I dont know... Im a newbie

Its right in front of u
And by the way, i have defined the PlayerKills...
Just tell us the line "298" .. Just write(copy paste) the code so that we know te line of error
Find
Reply
Tigerbeast11
Gangsta
*****
Posts: 771
Threads: 135
Joined: Aug 2009
Reputation: 0
#8

12.10.2009, 20:37
Here is the link for paste bin

http://pastebin.com/m1d5070e8
Find
Reply
Peter_Corneile
Gangsta
*****
Posts: 870
Threads: 26
Joined: Mar 2009
Reputation: 0
#9

12.10.2009, 20:38
Change the line no "298" to this
pawn Код:
if(!killerid == INVALID_PLAYER_ID);
Find
Reply
Tigerbeast11
Gangsta
*****
Posts: 771
Threads: 135
Joined: Aug 2009
Reputation: 0
#10

12.10.2009, 20:40
Quote:
Originally Posted by Peter Corneile [hugu-hosting.co.uk
]
Change the line no "298" to this
pawn Код:
if(!killerid == INVALID_PLAYER_ID);
Now 5 errors
Код:
\DEATHM~1.PWN(298) : warning 217: loose indentation
\DEATHM~1.PWN(298) : warning 213: tag mismatch
\DEATHM~1.PWN(298) : error 036: empty statement
\DEATHM~1.PWN(300) : error 012: invalid function call, not a valid address
\DEATHM~1.PWN(300) : warning 215: expression has no effect
\DEATHM~1.PWN(300) : error 001: expected token: ";", but found ")"
\DEATHM~1.PWN(300) : error 029: invalid expression, assumed zero
\DEATHM~1.PWN(300) : fatal error 107: too many error messages on one line
Find
Reply
« Next Oldest | Next Newest »


Forum Jump:


Users browsing this thread: 2 Guest(s)
  1. SA-MP Forums Archive
  2. SA-MP Scripting and Plugins
  3. Scripting Help
  4. Help Archive