Multiaccount-System funktioniert nicht.

Pagno · 3. Dezember 2016

Hey,

ich habe mithilfe von einer Person ein Multiaccount-Erkennungssystem geschrieben, dennoch klappt dies nicht wie gewünscht :

Das sind dennoch 2 völlig unterschiedliche Spieler mit verschiedenen IPs.
Das System erfasst aus irgendeinem Grund alle Spieler die in der Datenbank existieren....

Aber es soll nur Leute erfassen die mit der gleicher IP registriert sind ( wurden ).

Hier der Code bei OnPlayerConnect :

Code

/*
    new huan[256];
	for(new i=0, j = GetPlayerPoolSize(); i <= j; i++)
	{
		if(!IsPlayerConnected(i))continue;
		if(IsPlayerNPC(i))continue;
		if(strcmp(GetIP(playerid), GetIP(i), true))continue; // Wenn ip_address noch nicht auf dem Server gefunden wurde!
		//ip_address stimmt mit bereits verbundener IP überein
	    format(huan,sizeof(huan),"[MULTIACCOUNT] %s(%d) (IP: %s) wurde vom System erkannt! Multiaccount mit der ID %d (Name: %s, IP: %s)", PlayerName(playerid), playerid, GetIP(playerid), i, PlayerName(i), GetIP(i));
	    SendClientMessageToAdmins(huan, GELB);
	    print(huan);
	}


	//Check für offline accounts


	mysql_query("SELECT MAX(id) FROM spieler");
	mysql_store_result();
	new LAST_ID_ENTRY = mysql_fetch_int();
	mysql_free_result();


	for(new i=0; i<LAST_ID_ENTRY+1; i++)
	{
		if(mysql_CheckAccountID(i)==0)continue;
		if(strcmp(GetIP(playerid), mysql_GetString_ID("spieler", "last_ip", "id", i), true))continue;
		if(!strcmp(PlayerName(playerid), mysql_GetString_ID("spieler", "name", "id", i), true))continue;
		format(huan,sizeof(huan),"[MULTIACCOUNT] Der Spieler %s betreibt Multiaccounting! IP: %s (Übereinstimmung mit %s)", PlayerName(playerid), GetIP(playerid), mysql_GetString_ID("spieler", "name", "id", i));
		SendClientMessageToAdmins(huan, GELB);
		print(huan);
 	}
 	*/

Alles anzeigen

Manueller Befehl dafür :

Code

ocmd:macheck(playerid, params[])
{
    if(!IsAdmin(playerid,1))return noaccess
    new pid;
	if(sscanf(params,"u",pid))return SendClientMessage(playerid,WEIß,"Verwendung: /macheck (Player/ID)");
	if(!PlayerOnline(pid))return SendClientMessage(playerid, GRAU, "Der Spieler ist nicht online!");

	new bool:findings;

	new huan[256];
	for(new i=0, j = GetPlayerPoolSize(); i <= j; i++)
	{
	    if(i==pid)continue; // sich selbst ausblenden
		if(!IsPlayerConnected(i))continue;
		if(IsPlayerNPC(i))continue;
		if(strcmp(GetIP(pid), GetIP(i), true))continue; // Wenn ip_address noch nicht auf dem Server gefunden wurde!
		//ip_address stimmt mit bereits verbundener IP überein
	    format(huan,sizeof(huan),"[MULTIACCOUNT] %s(%d) (IP: %s) wurde vom System erkannt! Multiaccount mit der ID %d (Name: %s, IP: %s)", PlayerName(pid), pid, GetIP(pid), i, PlayerName(i), GetIP(i));
	    SendClientMessageToAdmins(huan, ROT);
	    findings=true;
	}


	//Check für offline accounts


	mysql_query("SELECT MAX(id) FROM spieler");
	mysql_store_result();
	new LAST_ID_ENTRY = mysql_fetch_int();
	mysql_free_result();


	for(new i=0; i<LAST_ID_ENTRY+1; i++)
	{
		if(mysql_CheckAccountID(i)==0)continue;
		if(strcmp(GetIP(pid), mysql_GetString_ID("spieler", "last_ip", "id", i), true))continue;
		if(!strcmp(PlayerName(pid), mysql_GetString_ID("spieler", "name", "id", i), true))continue; // sich selbst ausblenden
		format(huan,sizeof(huan),"[MULTIACCOUNT] Der Spieler %s betreibt Multiaccounting! IP: %s (Übereinstimmung mit %s)", PlayerName(pid), GetIP(pid), mysql_GetString_ID("spieler", "name", "id", i));
		SendClientMessageToAdmins(huan, ROT);
		findings=true;
 	}

	if(!findings)return SendClientMessage(playerid, GRAU, "Keine Multiaccounts gefunden bzw. vorhanden.");
	return 1;
}

Alles anzeigen

Jeffry · 4. Dezember 2016

Wenn du mal 10000 registrierte User hast, dann schmiert dir bei diesem Code der Server ab, ob steht für einige Zeit, da du dann beim Connect des Users mindestens 30000 Queries absendest. Selbst wenn jedes Query nur 10ms braucht um zu antworten (was sehr gut wäre), würde der Server 300 Sekunden lang stehen (5 Minuten).
Danach fliegen wahrscheinlich alle User vom Server wegen Timeout oder ähnlichem.

Selbst bei 300 registrierten Usern wären es schon mindestens 10 Sekunden Stillstand.

Du kannst das ganze mit einem einzigen Query abfragen, anstatt jeden Spieler in der Datenbank zu prüfen.

SQL

SELECT id, name FROM spieler WHERE last_ip = '%s' AND name != '%s' ORDER BY id DESC;

So bekommst du alle Konten mit der gleichen IP zurück, das neuste als erstes.

Pagno · 4. Dezember 2016

Wo genau muss ich diesen Code dann ersetzen?

Jeffry · 4. Dezember 2016

Das ersetzt dann alles was zu
//Check für offline accounts
gehört.

Pagno · 4. Dezember 2016

Also der Code lautet jetzt so :

Code

public OnPlayerConnect(playerid)
{
	if(IsPlayerNPC(playerid))return 1;


    new huan[256];
	for(new i=0, j = GetPlayerPoolSize(); i <= j; i++)
	{
		if(!IsPlayerConnected(i))continue;
		if(IsPlayerNPC(i))continue;
		if(strcmp(GetIP(playerid), GetIP(i), true))continue; // Wenn ip_address noch nicht auf dem Server gefunden wurde!
		//ip_address stimmt mit bereits verbundener IP überein
	    format(huan,sizeof(huan),"[MULTIACCOUNT] %s(%d) (IP: %s) wurde vom System erkannt! Multiaccount mit der ID %d (Name: %s, IP: %s)", PlayerName(playerid), playerid, GetIP(playerid), i, PlayerName(i), GetIP(i));
	    SendClientMessageToAdmins(huan, GELB);
	    print(huan);
	}


	//Check für offline accounts


	SELECT id, name FROM spieler WHERE last_ip = '%s' AND name != '%s' ORDER BY id DESC;
	format(huan,sizeof(huan),"[MULTIACCOUNT] Der Spieler %s betreibt Multiaccounting! IP: %s (Übereinstimmung mit %s)", PlayerName(playerid), GetIP(playerid), mysql_GetString_ID("spieler", "name", "id", i));
	SendClientMessageToAdmins(huan, GELB);
	print(huan);

Alles anzeigen

Bekomme 4 Errors :

Code

C:\Users\Drai\Desktop\Server\gamemodes\gamemode.pwn(3376) : error 017: undefined symbol "SELECT"
C:\Users\Drai\Desktop\Server\gamemodes\gamemode.pwn(3376) : error 017: undefined symbol "id"
C:\Users\Drai\Desktop\Server\gamemodes\gamemode.pwn(3376) : error 017: undefined symbol "FROM"
C:\Users\Drai\Desktop\Server\gamemodes\gamemode.pwn(3376) : fatal error 107: too many error messages on one line


Compilation aborted.Pawn compiler 3.2.3664	 	 	Copyright (c) 1997-2006, ITB CompuPhase




4 Errors.

Alles anzeigen

Jeffry · 4. Dezember 2016

Natürlich sollst du das Query nicht einfach nur in den Code einfügen, das macht ja keinen Sinn.

format(query, sizeof(query), "SELECT id, name FROM spieler WHERE last_ip = '%s' AND name != '%s' ORDER BY id DESC;", GetName(playerid), GetIP(playerid));
mysql_query(query);
mysql_store_result();
new rows = mysql_num_rows();
//...Result hier auslesen.

Beispiel hier:
http://wiki.sa-mp.com/wiki/MySQL#mysql_retrieve_row

Das solltest du aber auch bereits in deinem Code an anderen Stellen so oder ähnlich haben, du lässt ja die Stats auch laden.

Pagno · 4. Dezember 2016

Habe wirklich 0 Plan von MYSQL wie man das macht, tut mir Leid.

Kenne mich damit kein bisschen, hatte den Code auch nicht selber geschrieben, sondern schreiben lassen, habe das jetzt so :

Errors :

Code

C:\Users\Drai\Desktop\Server\gamemodes\gamemode.pwn(3377) : error 035: argument type mismatch (argument 1)
C:\Users\Drai\Desktop\Server\gamemodes\gamemode.pwn(3377) : error 035: argument type mismatch (argument 1)
C:\Users\Drai\Desktop\Server\gamemodes\gamemode.pwn(3378) : error 035: argument type mismatch (argument 1)
C:\Users\Drai\Desktop\Server\gamemodes\gamemode.pwn(3380) : warning 204: symbol is assigned a value that is never used: "rows"
C:\Users\Drai\Desktop\Server\gamemodes\gamemode.pwn(3376) : warning 203: symbol is never used: "query"
Pawn compiler 3.2.3664	 	 	Copyright (c) 1997-2006, ITB CompuPhase




3 Errors.

Alles anzeigen

Code

//Check für offline accounts
   	for(new i=0, j = GetPlayerPoolSize(); i <= j; i++)
	{
		new query;
		format(query, sizeof(query), "SELECT id, name FROM spieler WHERE last_ip = '%s' AND name != '%s' ORDER BY id DESC;", GetName(playerid), GetIP(playerid));
		mysql_query(query);
		mysql_store_result();
		new rows = mysql_num_rows();
		format(huan,sizeof(huan),"[MULTIACCOUNT] Der Spieler %s betreibt Multiaccounting! IP: %s (Übereinstimmung mit %s)", PlayerName(playerid), GetIP(playerid), mysql_GetString_ID("spieler", "name", "id", i));
		SendClientMessageToAdmins(huan, GELB);
		print(huan);
	}

Alles anzeigen

Jeffry · 4. Dezember 2016

Dann solltest du es dringendst lernen.
Es bringt ja nichts, wenn dir dein Server von jemand anderem geschrieben wird.

Der Code sollte so aussehen:
new huan[145];
for(new i=0, j = GetPlayerPoolSize(); i <= j; i++)
{
if(!IsPlayerConnected(i))continue;
if(IsPlayerNPC(i))continue;
if(strcmp(GetIP(playerid), GetIP(i), true))continue; // Wenn ip_address noch nicht auf dem Server gefunden wurde!
//ip_address stimmt mit bereits verbundener IP überein
format(huan,sizeof(huan),"[MULTIACCOUNT] %s(%d) (IP: %s) wurde vom System erkannt! Multiaccount mit der ID %d (Name: %s, IP: %s)", PlayerName(playerid), playerid, GetIP(playerid), i, PlayerName(i), GetIP(i));
SendClientMessageToAdmins(huan, GELB);
print(huan);
}
//Check für offline accounts
format(query, sizeof(query), "SELECT id, name FROM spieler WHERE last_ip = '%s' AND name != '%s' ORDER BY id DESC;", PlayerName(playerid), GetIP(playerid));
mysql_query(query);
mysql_store_result();
new rows = mysql_num_rows(), id, name[MAX_PLAYER_NAME];
for(new i=0; i<rows; i++)
{
mysql_retrieve_row();
mysql_fetch_field_row(huan,"id"); id = strval(huan);
mysql_fetch_field_row(name,"name");
format(huan,sizeof(huan),"[MULTIACCOUNT] Der Spieler %s betreibt Multiaccounting! IP: %s (Übereinstimmung mit %s)", PlayerName(playerid), GetIP(playerid), name);
SendClientMessageToAdmins(huan, GELB);
print(huan);
}
mysql_free_result();

Pagno · 4. Dezember 2016

Code

C:\Users\--\Desktop\Server\gamemodes\gamemode.pwn(3374) : error 035: argument type mismatch (argument 1)
C:\Users\--\Desktop\Server\gamemodes\gamemode.pwn(3374) : error 035: argument type mismatch (argument 1)
C:\Users\--\Desktop\Server\gamemodes\gamemode.pwn(3375) : error 035: argument type mismatch (argument 1)
C:\Users\--\Desktop\Server\gamemodes\gamemode.pwn(3381) : warning 204: symbol is assigned a value that is never used: "id"
C:\Users\--\Desktop\Server\gamemodes\gamemode.pwn(3373) : warning 203: symbol is never used: "query"
Pawn compiler 3.2.3664	 	 	Copyright (c) 1997-2006, ITB CompuPhase




3 Errors.

Alles anzeigen

Jeffry · 4. Dezember 2016

Würdest du die entsprechenden Zeilen im Code markieren?

Pagno · 4. Dezember 2016

Code

new huan[145];
	for(new i=0, j = GetPlayerPoolSize(); i <= j; i++)
	{
		if(!IsPlayerConnected(i))continue;
		if(IsPlayerNPC(i))continue;
		if(strcmp(GetIP(playerid), GetIP(i), true))continue; // Wenn ip_address noch nicht auf dem Server gefunden wurde!
		//ip_address stimmt mit bereits verbundener IP überein
		format(huan,sizeof(huan),"[MULTIACCOUNT] %s(%d) (IP: %s) wurde vom System erkannt! Multiaccount mit der ID %d (Name: %s, IP: %s)", PlayerName(playerid), playerid, GetIP(playerid), i, PlayerName(i), GetIP(i));
		SendClientMessageToAdmins(huan, GELB);
		print(huan);
	}
	//Check für offline accounts
	new query;
	format(query, sizeof(query), "SELECT id, name FROM spieler WHERE last_ip = '%s' AND name != '%s' ORDER BY id DESC;", PlayerName(playerid), GetIP(playerid)); 
	mysql_query(query);
	mysql_store_result();
	new rows = mysql_num_rows(), id, name[MAX_PLAYER_NAME];
	for(new i=0; i<rows; i++)
	{
		mysql_retrieve_row();
		mysql_fetch_field_row(huan,"id"); id = strval(huan);
		mysql_fetch_field_row(name,"name");
		format(huan,sizeof(huan),"[MULTIACCOUNT] Der Spieler %s betreibt Multiaccounting! IP: %s (Übereinstimmung mit %s)", PlayerName(playerid), GetIP(playerid), name);
		SendClientMessageToAdmins(huan, GELB);
		print(huan);
	}
	mysql_free_result();

Alles anzeigen

3374 : Zeile 14

3375 : Zeile 15

3371 : Zeile 21

3373 : Zeile 13

Jeffry · 4. Dezember 2016

new query;
zu:
new query[256];

format(huan,sizeof(huan),"[MULTIACCOUNT] Der Spieler %s betreibt Multiaccounting! IP: %s (Übereinstimmung mit %s)", PlayerName(playerid), GetIP(playerid), name);
zu:
format(huan,sizeof(huan),"[MULTIACCOUNT] Der Spieler %s betreibt Multiaccounting! IP: %s (Übereinstimmung mit %s (ID: %d))", PlayerName(playerid), GetIP(playerid), name, id);

Pagno · 4. Dezember 2016

Alles Klar danke dir,

hat geklappt, ich versuche es mal wenn ich die Zeit dafür finde bzw. ein Tester online ist.

//
Habe es mit mir selber getestet und es printen lassen.

Hatte 2 Accounts und dann wurde geprintet :

[15:16:37] [MULTIACCOUNT] Der Spieler lolz betreibt Multiaccounting! IP:************** (Übereinstimmung mit Pagno)

Aber wenn ich /macheck (manueller Befehl) verwendet habe kam "Keine Multiaccounts vorhanden".

Hier der Befehl :

Code

ocmd:macheck(playerid, params[])
{
    if(!IsAdmin(playerid,1))return noaccess
    new pid;
	if(sscanf(params,"u",pid))return SendClientMessage(playerid,WEIß,"Verwendung: /macheck (Player/ID)");
	if(!PlayerOnline(pid))return SendClientMessage(playerid, GRAU, "Der Spieler ist nicht online!");
	new bool:findings;
	new huan[256];
	for(new i=0, j = GetPlayerPoolSize(); i <= j; i++)
	{
	    if(i==pid)continue; // sich selbst ausblenden
		if(!IsPlayerConnected(i))continue;
		if(IsPlayerNPC(i))continue;
		if(strcmp(GetIP(pid), GetIP(i), true))continue; // Wenn ip_address noch nicht auf dem Server gefunden wurde!
		//ip_address stimmt mit bereits verbundener IP überein
	    format(huan,sizeof(huan),"[MULTIACCOUNT] %s(%d) (IP: %s) wurde vom System erkannt! Multiaccount mit der ID %d (Name: %s, IP: %s)", PlayerName(pid), pid, GetIP(pid), i, PlayerName(i), GetIP(i));
	    SendClientMessageToAdmins(huan, ROT);
	    findings=true;
	}


	//Check für offline accounts


	new query[256];
	format(query, sizeof(query), "SELECT id, name FROM spieler WHERE last_ip = '%s' AND name != '%s' ORDER BY id DESC;", PlayerName(playerid), GetIP(playerid));
	mysql_query(query);
	mysql_store_result();
	new rows = mysql_num_rows(), id, name[MAX_PLAYER_NAME];
	for(new i=0; i<rows; i++)
	{
		mysql_retrieve_row();
		mysql_fetch_field_row(huan,"id"); id = strval(huan);
		mysql_fetch_field_row(name,"name");
		format(huan,sizeof(huan),"[MULTIACCOUNT] Der Spieler %s betreibt Multiaccounting! IP: %s (Übereinstimmung mit %s (ID: %d))", PlayerName(playerid), GetIP(playerid), name, id);
		SendClientMessageToAdmins(huan, GELB);
		print(huan);
	}
	mysql_free_result();
	if(!findings)return SendClientMessage(playerid, GRAU, "Keine Multiaccounts gefunden bzw. vorhanden.");
	return true;
}

Alles anzeigen

Stimmen die IDs? @Jeffry

Jeffry · 4. Dezember 2016

Füge nach dem print(huan) das ein:
findings=true;

Pagno · 4. Dezember 2016

Es kommt immeronch keine Multiaccounts vorhanden.

//

Ich habe bemerkt dass nicht alle User in der Datenbank eine IP zugewiesen bekommen haben in der Spalte last_ip.

Code

mysql_SetString("spieler", "last_ip", GetIP(playerid), "name", PlayerName(playerid));

Benutze ich wenn ich den Account erstelle (Stock).

Also mein ganzer Befehl sieht so aus :

Code

ocmd:macheck(playerid, params[])
{
    if(!IsAdmin(playerid,1))return noaccess
    new pid;
	if(sscanf(params,"u",pid))return SendClientMessage(playerid,WEIß,"Verwendung: /macheck (Player/ID)");
	if(!PlayerOnline(pid))return SendClientMessage(playerid, GRAU, "Der Spieler ist nicht online!");

	new bool:findings;

	new huan[256];
	for(new i=0, j = GetPlayerPoolSize(); i <= j; i++)
	{
	    if(i==pid)continue; // sich selbst ausblenden
		if(!IsPlayerConnected(i))continue;
		if(IsPlayerNPC(i))continue;
		if(strcmp(GetIP(pid), GetIP(i), true))continue; // Wenn ip_address noch nicht auf dem Server gefunden wurde!
		//ip_address stimmt mit bereits verbundener IP überein
	    format(huan,sizeof(huan),"[MULTIACCOUNT] %s(%d) (IP: %s) wurde vom System erkannt! Multiaccount mit der ID %d (Name: %s, IP: %s)", PlayerName(pid), pid, GetIP(pid), i, PlayerName(i), GetIP(i));
	    SendClientMessageToAdmins(huan, ROT);
	    findings=true;
	}


	//Check für offline accounts


	new query[256];
	format(query, sizeof(query), "SELECT id, name FROM spieler WHERE last_ip = '%s' AND name != '%s' ORDER BY id DESC;", PlayerName(playerid), GetIP(playerid));
	mysql_query(query);
	mysql_store_result();
	new rows = mysql_num_rows(), id, name[MAX_PLAYER_NAME];
	for(new i=0; i<rows; i++)
	{
		mysql_retrieve_row();
		mysql_fetch_field_row(huan,"id"); id = strval(huan);
		mysql_fetch_field_row(name,"name");
		format(huan,sizeof(huan),"[MULTIACCOUNT] Der Spieler %s betreibt Multiaccounting! IP: %s (Übereinstimmung mit %s (ID: %d))", PlayerName(playerid), GetIP(playerid), name, id);
		SendClientMessageToAdmins(huan, GELB);
		print(huan);
		findings=true;
	}
	mysql_free_result();
	if(!findings)return SendClientMessage(playerid, GRAU, "Keine Multiaccounts gefunden bzw. vorhanden.");
	return 1;
}

Alles anzeigen

Akino Kiritani · 4. Dezember 2016

Zitat von Drai

Benutze ich wenn ich den Account erstelle (Stock).

Das sollte eigentlich immer benutzt werden, quasi sobald der Spieler auf den Server connected.

Ich find deinen Code auch super Merkwürdig,
du schleißt aus deinem "Online Check" den Spieler den du suchst bereits aus

for(new i=0, j = GetPlayerPoolSize(); i <= j; i++)
{
if(i==pid)continue; // sich selbst ausblenden <-- hier sollte vermutlich playerid stehen und nicht pid
// pid ist die ID die du suchst

Du hast hier aber ein problem, das System würde immer anschlagen, schließlich befindet sich der Spieler ja auf dem Server.

Es besteht natürlich das Problem, dass es 2 Spieler im gleichen Haushalt geben könnte, nur wäre das irgendwie kein (wirklicher) Multiaccount.
Es handelt sich da ja um 2 Leute.

mMn solltest du den Online Check einfach streichen, denn der "offline" Check reicht völlig aus.
Die IPs(Daten) sind ohnehin schon in der Datenbank, warum nicht einfach die Daten nehmen die man schon hat ?

Pagno · 5. Dezember 2016

Hey,

habe die Online-Prüfung weggelassen, sondern lasse jetzt durch /macheck (pid) jetzt die Offline-Accounts bzw. die Accounts in der Datenbank finden, aber leider klappt das nicht so ganz.

Code

ocmd:macheck(playerid, params[])
{
    if(!IsAdmin(playerid,1))return noaccess
    new pid;
if(sscanf(params,"u",pid))return SendClientMessage(playerid,WEIß,"Verwendung: /macheck (Player/ID)");
if(!PlayerOnline(pid))return SendClientMessage(playerid, GRAU, "Der Spieler ist nicht online!");
new bool:findings;
new huan[256];


//Check für offline accounts
new query[256];
format(query, sizeof(query), "SELECT id, name FROM spieler WHERE last_ip = '%s' AND name != '%s' ORDER BY id DESC;", PlayerName(playerid), GetIP(playerid));
mysql_query(query);
mysql_store_result();
new rows = mysql_num_rows(), id, name[MAX_PLAYER_NAME];
for(new i=0; i<rows; i++)
{
mysql_retrieve_row();
mysql_fetch_field_row(huan,"id"); id = strval(huan);
mysql_fetch_field_row(name,"name");
format(huan,sizeof(huan),"[MULTIACCOUNT] Der Spieler %s betreibt Multiaccounting! IP: %s (Übereinstimmung mit %s (ID: %d))", PlayerName(playerid), GetIP(playerid), name, id);
SendClientMessageToAdmins(huan, GELB);
print(huan);
findings=true;
}
mysql_free_result();
if(!findings)return SendClientMessage(playerid, GRAU, "Keine Multiaccounts gefunden bzw. vorhanden.");
return 1;
}

Alles anzeigen

Es kommt einfach immer keine Multiaccounts gefunden, obwohl ich 2 Accounts in der Datenbank habe, mit der ich mit der gleichen IP registriert habe.

Akino Kiritani · 5. Dezember 2016

Zitat von Drai

format(query, sizeof(query), "SELECT id, name FROM spieler WHERE last_ip = '%s' AND name != '%s' ORDER BY id DESC;", PlayerName(playerid), GetIP(playerid));

Du hast hier
1. PlayerName und GetIP vertauscht (das müsstest du sogar in deiner Log sehen)
2. nimmst du die IP von ausführenden Spieler, den suchst du aber nicht.
Statt "playerid" einfach pid nehmen
Den gleichen Fehler machst du auch in deiner SendClientMessage

Pagno · 5. Dezember 2016

Also wäre es so richtig ? :

Code

new query[256];
	format(query, sizeof(query), "SELECT id, name FROM spieler WHERE last_ip = '%s' AND name != '%s' ORDER BY id DESC;", GetIP(pid), PlayerName(pid));
	mysql_query(query);
	mysql_store_result();
	new rows = mysql_num_rows(), id, name[MAX_PLAYER_NAME];
	for(new i=0; i<rows; i++)
	{
		mysql_retrieve_row();
		mysql_fetch_field_row(huan,"id"); id = strval(huan);
		mysql_fetch_field_row(name,"name");
		format(huan,sizeof(huan),"[MULTIACCOUNT] Der Spieler %s betreibt Multiaccounting! IP: %s (Übereinstimmung mit %s (ID: %d))", PlayerName(pid), GetIP(pid), name, id);
		SendClientMessageToAdmins(huan, GELB);
		print(huan);
		findings=true;
	}
	mysql_free_result();
	if(!findings)return SendClientMessage(playerid, GRAU, "Keine Multiaccounts gefunden bzw. vorhanden.");

Alles anzeigen

Akino Kiritani · 5. Dezember 2016

Zitat von Drai

format(query, sizeof(query), "SELECT id, name FROM spieler WHERE last_ip = '%s' AND name != '%s' ORDER BY id DESC;", GetIP(pid), PlayerName(pid));

Nicht ganz. Du willst, ja den der sucht ausschließen, also musst du schon playerid bei PlayerName lassen.
Ansonsten ist der Code in Ordnung.