probleme beim geld speichern

MindGrind · 29. März 2014

moin leute,

ja ich hab da son kleines problem mit meinem script unzwar wird das geld von spielern nicht richtig geladen.

ich hab die spieler daten auf na MySQL datenbank und mein sever verbindet sich auch vernünftig damit und auch das laden und auslesen der daten funktioniert richtig also wenn ich das geld manuell in der datenbank ändere wird es beim einloggen problemlos geladen aber beim ausloggen wirds nicht richtig gespeichert.

hier wird eigentlich der spieler gespeichert

Code

savePlayer(playerid)
{
	if(sInfo[playerid][eingeloggt]==0)return 1;
	new query[128];
	format(query,sizeof(query),"UPDATE user SET level='%i',money='%i',alevel='%i',fraktion='%i,rang='%i' WHERE id='%i'",
	sInfo[playerid][level],GetPlayerMoney(playerid),sInfo[playerid][alevel],sInfo[playerid][fraktion],sInfo[playerid][rang],sInfo,sInfo[playerid][db_id]);
	mysql_function_query(dbhandle,query,false,"","");
	return 1;
}

die function wird bei OnPlayerDisconnect aufgerufen.

wie gesagt das auslesen geht aber das speichern garnicht

hat jemand ne idee wie das klappen könnte oder so oder woran es liegt?

ich bin ein kompletter neueinsteiger was das scripten angeht und weiß desshalb nichtmehr weiter.

#include · 29. März 2014

" GetPlayerMoney(playerid),sInfo[playerid][alevel],sInfo[playerid][fraktion],sInfo[playerid][rang],sInfo,sInfo[playerid][db_id]);"
Du veruschst also das Geld IN dem Level, Fraktion, dem Rang und die ID zu speichern?
Da stimmt glaub ich was nicht.

MindGrind · 29. März 2014

ich hab den beitrag oben nochmal bearbeitet das eine ist bisschen in der zeile verrutscht

ich habs halt genauso gemacht wie in nem tutorial und da hats gefunzt unzwar genauso warum geht das bei mir nicht

SoR'Mirco · 29. März 2014

3
4
5
6
7
8
9

savePlayer(playerid)
{
if(sInfo[playerid][eingeloggt]==0)return 1;
new query[128];
format(query,sizeof(query),"UPDATE user SET level='%i',money='%i',alevel='%i',fraktion='%i,rang='%i' WHERE id='%i'",
sInfo[playerid][level],GetPlayerMoney(playerid),sInfo[playerid][alevel],sInfo[playerid][fraktion],sInfo[playerid][rang],sInfo,sInfo[playerid][db_id]);
mysql_function_query(dbhandle,query,false,"","");
return 1;
}

"GetPlayerMoney(playerid)" - An sich eine ganz tolle sache aber um Geld "hacks" zu vermeiden solltest du eine neue Variable auf machen.
MySQL - hast du geschaut das der Datenbank connect stimmt?
- Lass dir den Query mal ausprinten und führ ihn mal aus...
- "Where id='%i'" ganz nett aber warum nimmst du nicht den Namen (die unterscheidung zwischen namen und ids macht in MySQL millisekunden)
- Generell mach money zu einem Double ein int geht nur bist 32000 noch was ...

MfG

Mirco

Zeuto · 29. März 2014

Zitat von MindGrind

if(sInfo[playerid][eingeloggt]==0)return 1;

Vielleicht liegt ja das Problem in dieser Abfrage, kann sein, dass diese Variable nicht richtig gesetzt wird.

Zitat von MindGrind

new query[128];
format(query,sizeof(query),"UPDATE user SET level='%i',money='%i',alevel='%i',fraktion='%i,rang='%i' WHERE id='%i'",
sInfo[playerid][level],GetPlayerMoney(playerid),sInfo[playerid][alevel],sInfo[playerid][fraktion],sInfo[playerid][rang],sInfo,sInfo[playerid][db_id]);

Versuch mal nur das Geld abzuspeichern und nicht die anderen Variablen. Vielleicht ist das Query einfach nur zu groß

Ansonsten schau mal, was die mysql_log.txt sagt. Da stehen meistens auch hilfreiche Informationen

MindGrind · 29. März 2014

oha da seh ich grad paar sachen in der mysql_log

Code

[02:14:01] [WARNING] CMySQLResult::GetRowDataByName - field not found ("id")
[02:25:44] [ERROR] CMySQLQuery::Execute[()] - (error #1064) You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '0' WHERE id='2000'' at line 1

die sind da mehrfach drinne ich weiß nur nicht genau was mir das sagen soll

Zeuto · 29. März 2014

Gibt es überhaupt die "id"-Spalte? - Blöde Frage, aber fragen schadet nicht.

MindGrind · 29. März 2014

ja die spalte gibt es tatsächlich gibt alle nötien spalten und ID kommt sogar als erste noch vor dem usernamen

Zeuto · 29. März 2014

Hast du mal versucht, aus dem langen Query zwei kürzere Querys zu machen? Wie gesagt, kann sein dass der Query einfach zu lang ist..

//edit: versuch das mal so: (habe nur leerzeichen nach den Kommas hinzugefügt

format(query,sizeof(query),"UPDATE user SET level='%i', money='%i', alevel='%i', fraktion='%i, rang='%i' WHERE id='%i'",
sInfo[playerid][level],GetPlayerMoney(playerid),sInfo[playerid][alevel],sInfo[playerid][fraktion],sInfo[playerid][rang],sInfo,sInfo[playerid][db_id]);

**XeroX** · 29. März 2014

Zitat von SoR'Mirco

int geht nur bist 32000 noch was ...

Woher hast du den müll den her ? ein 16 Bit int ja aber kein 32 Bit int wie es heutzutage üblich ist und cell (welches pawn nutzt) ist einfach ein typedef für ein 32 bit int und somit ist −2.147.483.648 der minimalste wert und 2.147.483.647 der größte siehe hier

SoR'Mirco · 29. März 2014

Zitat von XeroX

Woher hast du den müll den her ? ein 16 Bit int ja aber kein 32 Bit int wie es heutzutage üblich ist und cell (welches pawn nutzt) ist einfach ein typedef für ein 32 bit int und somit ist −2.147.483.648 der minimalste wert und 2.147.483.647 der größte siehe hier

Oh ok sry mein Fehler ...

Jedoch ist 2 Milliarden immer noch zu wenig je nach dem ....

MindGrind · 29. März 2014

also ich habs mal in zwei querys gemacht etwa so:

Code

savePlayer(playerid)
{
	if(sInfo[playerid][eingeloggt]==0)return 1;
	new query[128],query2[128];
	new name[MAX_PLAYER_NAME];
	GetPlayerName(playerid,name,sizeof(name));
	format(query,sizeof(query),"UPDATE user SET level='%i', money='%i' WHERE id='%i'",
	sInfo[playerid][level],GetPlayerMoney(playerid),sInfo[playerid][db_id]);
	format(query2,sizeof(query2),"UPDATE user SET alevel='%i', fraktion='%i, rang='%i' WHERE id='%i'",
	sInfo[playerid][alevel],sInfo[playerid][fraktion],sInfo[playerid][rang],sInfo[playerid][db_id]);
	mysql_function_query(dbhandle,query,false,"","");
	return 1;
}

Alles anzeigen

aber es funzt immernoch nicht

Zeuto · 29. März 2014

Schreib mal auch immer, was die mysql_log.txt Datei sagt, weil ich im Quellcode keinen Fehler sehe..

MindGrind · 29. März 2014

ich habs jetzt hingekrigt mit dem geld der speichert das jetzt ich hab halt geändert das er nicht nach der ID in der tabelle sucht sondern nach dem namen... das funzt jetzt gut aber habe im mysql-log jez diese meldungen.

Code

[19:01:12] [WARNING] CMySQLResult::GetRowDataByName - field not found ("id")

Zeuto · 29. März 2014

Ja anscheinend gibt es die Spalte "id" nicht.. Mehr verstehe ich darunter nicht, vielleicht schaust du die ganze zeit in der falschen Tabelle nach oder so?

//edit: Schick mal einen Screen von deiner Tabelle, dann kann man das besser beurteilen, wobei jetzt ist es egal, da es ja jetzt funktioniert

MindGrind · 29. März 2014

http://s1.directupload.net/images/140329/lk5shm6u.png

hoffe das funzt also so sieht meine tabelle aus

Zeuto · 29. März 2014

Liegt das vermutlich an der Groß-/Kleinschreibung? Versuch mal anstatt "id" "ID" zu schreiben

M_McGarrett · 29. März 2014

format(query,sizeof(query),"UPDATE `user` SET `level`='%i', `money`='%i', `alevel`='%i', `fraktion`='%i, `rang`='%i' WHERE `id`='%i'",
sInfo[playerid][level],GetPlayerMoney(playerid),sInfo[playerid][alevel],sInfo[playerid][fraktion],sInfo[playerid][rang],sInfo,sInfo[playerid][db_id]);

SO?

Zeuto · 29. März 2014

@Danno: das Problem ist schon geklärt^^

breadfish.de 12. März 2022