Weiß nicht wie ich den Fehler beheben soll

Iceberqk · 14. April 2017

Moin Leute!

Ich habe mal heute an einem Ban-System gearbeitet.

Code

ocmd:ban(playerid,params[])
{
    if(!isAdmin(playerid,3))return SendClientMessage(playerid,Kaminrot,"ERROR: Du besitzt nicht die benötigten Rechte!");
	new pID,grund[128],message[128],dmessage[128],partm[128],query[128];
	if(sscanf(params,"is",pID,grund)) return SendClientMessage(playerid,Orange,"INFO: /ban [id] [grund]");
	if(!IsPlayerConnected(pID)) return SendClientMessage(playerid,Kaminrot,"ERROR: Die Spieler-ID existiert nicht.");
	//if(sInfo[pID][alevel] >= sInfo[playerid][alevel])return SendClientMessage(playerid,Kaminrot,"ERROR: Du kannst keinen höheren bzw. gleichrangigen Admin kicken!");
	format(dmessage,sizeof(dmessage),"Du wurdest von %s permanent vom Server gebannt.\nGrund: %s\nEntbannanträge an: {FF0000}asd",getPlayerName(playerid),grund);
    format(partm,sizeof(partm),"%s hat den Server verlassen.(Banned)",getPlayerName(pID));
	format(message,sizeof(message),"[BANNED] %s wurde von %s permanent vom Server gebannt. Grund: %s - Typ: Accountban",getPlayerName(pID),getPlayerName(playerid),grund);
	SendClientMessageToAll(Blutrot,message);
	SendClientMessageToAll(Grau,partm);
	ShowPlayerDialog(pID,DIALOG_KICKED,DIALOG_STYLE_MSGBOX,">> BANNED",dmessage,"OK","");
	TogglePlayerControllable(pID,false);
	sInfo[pID][banned]=1;
 	mysql_escape_string(grund,bangrund,dbhandle);
  	format(query,sizeof(query),"INSERT INTO user (bangrund) VALUES ('%s') ",grund);
   	mysql_function_query(dbhandle,query,false,"","");
	savePlayer(pID);
	Kick(pID);
	return 1;
}

Alles anzeigen

Ich bekomme folgenden Error:
\script.pwn(318) : error 035: argument type mismatch (argument 2)

die fehlerhafte Zeile: mysql_escape_string(grund,bangrund,dbhandle);

Kann mir bitte jemand behilflich sein? Komme da echt nicht weiter...

#Apple · 14. April 2017

probiere es mal so.. habe mir das im Wiki angeschaut und bin der Meinung das dies so muss.
mysql_escape_string(bangrund,grund,dbhandle);

Iceberqk · 14. April 2017

Klappt nicht

Kaliber · 14. April 2017

Nutz am einfachsten einfach %q im format.

Man sieht nicht wie du bangrund deklariert hast...eventuell meinst du nur grund?

Zudem müsste das wenn dann so aussehen:

mysql_escape_string(bangrund,grund,sizeof(grund),dbhandle);

//Es würde aber folgendes reichen:
mysql_escape_string(bangrund,grund);

Iceberqk · 14. April 2017

Ich möchte den Grund des Banns in der Datenbank speichern. Dazu habe in MySQL eine Spalte namens bangrund hinzugefügt(natürlich vom Typ varchar). Ich habe dann bangrund auch in mein playerInfo Enum und in meine Funktion saveplayer eingefügt. Ist also eine normale Zeichenkette, welche ich versuche in der Datenbank zu speichern. Leider hat dein Vorschlag das Problem nicht gelöst, trotzdem danke.

Kaliber · 14. April 2017

Also um dir das mal zu erklären:

C

enum test
{
    bangrund[32],
    level
};
new Variable[test];

So du kannst nirgends direkt bangrund verwenden.

Sondern immer nur Variable[bangrund].

Also müsste es bei dir so aussehen:

mysql_escape_string(grund, sInfo[playerid][bangrund]);

Iceberqk · 14. April 2017

Es funktioniert semi. Also das Speichern in der Datenbank klappt. Aber das Problem ist dabei, dass der Grund automatisch in einer neuen Zeile gespeichert wird, das ist ja nicht Sinn der Sache...
Geldcheat einfach als Test
danke schon mal für deine bisherige Hilfe, könntest du mir bitte sagen wie ich das behebe?

Kaliber · 14. April 2017

Du hast beim speichern was durcheinander gebracht und nicht ein UPDATE Query genutzt, sondern ein INSERT INTO was hier nicht richtig ist.

Iceberqk · 14. April 2017

Funktion sieht so aus:

mysql_escape_string(grund, sInfo[playerid][bangrund]);
format(query,sizeof(query),"UPDATE user SET bangrund='%s'",grund);
mysql_function_query(dbhandle,query,false,"","");

mysql Logdatei : [21:44:10] [ERROR] mysql_escape_string - destination size is too small (must be at least as big as source)

Jetzt wird gar kein Grund gespeichert... Sry wenn ich euch langsam nerve aber bin echt noch ein Amateur wenns ums Programmieren geht..

Kaliber · 14. April 2017

Du hast das WHERE vergessen im Query.

bangrund muss die selbe Länge wie grund haben
In dem Fall also [128]

Iceberqk · 14. April 2017

Call me a stupid noob:

mysql_escape_string(grund, sInfo[playerid][bangrund]);
format(query,sizeof(query),"UPDATE user SET bangrund='%s' WHERE id='%i'",grund,sInfo[playerid][id]);
mysql_function_query(dbhandle,query,false,"","");

mysql log : [22:02:24] [ERROR] mysql_escape_string - destination size is too small (must be at least as big as source)

es geht immer noch nicht..

Edgar · 14. April 2017

Mal davon abgesehen das du im Format immer noch die Variable grund verwendest anstatt sInfo[playerid][bangrund],
du musst doch einfach lesen was da Steht. Im Grunde:
Die Variable sInfo[playerid][bangrund] ist zu klein, sie muss mindestens so groß sein wie grund.

Am besten Kalibers Tipp in betracht ziehen.

Iceberqk · 14. April 2017

Sorry aber jetzt peil ich das irgendwie gar nicht mehr...

Jeffry · 15. April 2017

Ich verstehe nicht warum hier so ein Zirkus darum gemacht wird und dann nicht weiter geholfen wird.
Es könnte so einfach sein.

Zitat von Iceberqk

Sorry aber jetzt peil ich das irgendwie gar nicht mehr...

So sollte es aussehen:
Einfach die Variable sich selbst wieder zuweisen (grund in grund schreiben), dann gibt es keine Probleme.
mysql_escape_string(grund, grund);
format(query,sizeof(query),"UPDATE user SET bangrund='%s' WHERE id='%i'",grund,sInfo[playerid][id]);
mysql_function_query(dbhandle,query,false,"","");

Den Grund zusätzlich ins Spieler-Array zu schreiben ist nicht notwendig, da der Spieler im selben Moment den Server verlässt, somit ist das nicht mehr notwendig.

Der ganze Befehl

Iceberqk · 15. April 2017

Hat leider immer noch nicht funktioniert... Der Grund wird nicht in der Datenbank bei "bangrund" gespeichert.

Jeffry · 15. April 2017

Ich sehe grade noch, dass hier die pID hin muss:
format(query,sizeof(query),"UPDATE user SET bangrund='%s' WHERE id='%i'",grund,sInfo[playerid][id]);
Zu
format(query,sizeof(query),"UPDATE user SET bangrund='%s' WHERE id='%i'",grund,sInfo[pID][id]);

Sollte das auch nicht klappen, lass dir das Query per Print ausgeben und poste es dann bitte mal.

Iceberqk · 17. April 2017

[DT]midomen · 17. April 2017

Der sscanf warning sagt dir einfach nur dass du lieber in sscanf bei dem parameter "s" die länge des string angeben sollst also z.b. bei einem string[32]

"s[32]"

Iceberqk · 18. April 2017

Weiß immer noch nicht wie ich weitermachen soll

Weiß nicht wie ich den Fehler beheben soll

Beitrag von Marschl (15. April 2017)

Ähnliche Themen

[SAMMELTHREAD] Kleine Scripting Fragen

Smalltalk

Gesucht wird jemand der mir Pawn beibringt

Fahrzeug hilfe

samp server. exe Systemfehler