/acp Script Fehler???

The Sensei · 11. Juli 2014

Guten Tag,

ich hatte die Idee das ich ein /acp Befehl Scripte.
Dieser Soll halt herausschreiben wie oft der oder die Person gebannt/gekickt oder gemutet wurde.

Code

C:\Users\Patrick\Desktop\GER\GER\GER\gamemodes\IRP-City.pwn(413) : warning 225: unreachable code
C:\Users\Patrick\Desktop\GER\GER\GER\gamemodes\IRP-City.pwn(425) : warning 225: unreachable code
C:\Users\Patrick\Desktop\GER\GER\GER\gamemodes\IRP-City.pwn(432) : warning 225: unreachable code
C:\Users\Patrick\Desktop\GER\GER\GER\gamemodes\IRP-City.pwn(603) : warning 225: unreachable code
C:\Users\Patrick\Desktop\GER\GER\GER\gamemodes\IRP-City.pwn(612) : warning 225: unreachable code
C:\Users\Patrick\Desktop\GER\GER\GER\gamemodes\IRP-City.pwn(691) : error 029: invalid expression, assumed zero
C:\Users\Patrick\Desktop\GER\GER\GER\gamemodes\IRP-City.pwn(691) : error 022: must be lvalue (non-constant)
C:\Users\Patrick\Desktop\GER\GER\GER\gamemodes\IRP-City.pwn(712) : warning 202: number of arguments does not match definition
Pawn compiler 3.2.3664	 	 	Copyright (c) 1997-2006, ITB CompuPhase




2 Errors.

Alles anzeigen

Script Code: ocmd:acp(playerid,params[])
{
if(!isPlayerAnAdmin(playerid,5)) return SendClientMessage(playerid,ROT,"Zugriff Verweigert");
new string1[200],string2[200],string3[128],pID,rest[500];
if(sscanf(params,"uis",pID))return SendClientMessage(playerid,BLAU,"/acp [playerid]");
if(IsPlayerConnected(pID))
{
format(string1, sizeof string1, "Name: %s",SpielerName(pID));
format(string2, sizeof string2, "Timeban: %s",GetPVarInt(playerid,"Banned?"));
format(string3, sizeof string3, "");
format(rest, sizeof rest, "%s\n%s\n%s\n", string1,string2,string3);
if(!isPlayerAnAdmin(playerid,5)) return ShowPlayerDialog(playerid,DIALOG_ACP,DIALOG_STYLE_MSGBOX,"Admin Control Panel",rest,"Ok","");
}
SendClientMessage(playerid,ROT,"Spieler ist nicht Online");
return 1;
}
Beispiel wo er es hochzählen soll
ocmd:tban(playerid,params[])
{
if(!isPlayerAnAdmin(playerid,2))return SendClientMessage(playerid,ROT,"Rang zu Niedrig.");
new pID, time, grund[128];
if(sscanf(params,"uis",pID,time,grund))return SendClientMessage(playerid,BLAU,"/tban [playerid] [zeit] [grund]");
if(IsPlayerConnected(pID))
{
if(time>0&&time<1000)
{
new string[128];
format(string,sizeof(string),"%s wurde von %s für %i Minuten gebannt. Grund: %s",SpielerName(pID),SpielerName(playerid),time,grund);
SendClientMessageToAll(ROT,string);
new wert;

wert=gettime()+(60*time);

SetPVarInt(pID,"timeban",wert);
SetTimerEx("WarnTimer2", 250, false, "i", pID);
SetPVarInt(playerid,"Banned?",++);//<- Da das ++ ist doch hochzählen oder?
Kick(pID);
return 1;
}
SendClientMessage(playerid,ROT,"Ungültige Zeit.");
return 1;
}
SendClientMessage(playerid,ROT,"Spieler ist nicht online.");
return 1;
}

Bitte helft mir. das ++ bedeutet doch das er was hochzählen soll?

Bananenstaude · 11. Juli 2014

" if(sscanf(params,"uis",pID))return SendClientMessage(playerid,BLAU,"/acp [playerid]");" da stimmt was nicht (u i s und nur pID).
++ ist an der stelle falsch.
SetPVarInt(playerid,"Banned",GetPVarInt(playerid,"Banned")+1); wäre richtig

Also es wird Banned auf x+1 gesetzt. x steht für den Banned-Wert.

The Sensei · 11. Juli 2014

Code

C:\Users\Patrick\Desktop\GER\GER\GER\gamemodes\IRP-City.pwn(413) : warning 225: unreachable code
C:\Users\Patrick\Desktop\GER\GER\GER\gamemodes\IRP-City.pwn(425) : warning 225: unreachable code
C:\Users\Patrick\Desktop\GER\GER\GER\gamemodes\IRP-City.pwn(432) : warning 225: unreachable code
C:\Users\Patrick\Desktop\GER\GER\GER\gamemodes\IRP-City.pwn(603) : warning 225: unreachable code
C:\Users\Patrick\Desktop\GER\GER\GER\gamemodes\IRP-City.pwn(612) : warning 225: unreachable code
C:\Users\Patrick\Desktop\GER\GER\GER\gamemodes\IRP-City.pwn(712) : warning 202: number of arguments does not match definition
Pawn compiler 3.2.3664	 	 	Copyright (c) 1997-2006, ITB CompuPhase




6 Warnings.

Alles anzeigen

Server geht an und aus wenn ich starte.

Hab das mit ++ geändert das andere nicht weis nicht was du meinst.

**theTV** · 12. Juli 2014

markier mal die zeilen wo die warnings kommen

The Sensei · 12. Juli 2014

Hat alles nichts mit dem System zutuhn....
Alles andere Warnings.

ErikSon · 12. Juli 2014

Die markierte Zeile muss so aussehen:

SetPVarInt (playerid, "Banned?", GetPVarInt (playerid,"Banned?")+1);

The Sensei · 12. Juli 2014

ocmd:tban(playerid,params[])
{
if(!isPlayerAnAdmin(playerid,2))return SendClientMessage(playerid,ROT,"Rang zu Niedrig.");
new pID, time, grund[128];
if(sscanf(params,"uis",pID,time,grund))return SendClientMessage(playerid,BLAU,"/tban [playerid] [zeit] [grund]");
if(IsPlayerConnected(pID))
{
if(time>0&&time<1000)
{
new string[128];
format(string,sizeof(string),"%s wurde von %s für %i Minuten gebannt. Grund: %s",SpielerName(pID),SpielerName(playerid),time,grund);
SendClientMessageToAll(ROT,string);
new wert;

wert=gettime()+(60*time);

SetPVarInt(pID,"timeban",wert);
SetTimerEx("WarnTimer2", 250, false, "i", pID);
SetPVarInt(playerid,"Banned?",GetPVarInt(playerid,"Banned?")+1);//Da
Kick(pID);
return 1;
}
SendClientMessage(playerid,ROT,"Ungültige Zeit.");
return 1;
}
SendClientMessage(playerid,ROT,"Spieler ist nicht online.");
return 1;
}
ErikSon:

ErikSon · 12. Juli 2014

...?
Was klappt denn nun nicht bzw wo und welche warnings / errors bekommt du und wo bekommst du die?

Mehr Informationen wären toll ; )

The Sensei · 12. Juli 2014

Keine Error´s wenn ich Server an mache geht er wieder aus.
Muss mit /acp zusammen hängen.

ErikSon · 12. Juli 2014

Zitat von FerienFreak

Muss mit /acp zusammen hängen.

Sicher?
Hast du mal den Befehl rausgenommen und dann getestet?

PS; Setze den Arrayindex von "Grund" auf 64..
Soviel Zeichen wirst du eh nie in deinem Grund verwenden..

The Sensei · 12. Juli 2014

Der fehler war die Server.cfg... Sorry
Dennoch besteht der Fehler das wenn ich /acp mache und dann z.B. ID 0 kommt immer /acp [playerid] warum???...

ErikSon:

ErikSon · 12. Juli 2014

Deshalb:
if(sscanf(params,"uis/*i und s sind noch zwei parameter, die man eingeben muss, damit der Befehl läuft*/",pID))return SendClientMessage(playerid,BLAU,"/acp [playerid]");

//Machs so:
if(sscanf(params,"u",pID))return SendClientMessage(playerid,BLAU,"/acp [playerid]");

The Sensei · 12. Juli 2014

Jetzt kommt Spieler ist nicht Online????

niklasbollmer · 12. Juli 2014

Zeig mal bitte deinen aktuellen Code.

The Sensei · 12. Juli 2014

ocmd:acp(playerid,params[])
{
if(!isPlayerAnAdmin(playerid,5)) return SendClientMessage(playerid,ROT,"Zugriff Verweigert");
new string1[200],string2[200],string3[128],pID,rest[500];
if(sscanf(params,"u",pID))return SendClientMessage(playerid,-1,"/acp [playerid]");
if(IsPlayerConnected(pID))
{
format(string1, sizeof string1, "Name: %s",SpielerName(pID));
format(string2, sizeof string2, "Timebans: %s",GetPVarInt(playerid,"Banned1"));
format(string3, sizeof string3, "Kicks: %s",GetPVarInt(playerid,"Banned2"));
format(rest, sizeof rest, "%s\n%s\n%s\n", string1,string2,string3);
if(!isPlayerAnAdmin(playerid,5)) return ShowPlayerDialog(playerid,DIALOG_ACP,DIALOG_STYLE_MSGBOX,"Admin Control Panel",rest,"Ok","");
}
SendClientMessage(playerid,ROT,"Spieler ist nicht Online");
return 1;
}

Bananenstaude · 12. Juli 2014

Mach mal:
ocmd:acp(playerid,params[])
{
if(!isPlayerAnAdmin(playerid,5)) return SendClientMessage(playerid,ROT,"Zugriff Verweigert");
new string1[200],string2[200],string3[128],pID,rest[500];
if(sscanf(params,"u",pID))return SendClientMessage(playerid,-1,"/acp [playerid]");
if(IsPlayerConnected(pID))
{
format(string1, sizeof string1, "Name: %s",SpielerName(pID));
format(string2, sizeof string2, "Timebans: %s",GetPVarInt(playerid,"Banned1"));
format(string3, sizeof string3, "Kicks: %s",GetPVarInt(playerid,"Banned2"));
format(rest, sizeof rest, "%s\n%s\n%s\n", string1,string2,string3);
if(!isPlayerAnAdmin(playerid,5)) return ShowPlayerDialog(playerid,DIALOG_ACP,DIALOG_STYLE_MSGBOX,"Admin Control Panel",rest,"Ok","");
}
else return SendClientMessage(playerid,ROT,"Spieler ist nicht Online");
return 1;
}

ErikSon · 12. Juli 2014

Probiers mal so:
ocmd:acp(playerid,params[])
{
if(!isPlayerAnAdmin(playerid,5)) return SendClientMessage(playerid,ROT,"Zugriff Verweigert");
new pID;
if(sscanf(params,"u",pID))return SendClientMessage(playerid,-1,"/acp [playerid]");
if(!IsPlayerConnected(pID))return SendClientMessage(playerid,ROT,"Spieler ist nicht Online");
new string[256],str[64];
format(str, sizeof str, "Name: %s",SpielerName(pID)), strcat(string,str);
format(str, sizeof str, "Timebans: %s",GetPVarInt(playerid,"Banned1")), strcat(string,str);
format(str, sizeof str, "Kicks: %s",GetPVarInt(playerid,"Banned2")), strcat(string,str);
return ShowPlayerDialog(playerid,DIALOG_ACP,DIALOG_STYLE_MSGBOX,"Admin Control Panel",string,"Ok","");
}
Könnte man eig noch ein wenig Ressourcensparender machen..

Zitat von Johan_Wolf

Mach mal:
ocmd:acp(playerid,params[])
{
if(!isPlayerAnAdmin(playerid,5)) return SendClientMessage(playerid,ROT,"Zugriff Verweigert");
new string1[200],string2[200],string3[128],pID,rest[500];
if(sscanf(params,"u",pID))return SendClientMessage(playerid,-1,"/acp [playerid]");
if(IsPlayerConnected(pID))
{
format(string1, sizeof string1, "Name: %s",SpielerName(pID));
format(string2, sizeof string2, "Timebans: %s",GetPVarInt(playerid,"Banned1"));
format(string3, sizeof string3, "Kicks: %s",GetPVarInt(playerid,"Banned2"));
format(rest, sizeof rest, "%s\n%s\n%s\n", string1,string2,string3);
if(!isPlayerAnAdmin(playerid,5)) return ShowPlayerDialog(playerid,DIALOG_ACP,DIALOG_STYLE_MSGBOX,"Admin Control Panel",rest,"Ok","");
}
else return SendClientMessage(playerid,ROT,"Spieler ist nicht Online");
return 1;
}

Alles anzeigen

Zu viele unnötige Strings und wozu 2x Adminabfrage?

//Edit: Außerdem ist die 2. Adminabfrage falsch.. Wenn er nicht min Admin Level 5 ist, dann wird ihm das Adminpanel angezeigt.. ?
PS: Das return am Ende kannste dir sparen, da du den Code schon beendest..

Bananenstaude · 12. Juli 2014

ich hab's halt so gemacht, wie er es wahrscheinlich haben will und wie er es wahrscheinlich im restlichen Script hat

Rocky--BANNED · 12. Juli 2014

Und du meinst nicht, dass deine Probleme mit den Warning zu tun hat?

Zitat

C:\Users\Patrick\Desktop\GER\GER\GER\gamemodes\IRP-City.pwn(612) : warning 225: unreachable code

Irgendwo muss da ein oder mehrere returns fehlen.

Jeffry · 12. Juli 2014

Man braucht hier weder strcat noch sonstiges Zeugs. Außerdem war die string-Formatierung völlig daneben bei sämtlichen Posts oben.
ocmd:acp(playerid,params[])
{
if(!isPlayerAnAdmin(playerid,5)) return SendClientMessage(playerid,ROT,"Zugriff Verweigert");
new pID;
if(sscanf(params,"u",pID))return SendClientMessage(playerid,-1,"/acp [playerid]");
if(!IsPlayerConnected(pID))return SendClientMessage(playerid,ROT,"Spieler ist nicht Online");
new string[256];
format(string, sizeof(string), "Name: %s\n",SpielerName(pID));
format(string, sizeof(string), "%sTimebans: %d\n",string,GetPVarInt(playerid,"Banned1"));
format(string, sizeof(string), "%sKicks: %d\n",string,GetPVarInt(playerid,"Banned2"));
return ShowPlayerDialog(playerid,DIALOG_ACP,DIALOG_STYLE_MSGBOX,"Admin Control Panel",string,"Ok","");
}
Sollte es dennoch nicht funktionieren, und dir weiter gesagt werden, dass der Spieler nicht online ist, dann hast du wahrscheinlich noch die sscanf Funktion im Code definiert, zu dem Plugin dazu. Diese musst du aus dem Code entfernen wenn du das Plugin nutzt. Und wenn es dann immer noch nicht geht, sag nochmal Bescheid, dann zeige ich dir wie man das mit strtok macht.

Außerdem:
SetPVarInt(playerid,"Banned?",++);//<- Da das ++ ist doch hochzählen oder?
zu:
SetPVarInt(playerid,"Banned?",GetPVarInt(playerid,"Banned?")+1);

Zitat von Rocky

Irgendwo muss da ein oder mehrere returns fehlen.

Die Warnung 225 hat rein gar nichts mit fehlenden return's zu tun, im Gegenteil! Unreachable Code heißt "Unereichbarer Code", somit wurden zu viele returns verbaut. Ein fehlendes return siehst du an der Warnung "does not return a value".
Warnung 225 bekommst du hingegen bei sowas:
stock Test()
{
print("Hallo");
return 1;
print("Welt"); //unreachable Code
return 1; //unreachable Code
}