Problem mit CMD-Varianten

Heyy,

Wenn ich /sethp mache muss man dafür Aduty sein das funktioniert auch aber das gibt mir Warnings wenn ich das bsp bei /setarmor dann auch mache gibt das wieder ein Error hier der Code:
CMD:sethp(playerid, params[])
{
if(!IsLoggedIn(playerid))return 1;
new Float:hp, pID;
if(Spieler[playerid][pAdmin] < 3)return SendClientMessage(playerid, COLOR_GREY, ERROR_ADMIN);
if(Spieler[playerid][pAdminDuty] == false)return SendClientMessage(playerid,COLOR_GREY,ERROR_ADUTY);
if(Spieler[pID][pAdmin] > Spieler[playerid][pAdmin])return SendClientMessage(playerid, COLOR_GREY, "Du kannst die HP höherer Team-Mitglieder nicht ändern.");
if(sscanf(params, "uf", pID, hp))return SendClientMessage(playerid, COLOR_GREY, "Benutze /sethp [SpielerID/Name] [Leben]");
if(hp < 0 || hp > 10000)return SendClientMessage(playerid, COLOR_GREY, "Die Leben müssen zwischen 0 und 10.000 sein.");
if(!IsPlayerConnected(pID))return SendClientMessage(playerid, COLOR_GREY, ERROR_OFFLINE);
if(IsPlayerNPC(pID))return 1;
SetPlayerHealth(pID, hp);
if(hp > 125.0)pSetHP[pID] = 1;
return 1;
}

Die Zeile:
if(Spieler[playerid][pAdminDuty] == false)return SendClientMessage(playerid,COLOR_GREY,ERROR_ADUTY);
#define ERROR_ADUTY"Du bist nicht im Administrations-Dienst."
Woran kann das liegen?
Bitte um hilfe.

Zitat von robbi2304

Wo wird dir der Warning angezeigt ? In Pawn ?

if(Spieler[playerid][pAdminDuty] == false)return SendClientMessage(playerid,COLOR_GREY,ERROR_ADUTY);
In der Zeile bei cmd:sethp hatte was falsch reingeschrieben habs editiert.

Zitat von robbi2304

Alles klar. Da arbeitest ja nicht mit einem Bool daher kannst da nicht mit true und falses arbeiten.

Thema dann erledigt ?

oh, dummer Fehler von mir Hätte ich auch selbst drauf kommen könn' danke dir.

Zitat von Szertockz

bei sethp steht auch ein bool wert und wie Ich es verstanden habe funktioniert es da nur bei setarmor nicht. Schick uns mal dein setarmor Befehl.

hat sich schon erledigt? habs selbst hinbekommen hab mich doch schon bedankt danke dir trotzdem auch nochmal.

Zitat von Szertockz

Du kannst das auch so schreiben
if(!Spieler[playerid][pAdminDuty])return SendClientMessage(playerid,COLOR_GREY,ERROR_ADUTY);

So habe ich das auch gemacht das funktioniert auch jetze^^ aber vielen Dank dir trotzdem mal war halt dumm.

Zitat von robbi2304

Da ich mal vermute das du MySQL verwendest finde ich den Wert

if(!Spieler[playerid][pAdminDuty])return SendClientMessage(playerid,COLOR_GREY,ERROR_ADUTY);

komisch

Ich hätte es so geschrieben

if(!Spieler[playerid][pAdminDuty] <3)return SendClientMessage(playerid,COLOR_GREY,ERROR_ADUTY);

Da heißt es dann ist pAdminDuty Wert kleiner als 3 gibt den SendClientMessage aus.
Ich glaub denn so kann jeder den Befehl ausführen weil ja nicht angegeben wird welchen wert er verwenden soll

Alles anzeigen

So wie ich es gemacht habe und Szertockz es gesagt hat war es richtig hab ein anderen Fehler nun:

if (PRESSED(KEY_N))
{
cmd_heal(playerid);
cmd_ausruesten(playerid);
return 1;
}
Gibt 2 Warnings für cmd_heal & cmd_ausruesten nur noch das dann ist jut. xD

Zitat von robbi2304

Zeig uns die beiden befehle bitte einmal + den genauen fehlercode

Der Befehl hat damit nichts zutun da das einfach nur ein return des Befehls ist und der Befehl funktioniert perfekt. Er gibt mir halt Warnings wegen den zwei cmd_heal & ausruesten wenn ich das rausnehme dann keine mehr.

Zitat von Szertockz

Versuch es mal so
if (PRESSED(KEY_NO))[size=12]{cmd_heal(playerid, "");cmd_ausruesten(playerid, "");return 1;}[/size]

Vielen Dank funktioniert

Zitat von JustMe.77

Du redest die ganze Zeit von Warnings, wie wär's wenn du die mal mit entsprechenden Zeilenmarkierungen posten würdest?

Hab ich du schlaubi und siehst ja er kann mir das erklären rede einfach nicht mit wenn du mir nicht hilfst dumm sowas...

Zitat von robbi2304

Wo hast du z.b hier Preis gegeben wo die Warnings sind bzw welche nirgendwo also mal die klappe halten.

In den beiden Zeilen du hirni war doch was falsch bzw nicht falsch hat aber was gefehlt also klappe halten.