Ich versteht nicht was falsch ist | ocmd problem

ocmd:DJ
{
return 1; // das fehlt :)
}

So hast du es:

ocmd:DJ(playerid,params[])
{
if(!IsPlayerConnected(playerid))return 0;
{
if(GetPVarInt(playerid, "Job")== 1);
{
if(!IsPlayerInRangeOfPoint(playerid,1,1211.2683,-905.6534,48.0557))return SendClientMessage(playerid,ROT,"Du bist an keinem Dj Pult!");
ShowPlayerDialog(playerid,DIALOG_DISCO,DIALOG_STYLE_LIST,"Disco","Neon Lichter\nDisco Lichter\nNebelmaschine\nTanzfläche\nDiscokugeln\nBlitzlichter\nMusik","Weiter","Abbrechen");return 1;
}
else
{
SendClientMessage(playerid,ORANGE,"Du hast nicht den Job DJ!");
}
}
}

richtig wäre es so:

ocmd:DJ(playerid,params[])
{
if(!IsPlayerConnected(playerid))return 0;
if(GetPVarInt(playerid, "Job")== 1)
{
if(!IsPlayerInRangeOfPoint(playerid,1,1211.2683,-905.6534,48.0557))return SendClientMessage(playerid,ROT,"Du bist an keinem Dj Pult!");
ShowPlayerDialog(playerid,DIALOG_DISCO,DIALOG_STYLE_LIST,"Disco","Neon Lichter\nDisco Lichter\nNebelmaschine\nTanzfläche\nDiscokugeln\nBlitzlichter\nMusik","Weiter","Abbrechen");
}
else
{
SendClientMessage(playerid,ORANGE,"Du hast nicht den Job DJ!");
}
return 1;
}

Dieser Fehler:

warning 209: function "ocmd_DJ" should return a value

sagt alles was du Wissen soltlest! Wenn du das schon nicht Wusstest.... und ausserdem nach den dialog return 1; ?!? WTF

//E: Fehler behoben, wo hast du Scripten gelernt? Nach einer if Abfrage kommt NIE ein ; -.-
//E: So jetzt hatte ich den Fehler wieder gut gemacht^^

ocmd:DJ(playerid,params[])
{
if(!IsPlayerConnected(playerid))return 0; // return 0 an der Stelle geht nicht, besser anders rum if(IsPlayerConnected(..))
{
if(GetPVarInt(playerid, "Job")== 1);
{
if(!IsPlayerInRangeOfPoint(playerid,1,1211.2683,-905.6534,48.0557))return SendClientMessage(playerid,ROT,"Du bist an keinem Dj Pult!");
ShowPlayerDialog(playerid,DIALOG_DISCO,DIALOG_STYLE_LIST,"Disco","Neon Lichter\nDisco Lichter\nNebelmaschine\nTanzfläche\nDiscokugeln\nBlitzlichter\nMusik","Weiter","Abbrechen");return 1;
return 1;
}
else
{
SendClientMessage(playerid,ORANGE,"Du hast nicht den Job DJ!");
return 1;
}
}
return 1;
}

Zitat

hilft mir nicht viel weiter , denn wie ich schon geschrieben hatte habe ich nurnoch die 2 Errors.

@Alle danke für die Antworten!

welche zwei errors?

Du hast nach der if Abfrage auch ein Simikolon rein geballer was niemals sein sollte!

//E: Rechtschreibung
//E:

Zitat

ocmd:DJ(playerid,params[])
{
if(!IsPlayerConnected(playerid))return 0; // return 0 an der Stelle geht nicht, besser anders rum if(IsPlayerConnected(..))
{
if(GetPVarInt(playerid, "Job")== 1);
{
if(!IsPlayerInRangeOfPoint(playerid,1,1211.2683,-905.6534,48.0557))return SendClientMessage(playerid,ROT,"Du bist an keinem Dj Pult!");
ShowPlayerDialog(playerid,DIALOG_DISCO,DIALOG_STYLE_LIST,"Disco","Neon Lichter\nDisco Lichter\nNebelmaschine\nTanzfläche\nDiscokugeln\nBlitzlichter\nMusik","Weiter","Abbrechen");return 1;
return 1;
}
else
{
SendClientMessage(playerid,ORANGE,"Du hast nicht den Job DJ!");
return 1;
}
}
return 1;
}

Alles anzeigen

Hahahaha leider auch falsch Guck dir die if Abfrage mal an :p

Zitat

PawnFox ich probiere es.

@LogispireMode

hier die Errors.

PAWN Quelltext

1
2

C:\Users\Lennart\Desktop\disco.pwn(179) : error 036: empty statement
C:\Users\Lennart\Desktop\disco.pwn(184) : error 029: invalid expression, assumed zero

Alles anzeigen

Und meins ?

Du hast nach der if(GetPVarInt(playerid,"Job") == 1)) //Hier hattest du ein ";" was NIEMALS sein darf!(Nach if kein ; am ende der klammer setzen^^)

Dann wäre der erste weg,der zweite müsste dann eig. auch weg gehen^^

PS: Probier doch einfach mal mein Code von oben