Tutorial Komplikationen

SpielerSpeichern(playerid); //Wieso speichert man die Stats von dem Spieler ab, welcher den Befehl nuttz ?
vollkommen richtig, vollkommen unnötig

SpielerSpeichern(pID); // Es macht doch gar keinen Sinn die Stats abzuspeichern, wenn der Spieler schon vom Server
Das gehört über Kick(pID)...

FreezePlayer(pID);
und das ist unnötig

So ist es richtig:

format(string,sizeof(string),"[AdmCmd] %s hat %s vom Server gekickt. Grund: %s",SpielerName(playerid),SpielerName(pID),grund);
SendClientMessageToAll(ROT,string);
SpielerSpeichern(pID); //ausser du hast es schon unter OnPlayerDisconnect
Kick(pID);
return 1;

Das macht keinen Sinn, schreibe das mal so:

ocmd:kick(playerid,params[])
{
new pID,string[128],grund[128];
if(!IstSpielerAdmin(playerid,1)) return SendClientMessage(playerid,ROT,"Dein Adminlevel ist zu niedrig");
if(sscanf(params,"us",pID,grund))return SendClientMessage(playerid,ROT,"Benutzung: /kick [ID] [Grund]");
if(!IsPlayerConnected(pID))return SendClientMessage(playerid,ROT,"Der angegebene Spieler ist nicht connected!");
format(string,sizeof(string),"[AdmCmd] %s hat %s vom Server gekickt. Grund: %s",SpielerName(pID),SpielerName(playerid),grund); // Müssten die zwei Methoden nicht genau anders herum ? Also zuerst SpielerName(playerid) ?
SendClientMessageToAll(ROT,string);
FreezePlayer(pID);
Kick(pID);
return 1;
}

Ich habe die Save Sachen rausgenommen, da der Spieler eh gespeichert wird, durch den Callback OnPlayerDisconnect

mfg.