[Filterscript]|><| If-Abfrage(?) funktioniert nicht! |><|

Zeile 21 !

Ausschnitt aus deinem Skript:
for(new strs=0;strs!=16;strs++)
if(pIP[strs]=='.'||!pIP[strs]) {
strmid(blocks[step],pIP,strste,strs);
strste=strs+1;
step++;
}
Ich tippe mal grob darauf...

Ansonsten den üblichen strcmp() == 0, anstatt !strcmp() mal ausprobieren

An dem Code kann es eigentlich nicht liegen, denn er ist von mir :D, und so funktioniert es bei mir wunderbar:

new pIP[16],strste,step,blocks[4][4];
GetPlayerIp(playerid,pIP,16);
for(new strs=0;strs!=16;strs++)
if(pIP[strs]=='.'||!pIP[strs]) {
strmid(blocks[step],pIP,strste,strs);
strste=strs+1;
step++;
}
printf("banip %s.%s.***.***",blocks[0],blocks[1]);

Du kannst es ja auch nochmal so versuchen. Sollte normalerweise aber eigentlich keinen Unterschied bringen:

new pIP[16],strste,step,blocks[4][4];
GetPlayerIp(playerid,pIP,16);
for(new strs=0;strs!=16;strs++) {
if(pIP[strs]=='.'||!pIP[strs]) {
strmid(blocks[step],pIP,strste,strs);
strste=strs+1;
step++;
}
}
printf("banip %s.%s.***.***",blocks[0],blocks[1]);

Frage:
Wieso funktioniert das Skript überhaupt ?

Bei der for-Schleife fehlt die Klammer auf,
und das nach dem strs++ ist, wenn ichs richtig sehe,
eine geschweifte Klammer zu, und keine runde...

Sicher, dass der Compiler da nicht meckert ?

//edit:

Zitat

An dem Code kann es eigentlich nicht liegen, denn er ist von mir

Das lässt dich jetzt nicht gerade im guten Licht erscheinen

Ich weiß nicht was du meinst. Guck doch mal hinter die Zeilen.

Zitat von Trooper[Y]

Das lässt dich jetzt nicht gerade im guten Licht erscheinen

Du solltest schon raten können wieviel recht ich dir gebe, Trooper

Und nein bei Pawno gibts wirkilch kein Error, hab's grad getestet
Versuche leider noch die Codes zu verstehen o.0

Ihr versteht wohl keine Ironie, oder? Der Spruch war eher eine Art amysanter Witz.
Der Code hat keine Fehler, weil die Klammern hinter den Zeilen stehen.
Trooper: Kannst du mir denn sagen, wo der Sinn darin wäre, wenn hinter strs++; ein ")" wäre?

Ich frage mich gerade, was an dem Code nicht zu verstehen ist. Die Schleife geht jede Zelle vom Array durch und guckt, ob ein Punkt vorhanden ist. Dann Splittet er den String einfach...

Auf gutdeutsch: Dein Code ist kacke

Eine for-Schleife erhält ihre Schleifen-Informationen in runden Klammern (ich meine, du machst ja auch ne runde auf, aber mit ner geschwungenen zu...), woraufhin eine geschwungene Klammer die Schleifenaktionen ankündigt (bei einleinern gehts auch ohne Klammern )

Ohh, jetzt weiß ich was du meintest. Ich habe gerade an diese Stelle "step++;" gedacht. Bei der Schleife ist es natürlich logisch, dass diese mit ) geschlossen wird.
Die Schleife ist "im Originalzustand", wie du 2 Posts höher auch sehen kannst ohne "{,}" Klammern.
Aber dennnoch ist der Code nicht kacke! Probiere ihn doch einfach mal aus, dafür brauchst du nichtmal auf irgendeinen Server gehen:

new pIP[16],strste,step,blocks[4][4];
//GetPlayerIp(playerid,pIP,16);
format(pIP,16,"123.123.123.123");
for(new strs=0;strs!=16;strs++)
if(pIP[strs]=='.'||!pIP[strs]) {
strmid(blocks[step],pIP,strste,strs);
strste=strs+1;
step++;
}
printf("banip %s.%s.***.***",blocks[0],blocks[1]);

Ich schätze trotzdem, dass es daran liegt, weil da pawn durcheinander kommt,
wenn man es so schändet

Und wenn man sowas im Skript hat, braucht man auch erstmal nicht nach anderen Fehlern zu suchen

Möchtest du die RangeIp haben oder wie??

Geht ganz einfach mit split oder sscanf ...

MfG Kevin

Die Funktion womit die IP in Blöcke geteilt wird, ist wirklich nicht gut.Nicht sehr übersichtlich und auch die Blöcke als Array zu speichern ist nicht gerade klever.Das könnte man als Integer pro Block besser speichern oder via Bitshift in nur einer Variable alle Werte.

printf("banip %s.%s.*.*",blocks[0],blocks[1]);

Hätte das auch viel eher mit sscanf gemacht.Wie auch immer,hab hier noch eine sehr alte Funktion die mir breadfish mal vor bestimmt über 3Jahren geschrieben hatte, als Ich gerade mit PAWN angefangen hatte:
stock IP2RootIP(ip[16], blocks) { //breadfish
new
rootend,
i,
rootip[16];
if (blocks >= 4) {
rootip = ip;
} else {
for (i=1;i<=blocks;i++) {
rootend = strfind(ip, ".", true, rootend) + 1;
}
strmid(rootip, ip, 0, rootend - 1);
}

for (i=1;i<=4-blocks;i++) {
strcat(rootip, ".*");
}
return rootip;
}

So funktioniert es definitiv:
http://pastebin.com/y7uwYcCi