Hey,
ich bastel grad ein Adminsystem und habe ein Problem. Ich habe es bereits geschafft ein Adminlevel zu erstellen und dieses beim einloggen auch auslesen zu lassen. Wenn ich nun den Befehl /nschild eingebe bekomme ich auch die Nachricht, dass ich nicht befugt bin den Befehl zu nutzen. Aber er wird trotzdem durchgeführt.
Adminbefehl
- .Phiver
- Geschlossen
- Erledigt
Wichtiger Hinweis: Bitte ändert nicht manuell die Schriftfarbe auf schwarz sondern belasst es bei der Standardeinstellung. Somit tragt ihr dazu bei dass euer Text auch bei Verwendung unseren dunklen Forenstils noch lesbar ist!
Tipp: Ihr wollt längere Codeausschnitte oder Logfiles bereitstellen? Benutzt unseren eigenen PasteBin-Dienst Link
-
So müsste es funktionieren:
Cstock isPlayerAnAdmin(playerid,rang) { if(GetPVarInt(playerid,"Adminlevel")>=rang) return 1; return 0; } -
So müsste es funktionieren:
Cstock isPlayerAnAdmin(playerid,rang) { if(GetPVarInt(playerid,"Adminlevel")>=rang) return 1; return 0; }Wenn ich das mache, dass wird zwar der Befehl blockiert aber die Nachricht, dass ich kein Admin bin wird nicht angezeigt.
-
C
Alles anzeigen{ if(strcmp(cmdtext,"/nschild",true)==0) { if(isPlayerAnAdmin(playerid,3)) { if(IsPlayerInAnyVehicle(playerid)) { new vehicleid; vehicleid = GetPlayerVehicleID(playerid); SetVehicleNumberPlate(vehicleid,"Test 1"); SetVehicleToRespawn(vehicleid); PutPlayerInVehicle(playerid,vehicleid,0); } } else { SendClientMessage(playerid,BLAU,"Du bist kein Administrator!"); return 1; } }Du musst ein "else" unter die Klammer machen sonst ist es logisch das jedes mal die Nachricht ausgegeben wird
-
Habe dir kurz mal markiert was hier wichtig wäre:
Code
Alles anzeigenif(strcmp(cmdtext,"/nschild",true)==0) { if(isPlayerAnAdmin(playerid,3)) { if(IsPlayerInAnyVehicle(playerid)) { new vehicleid; vehicleid = GetPlayerVehicleID(playerid); SetVehicleNumberPlate(vehicleid,"Test 1"); SetVehicleToRespawn(vehicleid); PutPlayerInVehicle(playerid,vehicleid,0); return 1; //Wichtig!! } SendClientMessage(playerid,BLAU,"Du bist kein Administrator!"); return 1; } }Du musst (fast) jedes mal, wenn der Code den du ausführen willst fertig ist, ein "return" einfügen. Meistens ist das ein "return 1;" in seltenen Fällen, vor allem selbst geschriebenen publics/stocks kann es auch ein "return 0;" sein.
Ist das return nicht da, dann wird der Code der darunter steht auch einfach ausgeführt, weil du keinen Wert zurückgibst.
Ja, du kannst auch einfach ein "else" einbauen, ich selbst finde es aber so schöner und es spart in Summe bei mehreren 10.000 - 100.000 Zeilen Code auch Schreibarbeit und Zeilen:)
Wenn du noch fragen hast kannst du mir auch ne PN schreiben, mfg
-
Du kannst es aber auch so schreiben:
C
Alles anzeigenif(strcmp(cmdtext,"/nschild",true)==0) { if(!isPlayerAnAdmin(playerid,3))return SendClientMessage(playerid,BLAU,"Du bist kein Administrator!"); //Nun wird mit dem "!" abgefragt ob er ein Admin ist wenn nicht return SendClientMessage... if(!IsPlayerInAnyVehicle(playerid))return SendClientMessage(playerid,BLAU,"Du bist in keinem Fahrzeug."); new vehicleid; vehicleid = GetPlayerVehicleID(playerid); SetVehicleNumberPlate(vehicleid,"Test 1"); SetVehicleToRespawn(vehicleid); PutPlayerInVehicle(playerid,vehicleid,0); return 1; } -
breadfish.de
Hat das Thema geschlossen.
