Need help :D

[MATHIJS]

Ik heb wat lopen vogelen maar krijg wat errors.

Ik zou het zeer op prijs stellen als iemand het voor me kan fixe =3

Het Stukje script :

public OnPlayerClickPlayer(playerid, clickedplayerid, source)
{
if (IsPlayerAdmin(playerid));
new message[128];
format(message, 128, "player ID: %d got kicked!!", clickedplayerid);
SendClientMessageToAll(COLOR_BLUE, message);
Kick(clickedplayerid);
else SendClientMessage(playerid, COLOR_RED, "Only admins can do that!!");
return 1;
}

De Errors :

C:\Users\Mathijs\Desktop\[Name] Server\gamemodes\NewScript.pwn(725) : error 036: empty statement
C:\Users\Mathijs\Desktop\[Name] Server\gamemodes\NewScript.pwn(730) : error 029: invalid expression, assumed zero

heel erg bedankt als je het kan fixen

[Gast]

Probeer:

public OnPlayerClickPlayer(playerid, clickedplayerid, source)
{
    	if (IsPlayerAdmin(playerid));
       new message[128];
       format(message, 128, "player ID: %d got kicked!!", clickedplayerid);
       SendClientMessageToAll(COLOR_BLUE, message);
       Kick(clickedplayerid);
        } else { 
SendClientMessage(playerid, COLOR_RED, "Only admins can do that!!");
       return 1;
}

[.Timothy]

public OnPlayerClickPlayer(playerid, clickedplayerid, source)
{
if(IsPlayerAdmin(playerid))
{
	new message[128];
	format(message, 128, "player ID: %d got kicked!!", clickedplayerid);
	SendClientMessageToAll(COLOR_BLUE, message);
	Kick(clickedplayerid);
}
else SendClientMessage(playerid, COLOR_RED, "Only admins can do that!!");

return 1;
}

Omdat er meer dan één functie wordt opgeroepen na je if-constructie, moet je er wel accolades achter zetten, zodat alle code wordt uitgevoerd. Bij de else kun je die accolades weglaten, want er wordt maar één functie opgeroepen.

@ Biesmen: ook jouw code is nog fout, want je sluit de if-structuur wel, maar opent hem niet.

[Gast]

public OnPlayerClickPlayer(playerid, clickedplayerid, source)
{
if(IsPlayerAdmin(playerid))
{
	new message[128];
	format(message, 128, "player ID: %d got kicked!!", clickedplayerid);
	SendClientMessageToAll(COLOR_BLUE, message);
	Kick(clickedplayerid);
}
else SendClientMessage(playerid, COLOR_RED, "Only admins can do that!!");

return 1;
}

Omdat er meer dan één functie wordt opgeroepen na je if-constructie, moet je er wel accolades achter zetten, zodat alle code wordt uitgevoerd. Bij de else kun je die accolades weglaten, want er wordt maar één functie opgeroepen.

@ Biesmen: ook jouw code is nog fout, want je sluit de if-structuur wel, maar opent hem niet.

Daar had ik even niet opgelet, omdat hij niet aangaf welke lijn wat is. :/

Edit: maar volgens mij klopt jouw script ook niet.

Je hebt 'else' niet gesloten en ook niet geopend.

Dus ik denk dat dit het moet zijn:

public OnPlayerClickPlayer(playerid, clickedplayerid, source)
{
if(IsPlayerAdmin(playerid))
{
	new message[128];
	format(message, 128, "player ID: %d got kicked!!", clickedplayerid);
	SendClientMessageToAll(COLOR_BLUE, message);
	Kick(clickedplayerid);
} else {
       SendClientMessage(playerid, COLOR_RED, "Only admins can do that!!");
return 1;
    } 
}

Bewerkt: 14 november 2010 door Gast

[MATHIJS]

Bedankt voor de help het werkt nu en snap nu ook wat meer van die if en else