[fiki574]

Hi! So, I got 3 errors with this script:

Код:

public OnPlayerUpdate(playerid)
{
	if(GetPlayerWeapon(playerid) == 35);
	{
	    Ban(playerid);
	    new string[256], pname[MAX_PLAYER_NAME];
	    GetPlayerName(playerid, pname, sizeof(pname));
	    format(string, sizeof(string), "%s been banned for using forbidden weapon (RPG)", pname);
	    SendClientMessageToAll(orange, string);
	}
	if(GetPlayerWeapon(playerid) == 36);
	{
	    Ban(playerid);
	    new string[256], pname[MAX_PLAYER_NAME];
	    GetPlayerName(playerid, pname, sizeof(pname));
	    format(string, sizeof(string), "%s been banned for using forbidden weapon (Missile Launcher)", pname);
	    SendClientMessageToAll(orange, string);
	}
	if(GetPlayerWeapon(playerid) == 38);
	{
	    Ban(playerid);
	    new string[256], pname[MAX_PLAYER_NAME];
	    GetPlayerName(playerid, pname, sizeof(pname));
	    format(string, sizeof(string), "%s been banned for using forbidden weapon (Minigun)", pname);
	    SendClientMessageToAll(orange, string);
	}
	return 1;
}

I got this errors:

Код:

C:\Users\Bruno\Downloads\GTA San Andreas\SAMP Server\pawno\ac.pwn(192) : error 036: empty statement
C:\Users\Bruno\Downloads\GTA San Andreas\SAMP Server\pawno\ac.pwn(200) : error 036: empty statement
C:\Users\Bruno\Downloads\GTA San Andreas\SAMP Server\pawno\ac.pwn(208) : error 036: empty statement
Pawn compiler 3.2.3664	 	 	Copyright © 1997-2006, ITB CompuPhase


3 Errors.

Can someone help me with this PLS?

[Steve M.]

Here you go, this should work:

Код:

public OnPlayerUpdate(playerid)
{
	if(GetPlayerWeapon(playerid) == 35)
	{
	    Ban(playerid);
	    new string[256], pname[MAX_PLAYER_NAME];
	    GetPlayerName(playerid, pname, sizeof(pname));
	    format(string, sizeof(string), "%s been banned for using forbidden weapon (RPG)", pname);
	    SendClientMessageToAll(orange, string);
	}
	else if(GetPlayerWeapon(playerid) == 36)
	{
	    Ban(playerid);
	    new string[256], pname[MAX_PLAYER_NAME];
	    GetPlayerName(playerid, pname, sizeof(pname));
	    format(string, sizeof(string), "%s been banned for using forbidden weapon (Missile Launcher)", pname);
	    SendClientMessageToAll(orange, string);
	}
	else if(GetPlayerWeapon(playerid) == 38)
	{
	    Ban(playerid);
	    new string[256], pname[MAX_PLAYER_NAME];
	    GetPlayerName(playerid, pname, sizeof(pname));
	    format(string, sizeof(string), "%s been banned for using forbidden weapon (Minigun)", pname);
	    SendClientMessageToAll(orange, string);
	}
	return 1;
}

Why this didn't work is because you putted ; on the end of if() statement and that doesn't go there.

[GangsTa_]

Tell us the lines that were catched while on errors.

[fiki574]

Quote:

Originally Posted by Steve M. Here you go, this should work: Код: public OnPlayerUpdate(playerid) { if(GetPlayerWeapon(playerid) == 35) { Ban(playerid); new string[256], pname[MAX_PLAYER_NAME]; GetPlayerName(playerid, pname, sizeof(pname)); format(string, sizeof(string), "%s been banned for using forbidden weapon (RPG)", pname); SendClientMessageToAll(orange, string); } else if(GetPlayerWeapon(playerid) == 36) { Ban(playerid); new string[256], pname[MAX_PLAYER_NAME]; GetPlayerName(playerid, pname, sizeof(pname)); format(string, sizeof(string), "%s been banned for using forbidden weapon (Missile Launcher)", pname); SendClientMessageToAll(orange, string); } else if(GetPlayerWeapon(playerid) == 38) { Ban(playerid); new string[256], pname[MAX_PLAYER_NAME]; GetPlayerName(playerid, pname, sizeof(pname)); format(string, sizeof(string), "%s been banned for using forbidden weapon (Minigun)", pname); SendClientMessageToAll(orange, string); } return 1; } Why this didn't work is because you putted ; on the end of if() statement and that doesn't go there.

Works! TNX!

[K9-]

put this line "Ban (playerid);" to the last one, because it it goes baned before check if I named

[KeyWay]

You also dont want to use OnPlayerUpdate, as SA-MP calls it like 40 times a second!

Make own callback for ex.

pawn Код:

forward OnUpdatePlayer(playerid);
public OnUpdatePlayer(playerid)
{
          return1;
}

[BigETI]

Darn use timers for it instead of that damn OnPlayerUpdate (I still use OnPlayerUpdate only for my afk system)

[black_dota]

LOL errors are because you put ; in "if()" line