Problem mit Kick / Ban

Blood. · 17. Juli 2013

ocmd:kick(playerid, params[])
{
new pID,grund,string[256],pIDname[MAX_PLAYERS],pname[MAX_PLAYERS];
GetPlayerName(pID,pIDname,sizeof(pIDname));
GetPlayerName(playerid, pname,sizeof(pname));
if(sscanf(params,"ds",pID,grund)) return SendClientMessage(playerid,COLOR_WHITE,"Fehler: /kick [ID] [Grund]");
if(Spieler[playerid][Admin] < 3) return SendClientMessage(playerid,COLOR_RED2,"Du bist kein Server Leiter.");
format(string,sizeof(string),"%s wurde von %s gekickt. Grund: %s",pIDname,pname,grund);
SendClientMessageToAll(COLOR_RED2,string);
Kick(pID);
return 1;
}

ocmd:ban(playerid, params[])
{
new pID,grund,string[256],pIDname[MAX_PLAYERS],pname[MAX_PLAYERS];
GetPlayerName(pID,pIDname,sizeof(pIDname));
GetPlayerName(playerid, pname,sizeof(pname));
if(sscanf(params,"ds",pID,grund)) return SendClientMessage(playerid,COLOR_WHITE,"Fehler: /ban [ID] [Grund]");
if(Spieler[playerid][Admin] < 3) return SendClientMessage(playerid,COLOR_RED2,"Du bist kein Server Leiter.");
format(string,sizeof(string),"%s wurde von %s gebannt. Grund: %s",pIDname,pname,grund);
SendClientMessageToAll(COLOR_RED2,string);
Spieler[pID][gebannt] = 1;
Kick(pID);
return 1;
}

Wenn ich eins der beiden Befehle ausführe, bricht der Server zusammen (er stoppt)

Hoffentlch könnt ihr mirhefen

MfG

DeeKay(GER) · 17. Juli 2013

if(sscanf(params,"ds",pID,grund)) return SendClientMessage(playerid,COLOR_WHITE,"Fehler: /kick [ID] [Grund]");

Das ist der Fehler, die ID ist zwar eine Zahl aber sie wird mit "u" und nicht mit "d" abgefragt, das ist wahrscheinlich der Grund wieso Dein Server crasht.

So muss es richtig sein:

if(sscanf(params,"us",pID,grund)) return SendClientMessage(playerid,COLOR_WHITE,"Fehler: /kick [ID] [Grund]");

Und ab 0.3x musst Du mit einem Timer kicken sonst funktioniert der ganze Befehl nicht.

YaBroo · 17. Juli 2013

Zitat von DeeKay(GER)

if(sscanf(params,"ds",pID,grund)) return SendClientMessage(playerid,COLOR_WHITE,"Fehler: /kick [ID] [Grund]");

Das ist der Fehler, die ID ist zwar eine Zahl aber sie wird mit "u" und nicht mit "d" abgefragt, das ist wahrscheinlich der Grund wieso Dein Server crasht.

So muss es richtig sein:

if(sscanf(params,"us",pID,grund)) return SendClientMessage(playerid,COLOR_WHITE,"Fehler: /kick [ID] [Grund]");

Und ab 0.3x musst Du mit einem Timer kicken sonst funktioniert der ganze Befehl nicht.

Alles anzeigen

Man könnte auch "i" bei sscanf angeben bei "d" bin ich mir nicht sicher. Mit Beavis's [ INCLUDE ] Kick/Ban Fix
Include brauch man es nicht die Frage ist ob er's hat

DeeKay(GER) · 17. Juli 2013

Natürlich kann man das auch so machen das kommt auf's selbe heraus.

Das mit dem Include stimmt ja aber woher soll man wissen ob er dieses Include besitzt? Da sage ich es Ihm doch lieber nochmal zur Sicherheit.

YaBroo · 17. Juli 2013

Zitat von DeeKay(GER)

Natürlich kann man das auch so machen das kommt auf's selbe heraus.

Das mit dem Include stimmt ja aber woher soll man wissen ob er dieses Include besitzt? Da sage ich es Ihm doch lieber nochmal zur Sicherheit.

bei "i" crasht nichts und wenn du sagst es liegt an "d" dann kommt wohl nur indirekt das selbe raus.
"i" only ID u = playerid/name.
Wenn bei sscanf d&i das selbe ist dann kann es eig. nicht daran liegen.

firestormZ · 17. Juli 2013

muss nicht der grund auch zu nem String gemacht werden?
Also aus new grund -> new grund[256]

Blood. · 17. Juli 2013

Zitat von firestormZ

muss nicht der grund auch zu nem String gemacht werden?
Also aus new grund -> new grund[256]

Sicher? Soweit ich weiß nich.. Kann das wer bestätigen, da du dir ja nicht sicher bist.

Secres · 17. Juli 2013

Auf jeden Fall. Text ist immer String.
Außerdem solltest du aus MAX_PLAYERS bei pIDname und pname MAX_PLAYER_NAME machen.

YaBroo · 17. Juli 2013

Zitat von Blood.

Sicher? Soweit ich weiß nich.. Kann das wer bestätigen, da du dir ja nicht sicher bist.

d Integer Money
i Integer Money
c Character A single letter
u User ID or name Player
s String Any length of text
h Hex number Colour
x Hex number Colour
f Float Co-ordinate
z Optional string A reason e.g. Ban Reason, only optional at the end of a format string

firestormZ · 17. Juli 2013

ocmd:kick(playerid, params[])
{
new pID,grund[256],string[512],pIDname[MAX_PLAYERS],pname[MAX_PLAYERS];
GetPlayerName(pID,pIDname,sizeof(pIDname));
GetPlayerName(playerid, pname,sizeof(pname));
if(sscanf(params,"uz",pID,grund)) return SendClientMessage(playerid,COLOR_WHITE,"Fehler: /kick [ID] [Grund]");
if(Spieler[playerid][Admin] < 3) return SendClientMessage(playerid,COLOR_RED2,"Du bist kein Server Leiter.");
format(string,sizeof(string),"%s wurde von %s gekickt. Grund: %s",pIDname,pname,grund[0] ? grund : "/");
SendClientMessageToAll(COLOR_RED2,string);
Kick(pID);
return 1;
}

ocmd:ban(playerid, params[])
{
new pID,grund[256],string[512],pIDname[MAX_PLAYERS],pname[MAX_PLAYERS];
GetPlayerName(pID,pIDname,sizeof(pIDname));
GetPlayerName(playerid, pname,sizeof(pname));
if(sscanf(params,"uz",pID,grund)) return SendClientMessage(playerid,COLOR_WHITE,"Fehler: /ban [ID] [Grund]");
if(Spieler[playerid][Admin] < 3) return SendClientMessage(playerid,COLOR_RED2,"Du bist kein Server Leiter.");
format(string,sizeof(string),"%s wurde von %s gebannt. Grund: %s",pIDname,pname,grund[0] ? grund : "/");
SendClientMessageToAll(COLOR_RED2,string);
Spieler[pID][gebannt] = 1;
Kick(pID);
return 1;
}
Hab dir nochmal eben eingebaut, dass wenn man kein Grund angibt "/" angegeben wird.
Müsste so klappen.
mfg

Blood. · 17. Juli 2013

Kann man die Länge nicht mit strlen rausfinden? Wenn ja wie?
So gehts nicht :S:

if(strlen(grund[50] > 0)) return SendClientMessage(playerid,COLOR_WHITE,"Fehler: Du musst einen Grund angeben.");

YaBroo · 17. Juli 2013

Zitat von Blood.

Kann man die Länge nicht mit strlen rausfinden? Wenn ja wie?
So gehts nicht :S:

if(strlen(grund[50] > 0)) return SendClientMessage(playerid,COLOR_WHITE,"Fehler: Du musst einen Grund angeben.");

Wenn die Parameter nicht komplett Sind dann macht sscanf das doch schon automatisch.

Sakey · 17. Juli 2013

Zitat

grund[256],string[512]

Seit ihr Irre?! Seid wann gibt der Chat den über 760 Zeichen an?! Das zieht doch nur Unnötig...

new pID, grund[30], string[128], pIDname[MAX_PLAYERS], pname[MAX_PLAYERS];

So würde das nun richtig aussehen:

ocmd:kick(playerid, params[])
{
new pID,grund[30],string[128],pIDname[MAX_PLAYER_NAME],pname[MAX_PLAYER_NAME];
GetPlayerName(pID,pIDname,sizeof(pIDname));
GetPlayerName(playerid, pname,sizeof(pname));
if(sscanf(params,"uz",pID,grund)) return SendClientMessage(playerid,COLOR_WHITE,"Fehler: /kick [ID] [Grund]");
if(Spieler[playerid][Admin] < 3) return SendClientMessage(playerid,COLOR_RED2,"Du bist kein Server Leiter.");
format(string,128,"%s wurde von %s gekickt. Grund: %s",pIDname,pname,grund);
SendClientMessageToAll(COLOR_RED2,string);
Kick(pID);
return 1;
}

ocmd:ban(playerid, params[])
{
new pID, grund[30], string[128], pIDname[MAX_PLAYER_NAME], pname[MAX_PLAYER_NAME];
GetPlayerName(pID,pIDname,sizeof(pIDname));
GetPlayerName(playerid, pname,sizeof(pname));
if(sscanf(params,"uz",pID,grund)) return SendClientMessage(playerid,COLOR_WHITE,"Fehler: /ban [ID] [Grund]");
if(Spieler[playerid][Admin] < 3) return SendClientMessage(playerid,COLOR_RED2,"Du bist kein Server Leiter.");
format(string,128,"%s wurde von %s gebannt. Grund: %s",pIDname,pname,grund);
SendClientMessageToAll(COLOR_RED2,string);
Spieler[pID][gebannt] = 1;
Kick(pID);
return 1;
}

firestormZ · 17. Juli 2013

if(strlen(grund[0])) return SendClientMessage(playerid,COLOR_WHITE,"Fehler: Du musst einen Grund angeben.");

Blood. · 17. Juli 2013

Zitat von *GHOSTRIDER*

Wenn die Parameter nicht komplett Sind dann macht sscanf das doch schon automatisch.

Und wenn ich sagen will, dass man nur max. 20 Zeichen benutzen darf?

Zitat von Famacertech

Zitat

grund[256],string[512]

Seit ihr Irre?! Seid wann gibt der Chat den über 760 Zeichen an?! Das zieht doch nur Unnötig...

PAWN Quelltext

1

new pID, grund[30], string[128], pIDname[MAX_PLAYERS], pname[MAX_PLAYERS];

So würde das nun richtig aussehen:

Zitat

ocmd:kick(playerid, params[])
{
new pID,grund[30],string[128],pIDname[MAX_PLAYER_NAME],pname[MAX_PLAYER_NAME];
GetPlayerName(pID,pIDname,sizeof(pIDname));
GetPlayerName(playerid, pname,sizeof(pname));
if(sscanf(params,"uz",pID,grund)) return SendClientMessage(playerid,COLOR_WHITE,"Fehler: /kick [ID] [Grund]");
if(Spieler[playerid][Admin] < 3) return SendClientMessage(playerid,COLOR_RED2,"Du bist kein Server Leiter.");
format(string,128,"%s wurde von %s gekickt. Grund: %s",pIDname,pname,grund[0] ? grund : "/"); //Da kann ich nichts garantieren! Das hatte ich noch nie gemacht...
SendClientMessageToAll(COLOR_RED2,string);
Kick(pID);
return 1;
}

ocmd:ban(playerid, params[])
{
new pID, grund[30], string[128], pIDname[MAX_PLAYER_NAME], pname[MAX_PLAYER_NAME];
GetPlayerName(pID,pIDname,sizeof(pIDname));
GetPlayerName(playerid, pname,sizeof(pname));
if(sscanf(params,"uz",pID,grund)) return SendClientMessage(playerid,COLOR_WHITE,"Fehler: /ban [ID] [Grund]");
if(Spieler[playerid][Admin] < 3) return SendClientMessage(playerid,COLOR_RED2,"Du bist kein Server Leiter.");
format(string,128,"%s wurde von %s gebannt. Grund: %s",pIDname,pname,grund[0] ? grund : "/"); //Hier das gleiche wie da oben!
SendClientMessageToAll(COLOR_RED2,string);
Spieler[pID][gebannt] = 1;
Kick(pID);
return 1;
}

Alles anzeigen

Keine sorge hab ich schon gemacht ;D

firestormZ · 17. Juli 2013

if(!strlen(grund[0])) return SendClientMessage(playerid,ROT,"Fehler: Du musst einen Grund angeben.");

sry, so ises richtig

YaBroo · 17. Juli 2013

Dann kannste des auch so machen

if(strlen(grund < 20)) return SendClientMessage(playerid,-1,"Es müssen mindestens 20 Zeichen vorhanden sein");

Sakey · 17. Juli 2013

Zitat

Keine sorge hab ich schon gemacht ;D

Was hast du gemacht?

firestormZ · 17. Juli 2013

das "!" mus nach der "if("

Blood. · 17. Juli 2013

Zitat von *GHOSTRIDER*

if(strlen(grund < 20))

C:\Users\pc\Desktop\Server\pawno\Deathmatch.pwn(542) : error 033: array must be indexed (variable "grund")

Gibt er mir aus. bei grund[50] gibt er mir auch inen Error an