Antiwaffenhack

Der Fehler liegt in der ersten if-Abfrage.
Schau nochmal nach, vielleicht siehst du es selber.
Ansonsten nochmal kurz melden.

Da ist eindeutig ein Fehler.....
Aber da du ja anscheinend nur von Jeffry mit dem Cop Skin Hilfe haben willst, kannst du von mir aus auf ihn warten.

if(!isAdmin(playerid,1))

wofür steht die 1?

Außerdem find ich es ein wenig frech immer jeffry zu markieren...

Lol ich verstehe dein System überhaupt nicht?
Dein Adminrang ist passend returnst du? wtf?

if(!isAdmin(playerid,1)) return SendClientMessage(playerid,grün,"Dein Adminrang ist nicht hoch genug");
{

Das ist dein Fehler.

So klappt es:
forward Antiwaffensystem(playerid,params[]);
public Antiwaffensystem(playerid,params[])
{
new pID,wpID,ammo;
if(!isAdmin(playerid,1))
{
if(sscanf(params,"ui",pID,wpID))return SendClientMessage(playerid,rot,"INFO: Bist kein admin");
if(wpID<1||wpID>46)return SendClientMessage(playerid,blau,"Waffenid falsch");
if(ammo<1||ammo>999999)return SendClientMessage(playerid,rot,"Munition nicht gefunden");
GivePlayerWeapon(pID,wpID,ammo);
BanEx(pID,"Waffenhack");
SendClientMessage(playerid,blau,"Du hast dir waffen gehackt somit auch ein Bann bekommen");
}
else
{
if(isAdmin(pID,1))return SendClientMessage(pID,rot,"Dein Adminlevel ist passend.");
if(wpID<1||wpID>46)return SendClientMessage(pID,blau,"Waffenid falsch");
if(ammo<1||ammo>999999)return SendClientMessage(playerid,rot,"Munition nicht gefunden");
GivePlayerWeapon(pID,wpID,ammo);
SendClientMessage(pID,blau,"Du hast dir waffen gegeben");
return 1;
}
}

Naja richtig lesen hilft, er hat !isAdmin geschrieben.

Aber komisch dass mir vorher der Klammerfehler nicht aufgefallen ist, lol

So, kurze Erklärung:

Wenn du schon bei einer If-Abfrage ein return mit reinmachst kannst du da unter keine Klammer mehr setzen, so wie m4a_X das gemacht hat ist es richtig, deswegen ist das else auch falsch, weil es keine If-Funktion finden konnte

(Nach meiner Erfahrung,)

So sehe ich das auch. Ich weiß allerdings überhaupt nicht was das ganze bezweckt, musst du halt so machen wie du es verwenden willst den die if Abfrage ergibt für mich sowieso keinen Sinn.

Für mich ergibt die Abfrage kein Sinn was für ne ID die Waffe hat um den Hacker dann sogar ne Meldung zu schicken "Die Waffe gibt es nicht" xD
Selbe mit Munition etc.

Zitat von Markuslobicher

@Jeffry

Zitat von m4a_X

So klappt es:

Das wird zwar kompiliert, aber funktionieren wird es nicht.

Der Code von Anfang an an sich macht schon keinen Sinn, da man sich als Admin keine Waffen geben kann (kein sscanf in dem Abschnitt) und man als Spieler nur gebannt würde, wenn man sich eine Waffe gibt, über den Befehl gesteuert.
Des Weiteren würde nicht der Spieler selbst gebannt werden, sondern der Spieler, den er per params eingibt.
Außerdem wird in sscanf die ammo nicht ausgelesen.

Es wäre gut zu wissen, wie denn "Antiwaffensystem" überhaupt aufgerufen wird, und mit welchen Daten, denn so wird es nicht funktionieren.

Für mich hat der Code sowieso keinen Sinn ergeben.