Komme bei Adminbefehl nicht weiter

(waffe<0 && waffe>46)

(muni<1 && muni>1000)

du darfst nicht sondern && da beides stimmen muss

Kein wunder
du hast vergessen das er die Waffe überhaupt abspeichert...
mach mal eine Variable z.B. Waffe[MAX_PLAYERS]; und Munit[MAX_PLAYERS];
bei den ersten machst du dan halt
Waffe[playerid] = strlen(inputtext); Also das wo man da eingibt...
und bei Muni genauso

Zitat von [DT]Done

ShowPlayerDialog(playerid,DIALOG_WAFFENMENU2,DIALOG_STYLE_INPUT,"Munitions -Abfrage","Bitte gebe die Mution an (100-1000)","Fertig","");
if(!strlen(inputtext))
{
SendClientMessage(playerid,Farbe_DRot,"Bitte gebe die Munition an !");
return 1;
}
if(muni<1||muni>1000) return SendClientMessage(playerid,Farbe_DRot,"Bitte gebe die Munition von 1-1000 an !");
GivePlayerWeapon(playerid,waffe,muni);
return 1;
}
}

Alles anzeigen

Du zeigst einen Dialog dort erst und fragst direkt noch darunter die Eingabe des Spielers ab, obwohl du noch im alten Dialog steckst?

Nach

Zitat von [DT]Done

ShowPlayerDialog(playerid,DIALOG_WAFFENMENU2,DIALOG_STYLE_INPUT,"Munitions -Abfrage","Bitte gebe die Mution an (100-1000)","Fertig","");

kommt erstmal nichts, dann

if(dialogid == DIALOG_WAFFENMENU2)
{
if(!strlen(inputtext))
{
SendClientMessage(playerid,Farbe_DRot,"Bitte gebe die Munition an !");
return 1;
}
if(muni<1||muni>1000) return SendClientMessage(playerid,Farbe_DRot,"Bitte gebe die Munition von 1-1000 an !");
GivePlayerWeapon(playerid,waffe,muni);
return 1;
}

Zitat von [DT]Done

naja,wenn ich das so mache,dann fehlt mir aber wieder die Waffe

Speicher die Waffe in einer Variable zwischen. Anders geht es nicht!

ShowPlayerDialog(playerid,DIALOG_WAFFENMENU2,DIALOG_STYLE_INPUT,"Munitions -Abfrage","Bitte gebe die Mution an (100-1000)","Fertig","");
if(!strlen(inputtext))
{
SendClientMessage(playerid,Farbe_DRot,"Bitte gebe die Munition an !");
return 1;
}
if(muni<1||muni>1000) return SendClientMessage(playerid,Farbe_DRot,"Bitte gebe die Munition von 1-1000 an !");
GivePlayerWeapon(playerid,waffe,muni);
return 1;
}

Wenn du das hier hast, hast du unten, wenn du die Munition geben sollst immer noch als inputtext das vom vorherigen Dialog!

So erstmal mein lieber Mysterio:
"(muni<1 && muni>1000)"
Wenn du mir beantworten kannst wie eine Zahl kleiner als 1, aber gleichzeitig größer als 1000 sein soll, kriegste nen Keks.

-----------------------------------

@[DT]Done:
Ich werd dir jetzt dazu ein komplettes Tutorial schreiben damit du auch was draus lernst und nicht bloß C&P-Scrpting benutzt.
Erstmal, was in einem Dialog eingegeben wird, wird im Callback als der !String! "inputtext" aufgerufen.
Wir müssen den String nun erstmal in einen Integer umwandeln:
new wID = strval(inputtext);
Jetzt haben wir der Variable wID die Eingabe des Spielers als Integer gespeichert.

Dannach kommt die Abfrage, ob die eingegebene ID denn auch Gültig ist, das machen wir aber nicht so:
if(waffe<0 && waffe>46)
Dasselbe Prinzip wie bei Mysterio: Wie soll eine Zahl kleiner als 0, aber größer als 46 sein? Auch für dich ein Keks wenn du mir das beantwortest.
Richtig wäre zu überprüfen, ob die Zahl denn in einem berreich zwischen 0 und 46 liegt, also allen Verfügbaren Waffen-ID's.
Oder man man ersetzt das "und" in deiner Abfrage einfach durch ein "oder".

Jetzt aber zu dem was ich beim besten Willen nicht begreife:
Du speicherst die ID doch in keiner Variable wo du sie später wieder auslesen kannst und in einer Datei (Glücklicherweise) auch nicht.
Also, machen wir das auch noch eben. Wichtig hierbei ist eine Globale Variable zu verwenden, da eine Lokale Variable nach dem Aufrufen des Callbacks zurückgesetzt wird.
Dass du die Variablen "waffeM" und "muniM" oben im Script definiert hast ist somit richtig.
Aber wieso speicherst du denn die ID's nicht rein? Wie soll das alles funktionieren?
Machen wirs doch einfach mal:
waffeM[playerid] = wID;

Jetzt musst du nur noch einen neuen Dialog öffnen.
Achja, für die Munition brauchst du keine Globale Variable zu zwischenspeichern, da die Waffe ja mit dem Beenden des 2. Dialoges Gegeben werden soll.
Somit wäre diese auch noch unnötig gewesen.
Aber nach diesem Verfahren kannst du dir nun auch ganz einfach den Rest fertigschnippeln.

Vielleicht war ich ein bisschen Grob in diesem Text, ich hab grade auch keinen Bock ihn mir nochmal durchzulesen.
Ich war nur schockiert dass jemand soviel Unwissen an den Tag legt.
Grundwissen ist für soetwas nicht wegzudenken. Naja, eigentlich wird Grundwissen immer benötigt wenn du nicht nur Vorhast ein Script zu übersetzen.

MfG Firerfan