[Solved]4 Errors!!! - Printable Version

[Solved]4 Errors!!! - 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]4 Errors!!! (/showthread.php?tid=101940)

[Solved]4 Errors!!! - Tigerbeast11 - 12.10.2009

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!

Re: [SERIOUS ERRORS]4 Errors!!! - Imran.Abbas - 12.10.2009

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

Re: [SERIOUS ERRORS]4 Errors!!! - Tigerbeast11 - 12.10.2009

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!

Re: [SERIOUS ERRORS]4 Errors!!! - Eazy_Efolife - 12.10.2009

Whats on line "298"?

Re: [SERIOUS ERRORS]4 Errors!!! - Jose 510 - 12.10.2009

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

Re: [SERIOUS ERRORS]4 Errors!!! - Tigerbeast11 - 12.10.2009

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...

Re: [SERIOUS ERRORS]4 Errors!!! - Peter_Corneile - 12.10.2009

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

Re: [SERIOUS ERRORS]4 Errors!!! - Tigerbeast11 - 12.10.2009

Here is the link for paste bin

http://pastebin.com/m1d5070e8

Re: [SERIOUS ERRORS]4 Errors!!! - Peter_Corneile - 12.10.2009

Change the line no "298" to this

pawn Код:

if(!killerid == INVALID_PLAYER_ID);

Re: [SERIOUS ERRORS]4 Errors!!! - Tigerbeast11 - 12.10.2009

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