MySQL speichert nicht außer Passwort!

Zreeq · 22. August 2017

Hallo,

wie oben bereits steht speichert MySQL nur mein Name und Passwort nach Register.

Wenn ich mir z.B. Geld setze speichert er das nicht. Wenn ich mich töten lasse ändert sich in der Datenbank auch nichts. Dazu muss man aber sagen das er die IP speichert und lädt.

[23:11:25] [DEBUG] mysql_format - connection: 1, len: 256, format: "SELECT id FROM accounts WHERE name = '%e'"[23:11:25] [DEBUG] mysql_pquery - connection: 1, query: "SELECT id FROM accounts WHERE name = 'TaigX'", callback: "OnUserCheck2", format: "d"[23:11:25] [DEBUG] CMySQLQuery::Execute[OnUserCheck2] - starting query execution[23:11:25] [DEBUG] CMySQLQuery::Execute[OnUserCheck2] - query was successfully executed within 0.431 milliseconds[23:11:25] [DEBUG] CMySQLResult::CMySQLResult() - constructor called[23:11:25] [DEBUG] Calling callback "OnUserCheck2"..[23:11:25] [DEBUG] cache_get_data - connection: 1[23:11:25] [DEBUG] mysql_format - connection: 1, len: 256, format: "SELECT * FROM accounts WHERE name = '%e' AND password = MD5('%e')"[23:11:25] [DEBUG] mysql_pquery - connection: 1, query: "SELECT * FROM accounts WHERE name = 'TaigX' AND password = MD5('", callback: "OnUserLogin", format: "d"[23:11:25] [DEBUG] CMySQLResult::~CMySQLResult() - deconstructor called[23:11:25] [DEBUG] CMySQLQuery::Execute[OnUserLogin] - starting query execution[23:11:25] [DEBUG] CMySQLQuery::Execute[OnUserLogin] - query was successfully executed within 0.336 milliseconds[23:11:25] [DEBUG] CMySQLResult::CMySQLResult() - constructor called[23:11:25] [DEBUG] Calling callback "OnUserLogin"..[23:11:25] [DEBUG] cache_get_row_count - connection: 1[23:11:25] [DEBUG] cache_get_field_content_int - row: 0, field_name: "id", connection: 1[23:11:25] [DEBUG] CMySQLResult::GetRowDataByName - row: '0', field: "id", data: "2"[23:11:25] [DEBUG] cache_get_field_content_int - row: 0, field_name: "level", connection: 1[23:11:25] [DEBUG] CMySQLResult::GetRowDataByName - row: '0', field: "level", data: "0"[23:11:25] [DEBUG] cache_get_field_content_int - row: 0, field_name: "money", connection: 1[23:11:25] [DEBUG] CMySQLResult::GetRowDataByName - row: '0', field: "money", data: "0"[23:11:25] [DEBUG] cache_get_field_content_int - row: 0, field_name: "kills", connection: 1[23:11:25] [DEBUG] CMySQLResult::GetRowDataByName - row: '0', field: "kills", data: "0"[23:11:25] [DEBUG] cache_get_field_content_int - row: 0, field_name: "deaths", connection: 1[23:11:25] [DEBUG] CMySQLResult::GetRowDataByName - row: '0', field: "deaths", data: "0"[23:11:25] [DEBUG] cache_get_field_content_int - row: 0, field_name: "admin", connection: 1[23:11:25] [DEBUG] CMySQLResult::GetRowDataByName - row: '0', field: "admin", data: "5"[23:11:25] [DEBUG] cache_get_field_content_int - row: 0, field_name: "ip", connection: 1[23:11:25] [DEBUG] CMySQLResult::GetRowDataByName - row: '0', field: "ip", data: "127.0.0.1"[23:11:25] [DEBUG] CMySQLResult::~CMySQLResult() - deconstructor called[23:12:02] [DEBUG] mysql_format - connection: 1, len: 256, format: "UPDATE accounts SET level = '%d', money = '%d', kills = '%d', deaths = '%d', admin = '%d', ip = '%d' WHERE id = '%d'"[23:12:02] [DEBUG] mysql_pquery - connection: 1, query: "UPDATE accounts SET level = '0', money = '0', kills = '0', death", callback: "(null)", format: "(null)"[23:12:02] [DEBUG] CMySQLQuery::Execute[] - starting query execution[23:12:02] [DEBUG] CMySQLQuery::Execute[] - query was successfully executed within 0.310 milliseconds[23:12:02] [DEBUG] CMySQLQuery::Execute[] - no callback specified, skipping result saving

stock SavePlayer(playerid){ new query[256]; mysql_format(db_handle, query, sizeof(query), "UPDATE accounts SET level = '%d', money = '%d', kills = '%d', deaths = '%d', admin = '%d', ip = '%d' WHERE id = '%d'", PlayerInfo[playerid][plevel], PlayerInfo[playerid][pmoney], PlayerInfo[playerid][pkills], PlayerInfo[playerid][pdeaths], PlayerInfo[playerid][pid], PlayerInfo[playerid][padmin], PlayerInfo[playerid][pip]); mysql_pquery(db_handle, query); return 1;}

Und unter OnPlayerDisconnect hab ich halt SavePlayer(playerid);

Datenbank im Anhang.

Bitte um Hilfe

MfG
Zreeq

Whice · 23. August 2017

Der Fehler liegt an deiner MySQL-Abfrage, die du zum Speichern nutzt. Die Variable pid muss an letzter Stelle der Parameter sein, da diese ja auch als letztes in dem MySQL-Befehl gebraucht wird.

Code

stock SavePlayer(playerid)
{ 
	new query[256]; 
	mysql_format(db_handle, query, sizeof(query), "UPDATE accounts SET level = '%d', money = '%d', kills = '%d', deaths = '%d', admin = '%d', ip = '%d' WHERE id = '%d'", 
	PlayerInfo[playerid][plevel], PlayerInfo[playerid][pmoney], PlayerInfo[playerid][pkills], PlayerInfo[playerid][pdeaths], PlayerInfo[playerid][padmin], PlayerInfo[playerid][pip], PlayerInfo[playerid][pid]); 
	mysql_pquery(db_handle, query); 
	return 1;
}

Zreeq · 23. August 2017

Danke.

Deaths werden nun gespeichert und geladen. Jetzt hab ich noch das problem das money geladen wird aber nicht gespeichert.

KinGRaPTiiC · 23. August 2017

@Whice. Ist bei mir das selbe kann in der Datenbank mein Geld setten wird Ingame geladen aber wenn ich /setmoney 1 0 setzte und vom Server Disconnecte was ich eigentlich nichteinmal machen müsste da ich beim setmoney Befehl noch den Spieler Speicher. Aber in der Datenbank steht das ich trotzdem noch 50k habe und @Zreeq Sorry das ich dir jetzt nicht helfen kann

Whice · 23. August 2017

Zitat von Zreeq

Danke.

Deaths werden nun gespeichert und geladen. Jetzt hab ich noch das problem das money geladen wird aber nicht gespeichert.

Zitat von KinGRaPTiiC

@Whice. Ist bei mir das selbe kann in der Datenbank mein Geld setten wird Ingame geladen aber wenn ich /setmoney 1 0 setzte und vom Server Disconnecte was ich eigentlich nichteinmal machen müsste da ich beim setmoney Befehl noch den Spieler Speicher. Aber in der Datenbank steht das ich trotzdem noch 50k habe und @Zreeq Sorry das ich dir jetzt nicht helfen kann

Ich denke mal das dies daran liegen wird, dass ihr bei deinem GiveMoney Befehl das ganze nur über GivePlayerMoney macht, dort ist es allerdings wichtig, dass die Geldvariable "PlayerInfo[playerid][pmoney]" auch hochgesetzt wird.

KinGRaPTiiC · 23. August 2017

Denke das du recht hast das hier ist mein jetziger Befehl:

ocmd:setmoney(playerid,params[])
{
print("setmoney Befehl ausgeführt!");
[b]if[/b](!isAdmin(playerid,3))[b]return[/b] SendClientMessage(playerid,DUNKELROT,"Dein Adminlevel ist zu Niedrig!");
[b]new[/b] pID, test[128], string[128],BETRAG;
[b]if[/b](sscanf(params,"ui",pID,BETRAG))[b]return[/b] SendClientMessage(playerid,GELB,"[INFO]: /setmoney [SpielerID] [Betrag]");
ResetPlayerMoney(pID);
GivePlayerMoney(pID,BETRAG);
format(string,sizeof(string),"AdmCmd: %s hat dein Geld auf $%i gesetzt",PlayerInfo[playerid],BETRAG);
format(test,sizeof(test),"AdmCmd: Du hast %s den Geldbetrag auf $%i gesetzt!",PlayerInfo[pID],BETRAG);
SendClientMessage(pID,DUNKELROT,string);
SendClientMessage(playerid,DUNKELROT,test);
SaveUserStats(playerid);
[b]return[/b] 1;
}

Muss ich das zu dem ändern?:
ocmd:setmoney(playerid,params[])
{
print("setmoney Befehl ausgeführt!");
[b]if[/b](!isAdmin(playerid,3))[b]return[/b] SendClientMessage(playerid,DUNKELROT,"Dein Adminlevel ist zu Niedrig!");
[b]new[/b] pID, test[128], string[128],BETRAG;
[b]if[/b](sscanf(params,"ui",pID,BETRAG))[b]return[/b] SendClientMessage(playerid,GELB,"[INFO]: /setmoney [SpielerID] [Betrag]");
ResetPlayerMoney(pID);
GivePlayerMoney(pID,BETRAG);
format(string,sizeof(string),"AdmCmd: %s hat dein Geld auf $%i gesetzt",PlayerInfo[playerid],BETRAG);
format(test,sizeof(test),"AdmCmd: Du hast %s den Geldbetrag auf $%i gesetzt!",PlayerInfo[pID],BETRAG);
SendClientMessage(pID,DUNKELROT,string);
SendClientMessage(playerid,DUNKELROT,test);
SaveUserStats(playerid);
[b]return[/b] 1;
}

Wäre das so richtig? @Whice.

Whice · 23. August 2017

Zitat von KinGRaPTiiC

Wäre das so richtig? @Whice.

Code

ocmd:setmoney(playerid, params[])
{
	print("setmoney Befehl ausgeführt!");
	if(!isAdmin(playerid,3))return SendClientMessage(playerid, DUNKELROT, "Dein Adminlevel ist zu Niedrig!");
	new pID, string[128], BETRAG;
	if(sscanf(params, "ui", pID, BETRAG))return SendClientMessage(playerid, GELB, "[INFO]: /setmoney [SpielerID] [Betrag]");
	ResetPlayerMoney(pID);
	GivePlayerMoney(pID, BETRAG);
	PlayerInfo[pID][pmoney]+=BETRAG;
	format(string, sizeof(string), "AdmCmd: %s hat dein Geld auf $%i gesetzt", SpielerName(playerid), PlayerInfo[pID][pmoney]);
	SendClientMessage(pID, DUNKELROT, string);
	format(string, sizeof(string), "AdmCmd: Du hast %s den Geldbetrag auf  $%i gesetzt!", SpielerName(pID), PlayerInfo[pID][pmoney]);
	SendClientMessage(playerid, DUNKELROT, string);
	SaveUserStats(pID);
	return 1;
}

Alles anzeigen

KinGRaPTiiC · 23. August 2017

@Whice. Erstmal nochmal danke dafür aber wenn ich mein Geld auf 0 setzte steht das da: http://prntscr.com/gc585z

und mein Geldbetrag wird zwar auf 0 gesetzt aber in der Datenbank habe ich trotzdem noch 50010

/edit
Nachricht ist wieder ganz Normal also steht jetzt z.B 0 da... aber Datenbank wurde nicht verändert

Whice · 23. August 2017

Grade gesehen, dass du nicht der Ersteller des Themas bist und der Code oben somit nicht deine Variable ist.

Du müsstest also nun "PlayerInfo[pID][pmoney]" durch "PlayerInfo[pID][pMoney]" ersetzten

KinGRaPTiiC · 23. August 2017

@Whice. Ich weiß hab ich auch geändert aber mein Code sieht schon so aus aber funkt net:
ocmd:setmoney(playerid, params[])
{
print("setmoney Befehl ausgeführt!");
if(!isAdmin(playerid,3))return SendClientMessage(playerid, DUNKELROT, "Dein Adminlevel ist zu Niedrig!");
new pID, string[128], BETRAG;
if(sscanf(params, "ui", pID, BETRAG))return SendClientMessage(playerid, GELB, "[INFO]: /setmoney [SpielerID] [Betrag]");
ResetPlayerMoney(pID);
GivePlayerMoney(pID, BETRAG);
PlayerInfo[pID][pMoney]+=BETRAG;
format(string, sizeof(string), "AdmCmd: %s hat dein Geld auf $%i gesetzt", SpielerName(playerid), BETRAG);
SendClientMessage(pID, DUNKELROT, string);
format(string, sizeof(string), "AdmCmd: Du hast %s den Geldbetrag auf $%i gesetzt!", SpielerName(pID), BETRAG);
SendClientMessage(playerid, DUNKELROT, string);
SaveUserStats(pID);
return 1;
}

maddin · 23. August 2017

Code

PlayerInfo[pID][pMoney]+=BETRAG;

bedeutet du addierst BETRAG zu PlayerInfo[pID][pMoney] hinzu. Was du willst ist aber anscheinend den Betrag überschreiben, also

Code

PlayerInfo[pID][pMoney] = BETRAG;

Whice · 23. August 2017

Code

ocmd:setmoney(playerid, params[])
{
	print("setmoney Befehl ausgeführt!");
	if(!isAdmin(playerid,3))return SendClientMessage(playerid, DUNKELROT, "Dein Adminlevel ist zu Niedrig!");
	new pID, string[128], BETRAG;
	if(sscanf(params, "ui", pID, BETRAG))return SendClientMessage(playerid, GELB, "[INFO]: /setmoney [SpielerID] [Betrag]");
	ResetPlayerMoney(pID);
	GivePlayerMoney(pID, BETRAG);
	PlayerInfo[pID][pMoney] = BETRAG;
	format(string, sizeof(string), "AdmCmd: %s hat dein Geld auf $%i gesetzt", SpielerName(playerid), BETRAG);
	SendClientMessage(pID, DUNKELROT, string);
	format(string, sizeof(string), "AdmCmd: Du hast %s den Geldbetrag auf  $%i gesetzt!", SpielerName(pID), BETRAG);
	SendClientMessage(playerid, DUNKELROT, string);
	SaveUserStats(pID);
	return 1;
}

Alles anzeigen

Zreeq · 23. August 2017

Also ich hab denke mal ein anderes Problem, da ich das wie oben bereits hatte und es trd. nicht klappt.

So sieht z.B. mein setmoney aus:

ocmd:setmoney(playerid, params[])
{
new pID, money, string[128];
if(PlayerInfo[playerid][plogged] == false) return SendClientMessage(playerid, COLOR_LIGHTGREY, "[Warnung]: Du musst dich erst einloggen, um den Befehl zu benutzen!");
if(PlayerInfo[playerid][padmin] < 3) return SendClientMessage(playerid, COLOR_LIGHTGREY, "[Warnung]: Du bist nicht berechtigt diesen Befehl zu benutzen!");
if(PlayerInfo[playerid][pduty] == 0) return SendClientMessage(playerid, COLOR_LIGHTGREY, "[Warnung]: Du befindest dich nicht im Administrativen Dienst!");
if(sscanf(params, "ui", pID, money)) return SendClientMessage(playerid, COLOR_LIGHTGREY, "[Verwendung]: [/setmoney] [ID] [GELD]");
ResetPlayerMoney(pID);
GivePlayerMoney(pID, money);
PlayerInfo[pID][pmoney] = money;
format(string, sizeof(string), "{0099FF}[Admin]:{FFFFFF} %s %s hat dir %i$ gesettet.", GetAdminRank(playerid), PlayerInfo[playerid][pname], money);
SendClientMessage(pID, COLOR_WHITE, string);
format(string, sizeof(string), "{0099FF}[Admin]:{FFFFFF} Du hast dem Spieler %s %i$ gesettet.", PlayerInfo[pID][pname], money);
SendClientMessage(playerid, COLOR_WHITE, string);
return 1;
}

Gespeichert wird der Spieler wenn er den Server verlässt oder der Server beendet wird.

KinGRaPTiiC · 23. August 2017

@Zreeq versuchs vielleicht mal so:

ocmd:setmoney(playerid, params[])
{
new pID, BETRAG, string[128];
if(PlayerInfo[playerid][plogged] == false) return SendClientMessage(playerid, COLOR_LIGHTGREY, "[Warnung]: Du musst dich erst einloggen, um den Befehl zu benutzen!");
if(PlayerInfo[playerid][padmin] < 3) return SendClientMessage(playerid, COLOR_LIGHTGREY, "[Warnung]: Du bist nicht berechtigt diesen Befehl zu benutzen!");
if(PlayerInfo[playerid][pduty] == 0) return SendClientMessage(playerid, COLOR_LIGHTGREY, "[Warnung]: Du befindest dich nicht im Administrativen Dienst!");
if(sscanf(params, "ui", pID, money)) return SendClientMessage(playerid, COLOR_LIGHTGREY, "[Verwendung]: [/setmoney] [ID] [GELD]");
ResetPlayerMoney(pID);
GivePlayerMoney(pID, BETRAG);
PlayerInfo[pID][pmoney] = BETRAG;
format(string, sizeof(string), "{0099FF}[Admin]:{FFFFFF} %s %s hat dir %i$ gesettet.", GetAdminRank(playerid), PlayerInfo[playerid][pname], BETRAG);
SendClientMessage(pID, COLOR_WHITE, string);
format(string, sizeof(string), "{0099FF}[Admin]:{FFFFFF} Du hast dem Spieler %s %i$ gesettet.", PlayerInfo[pID][pname], BETRAG);
SendClientMessage(playerid, COLOR_WHITE, string);
return 1;
}

Zreeq · 23. August 2017

Hat sich erledigt!

Der Fehler lag beim speichern von money.

Danke an alle die mir geholfen haben

MySQL speichert nicht außer Passwort!

breadfish.de 12. März 2022

Ähnliche Themen

[jTuT] MySQL R41-4 (Installation, XAMPP, Verwendung, Bedienung & Registrations-System)