Kick System Problem

Broxel · 4. Januar 2014

Hallo Leute,
ich habe hier ein Kicksystem, was perfekt funktioniert, aber es steht dann nur: wurde von gekickt. Grund: [Grund]
Allerdings soll da stehen [Spielername] wurde von [Supportername] gekickt. Grund [Grund]
Alles, was ich dazu habe(unter public onplayercommand) :

Code

dcmd(kick, 4, cmdtext);


dcmd_kick(playerid,params[])
	{
	if(IstSpielerAdmin(playerid, 1))
	{
		new pID, string[128], reason, name, pname;
		if(sscanf(params,"us",pID, reason))return SendClientMessage(playerid,WEISS, "Benutzung: /kick [ID][Grund]");
		format(string,sizeof(string),"%s wurde von %s gekickt. Grund: %s", pname, name, reason);
		SendClientMessageToAll(ADMIN, string);  //ADMIN ist eine Farbe
		Kick(pID);
	}
	return 1;
}

Alles anzeigen

Campbell · 4. Januar 2014

Warum ist reason ein Interger?
Warum ist name, pname ein Integer?
Wo liest du die Namen ein?

Broxel · 4. Januar 2014

Alles läuft ja perfekt, nur das die Namen nicht angezeigt werden

Campbell · 4. Januar 2014

Zitat von Broxel

Alles läuft ja perfekt, nur das die Namen nicht angezeigt werden

Das beantwortet nicht die Fragen. Stellst du dir die oben genannten Fragen, dann kommst du auch auf dein Problem.

Mentalist · 5. Januar 2014

Ungetestet. KickEx solltest du benutzen, da sonst die Message nicht richtig gesendet wird....
dcmd_kick(playerid,params[]) {
if(IstSpielerAdmin(playerid, 1)) {
new pID, lStr[128], reason[128], name[MAX_PLAYER_NAME], pName[MAX_PLAYER_NAME];
if(sscanf(params,"us", pID, reason)) return SendClientMessage(playerid,WEISS, "Benutzung: /kick [ID][Grund]");
GetPlayerName(playerid, name, sizeof(name));
GetPlayerName(pID, pName, sizeof(pName));
format(lStr, sizeof(lStr), "%s wurde von %s gekickt. Grund: %s", pName, name, reason);
SendClientMessageToAll(ADMIN, string);
KickEx(pID);
}
}
stock KickEx(playerid) {
SetTimerEx("KickExTimer", 300, false, "i", playerid);
}
forward KickExTimer(playerid);
public KickExTimer(playerid) {
Kick(playerid);
return true;
}

T4125Gamer · 5. Januar 2014

Man kann es auch mit diesem Kick/Ban Fix von Beavis machen.
[ INCLUDE ] Kick/Ban Fix

Campbell · 5. Januar 2014

Zitat von T4125Gamer

Man kann es auch mit diesem Kick/Ban Fix von Beavis machen.
[ INCLUDE ] Kick/Ban Fix

Ist doch das genau gleiche Prinzip. Zum Quellcode von Mentalist. Warum verwendest du zwei Strings von der Größe von 128? Das macht keinen Sinn. Vor allem auch wenn du den einen String in den anderen einsetzen möchtest. Auch kann man niemals 128 Zeichen als Grund eingeben!

Broxel · 5. Januar 2014

Perfekt geklappt.
Endergebnis ohne Fehler:

dcmd_kick(playerid,params[]) {
if(IstSpielerAdmin(playerid, 1)) {
new pID, lStr[64], reason[64], name[MAX_PLAYER_NAME], pName[MAX_PLAYER_NAME];
if(sscanf(params,"us", pID, reason)) return SendClientMessage(playerid,WEISS, "Benutzung: /kick [ID][Grund]");
GetPlayerName(playerid, name, sizeof(name));
GetPlayerName(pID, pName, sizeof(pName));
format(lStr, sizeof(lStr), "%s wurde von %s gekickt. Grund: %s", pName, name, reason);
SendClientMessageToAll(ADMIN, lStr);
KickEx(pID);
}
return 1;
}

stock KickEx(playerid) {
SetTimerEx("KickExTimer", 300, false, "i", playerid);
}

forward KickExTimer(playerid);

public KickExTimer(playerid) {
Kick(playerid);
return true;
}

Slash™ · 5. Januar 2014

Zitat von Campbell

Ist doch das genau gleiche Prinzip. Zum Quellcode von Mentalist. Warum verwendest du zwei Strings von der Größe von 128? Das macht keinen Sinn. Vor allem auch wenn du den einen String in den anderen einsetzen möchtest. Auch kann man niemals 128 Zeichen als Grund eingeben!

Du brauchst dich aber nicht extra an ne neue funkion gewönen.

Aquacold · 5. Januar 2014

Zitat von Broxel

name[MAX_PLAYER_NAME], pName[MAX_PLAYER_NAME];

Man macht [MAX_PLAYER_NAME+1]

Sonst wird bei einem User der einen Namen mit 24 Zeichen hat nur mit 23 angezeigt. (Der String beginnt mit 0)

Broxel · 5. Januar 2014

Habe ich nun geändert, danke für die Ergänzung.

Campbell · 5. Januar 2014

Das ist so nicht ganz korrekt.

Ein String wird NULL-terminiert. D.h. ein String endet mit einer 0 oder '\0'.
Derzeit können bei SA:MP nur Leute mit Namen von einer Länge zwischen 3 und 20 Zeichen verbinden. Folglich kann man MAX_PLAYER_NAME, was derzeit auf 24 gesetzt ist auf 21 setzen.

// NULL-Terminator:

string[4] = 0;

// oder:

string[4] = '\0';

// Beispiel:

new string[9];
string[0] = 83;
string[1] = 65;
string[2] = 77;
string[3] = 80;
string[4] = 124;
string[5] = 83;
string[6] = 65;
string[7] = 77;
string[8] = 80;
print(string); // Gibt aus: "SAMP|SAMP"

new string2[10];
string2[0] = 83;
string2[1] = 65;
string2[2] = 77;
string2[3] = 80;
string2[4] = '\0';
string2[5] = 124;
string2[6] = 83;
string2[7] = 65;
string2[8] = 77;
string2[9] = 80;
print(string2); // Gibt aus: "SAMP" - String wurde bei string[4] terminiert!

// Redefine:

#if defined MAX_PLAYER_NAME
#undef MAX_PLAYER_NAME
#define MAX_PLAYER_NAME (21)
#endif

breadfish.de 12. März 2022