MySQL Problem

Ap0llinaris · 8. April 2012

Hallo.

Ich habe mir vorhin das MySQL Tutorial hier im Forum angesehen aber habe damit ein kleines Problem.

Die Daten werden nicht abgespeichert.
Sprich geladen/gespeichert wird nichts.

Ich habe bereits gedebuggt und theoretisch sollte es gehen.

Also bei LoadPlayer und SavePlayer kam nur dass es funktioniert hat.

Ich kann ja mal ein bisschen Code hier lassen:

stock Connect_To_Database()
{
mysql_connect(SQL_HOST, SQL_USER, SQL_DATA, SQL_PASS);
if(mysql_ping() == 1)
{
print("<-| Die Verbindung zur Datenbank wurde hergestellt. |->");
return 1;
}
else
{
print("<-| Es konnte keine Verbindung zur Datenbank hergestellt werden. |->");
print("<-| Versuche erneut ...");
mysql_connect(SQL_HOST, SQL_USER, SQL_DATA, SQL_PASS);
if(mysql_ping() == 1)
{
print("<-| Die Verbindung zur Datenbank wurde hergestellt. |->");
return 1;
}
else
{
print("<-| Die Verbindung konnte nicht hergestellt werden. |->");
print("<-| Server shutdown... |->");
SendRconCommand("exit");
return 1;
}
}
}

stock mysql_CheckAccount(playerid)
{
new Query[128], Name[MAX_PLAYER_NAME], count;
GetPlayerName(playerid, Name, MAX_PLAYER_NAME);
mysql_real_escape_string(Name, Name);
format(Query, sizeof(Query),"SELECT * FROM `accounts` WHERE `Name` = '%s'", Name);
mysql_query(Query);
mysql_store_result();
count = mysql_num_rows();
mysql_free_result();
return count;
}

stock CreateAccount(playerid, pass[])
{
new query[256], Name[MAX_PLAYER_NAME];
GetPlayerName(playerid, Name, MAX_PLAYER_NAME);
mysql_real_escape_string(Name, Name);
mysql_real_escape_string(pass, pass);
format(query, sizeof(query), "INSERT INTO `accounts` (`Name`, `Passwort`) VALUES ('%s', '%s')", Name, pass);
mysql_query(query);
return true;
}

stock mysql_ReturnPasswort(Name[])
{
new query[130], Get[130];
mysql_real_escape_string(Name, Name);
format(query, 128, "SELECT `passwort` FROM `accounts` WHERE `NAME` = '%s'", Name);
mysql_query(query);
mysql_store_result();
mysql_fetch_row(Get);
mysql_free_result();
return Get;
}

stock mysql_GetInt(Table[], Field[], Where[], Is[])
{
new query[128];
mysql_real_escape_string(Table, Table);
mysql_real_escape_string(Field, Field);
mysql_real_escape_string(Where, Where);
mysql_real_escape_string(Is, Is);
format(query, 128, "SELECT `%s` FROM `%s` WHERE `%s` = '%s'", Field, Table, Where, Is);
mysql_query(query);
mysql_store_result();
new sqlint = mysql_fetch_int();
mysql_free_result();
return sqlint;
}

stock mysql_GetString(Table[], Field[], Where[], Is[])
{
new query[128], Get[128];
mysql_real_escape_string(Table, Table);
mysql_real_escape_string(Field, Field);
mysql_real_escape_string(Where, Where);
mysql_real_escape_string(Is, Is);
format(query, 128, "SELECT `%s` FROM `%s` WHERE `%s` = '%s'", Field, Table, Where, Is);
mysql_query(query);
mysql_store_result();
mysql_fetch_row(Get);
mysql_free_result();
return Get;
}

stock Float:mysql_GetFloat(Table[], Field[], Where[], Is[])
{
new query[128], Float:sqlfloat;
mysql_real_escape_string(Table, Table);
mysql_real_escape_string(Field, Field);
mysql_real_escape_string(Where, Where);
mysql_real_escape_string(Is, Is);
format(query, 128, "SELECT `%s` FROM `%s` WHERE `%s` = '%s'", Field, Table, Where, Is);
mysql_query(query);
mysql_store_result();
mysql_fetch_float(sqlfloat);
mysql_free_result();
return sqlfloat;
}

stock mysql_SetInt(Table[], Field[], To, Where[], Where2[])
{
new query[128];
mysql_real_escape_string(Table, Table);
mysql_real_escape_string(Field, Field);
mysql_real_escape_string(Where, Where);
mysql_real_escape_string(Where2, Where2);
format(query, 128, "UPDATE `%s` SET `%s` = '%d' WHERE `%s` = '%s'", Table, Field, To, Where, Where2);
mysql_query(query);
return true;
}
stock mysql_SetString(Table[], Field[], To[], Where[], Where2[])
{
new query[128];
mysql_real_escape_string(Table, Table);
mysql_real_escape_string(Field, Field);
mysql_real_escape_string(To, To);
mysql_real_escape_string(Where, Where);
mysql_real_escape_string(Where2, Where2);
format(query, 128, "UPDATE `%s` SET `%s` = '%s' WHERE `%s` = '%s'", Table, Field, To, Where, Where2);
mysql_query(query);
return true;
}
stock mysql_SetFloat(Table[], Field[], Float:To, Where[], Where2[])
{
new query[128];
mysql_real_escape_string(Table, Table);
mysql_real_escape_string(Field, Field);
mysql_real_escape_string(Where, Where);
mysql_real_escape_string(Where2, Where2);
format(query, 128, "UPDATE `%s` SET `%s` = '%f' WHERE `%s` = '%s'", Table, Field, To, Where, Where2);
mysql_query(query);
return true;
}

stock LoadPlayer(playerid)
{
print("Load Player");
if(IsPlayerConnected(playerid) && !IsPlayerNPC(playerid))
{
print("Load Player#2");
GetPlayerName(playerid, Spieler[playerid][pName], MAX_PLAYER_NAME);
Spieler[playerid][pLevel] = mysql_GetInt("accounts", "Level", "Name", Spieler[playerid][pLevel]);
Spieler[playerid][pAdmin] = mysql_GetInt("accounts", "Admin", "Name", Spieler[playerid][pAdmin]);
Spieler[playerid][pCash] = mysql_GetInt("accounts", "Geld", "Name", Spieler[playerid][pCash]);
Spieler[playerid][pKills] = mysql_GetInt("accounts", "Kills", "Name", Spieler[playerid][pKills]);
Spieler[playerid][pTode] = mysql_GetInt("accounts", "Tode", "Name", Spieler[playerid][pTode]);
Spieler[playerid][pTut] = mysql_GetInt("accounts", "Tutorial", "Name", Spieler[playerid][pTut]);
Spieler[playerid][pSkin] = mysql_GetInt("accounts", "Skin", "Name", Spieler[playerid][pSkin]);
Spieler[playerid][pGeschlecht] = mysql_GetInt("accounts", "Geschlecht", "Name", Spieler[playerid][pGeschlecht]);
Spieler[playerid][pFraktion] = mysql_GetInt("accounts", "Fraktion", "Name", Spieler[playerid][pFraktion]);
}
return 1;
}

stock SavePlayer(playerid)
{
print("Save Player");
if(Spieler[playerid][pLoggedIn]== 1 && !IsPlayerNPC(playerid))
{
print("Save Player #2");
mysql_SetInt("accounts", "Level", Spieler[playerid][pLevel], "Name", Spieler[playerid][pName]);
mysql_SetInt("accounts", "Admin", Spieler[playerid][pAdmin], "Name", Spieler[playerid][pName]);
mysql_SetInt("accounts", "Geld", Spieler[playerid][pCash], "Name", Spieler[playerid][pName]);
mysql_SetInt("accounts", "Kills", Spieler[playerid][pKills], "Name", Spieler[playerid][pName]);
mysql_SetInt("accounts", "Tode", Spieler[playerid][pTode], "Name", Spieler[playerid][pName]);
mysql_SetInt("accounts", "Tutorial", Spieler[playerid][pTut], "Name", Spieler[playerid][pName]);
mysql_SetInt("accounts", "Skin", Spieler[playerid][pSkin], "Name", Spieler[playerid][pName]);
mysql_SetInt("accounts", "Geschlecht", Spieler[playerid][pGeschlecht], "Name", Spieler[playerid][pName]);
mysql_SetInt("accounts", "Fraktion", Spieler[playerid][pFraktion], "Name", Spieler[playerid][pName]);
}
return 1;
}

Wenn ich mir den Log anschaue und einwenig versuche daraus etwas zu entziffern:
[18:15:57] CMySQLHandler::Query(UPDATE `accounts` SET `Skin` = '12' WHERE `Name` = '') - Successfully executed.

Sieht es so aus alsob er keinen "Namen" hat wo er abspeichern soll.

Kann jemand helfen??

tonyalva · 8. April 2012

Spieler[playerid][pKills] = mysql_GetInt("accounts", "Kills", "Name", Spieler[playerid][pKills]);

Du möchtest den Wert abfragen, der unter "Kills" steht.
Jedoch liest du den Wert aus der Spalte aus, in der der Name eines Spielers seinen Kills entspricht (hat keinen Wert, da die Abfrage nicht klappt).
Tausche also die letze Variable innerhalb deiner Query mit der Variable aus, die den Namen des Spielers beeinhaltet.

Viele Grüße,
Alva

NicoAiko · 8. April 2012

Die Variable hat den Namen anscheinend nicht, am besten machst du vor dem SavePlayer bei OnPlayerDisconnect noch

new name[24];
GetPlayerName(playerid,name,24);
format(Spieler[playerid][pName],24,"%s",name);

Ich würde dazu raten, oder such nach dem Problem, dass es nicht geändert wird!

Ap0llinaris · 8. April 2012

Klappt nun wunderbar - Halbwegs.

Wenn ich nun einen gmx ausführe, werden alle Werte (Integers) auf 0 gesetzt.
Passwörter, Account-ID und Name bleibt bestehen.

//Edit

Ich habe bereits bei OnGameModeExit eine Schleife durch alle Spieler gehen lassen, die alle Daten abspeichert, solange der Spieler eingeloggt ist.

NicoAiko · 8. April 2012

mach bei /gmx ein SaveAll rein dann funktionierts auch

wenn du /rcon gmx machst machste einfach das hier

public OnRconCommand(cmd[])
{
if(!strcmp(cmd,"gmx",true))
{
SaveAll();
}
return 0;
}

ungefähr so

Edit: // Quelle: http://wiki.sa-mp.com/wiki/OnRconCommand
Edit2: Callback hatte falschen Namen

Ap0llinaris · 8. April 2012

Zählt OnRconCommand auch für die Console?

NicoAiko · 8. April 2012

Entschuldige ich habe gesucht wurde aber nicht fündig!

Was jedoch feststeht ist, dass du die RCON-Geschichte in einem Filterscript machen musst.

SaveAll kannste dann ja kopieren...

Zu der Sache mit der Console musst du einfach ausprobieren

Ap0llinaris · 8. April 2012

Und wie soll ich dann abfragen ob er connected & eingeloggt ist? (Naja gut IsPlayerConnected...) aber das mit dem eingeloggt.

Normal sollte es doch alles abgespeichert werden oder nicht?
Ich mein wenn ich gmx mache, dann wird auch OnPlayerDisconnect aufgerufen und sogleich SavePlayer.
Aber es wird alles auf 0 gesetzt and Variablen.

So und mir ist etwas aufgefallen:

Ich erstelle einen Account.
Mache einen GMX.
Logge mich wieder ins Spiel ein (nachdem GMX)
Muss mein Passwort eingeben und mache kurz das Tutorial durch(Wenn der Account existiert aber die Tutorial Variable auf 0 ist dann wird man ins Tutorial gebracht)
Nachdem Tutorial nochmal GMX und alles wird gespeichert.
Komisch!?

Aber ich will ja dass es trotzdem klappt. Aufeinmal kommen Fehler auf bla bla, später verlieren viele ihre Statistiken undso. Das will ich halt verhindern.

SSL · 8. April 2012

Zitat von [DgN]ReyDz

Zählt OnRconCommand auch für die Console?

Zitat

This callback is called when a player types things into the RCON console.

NicoAiko · 8. April 2012

Zitat von [DgN]ReyDz

Und wie soll ich dann abfragen ob er connected & eingeloggt ist? (Naja gut IsPlayerConnected...) aber das mit dem eingeloggt.

Mach einfach CallRemoteFunction -> Mache im Gamemode eine Funktion die Abfragt ob der Spieler online + Eingeloggt ist

Zitat von [DgN]ReyDz

Normal sollte es doch alles abgespeichert werden oder nicht?
Ich mein wenn ich gmx mache, dann wird auch OnPlayerDisconnect aufgerufen und sogleich SavePlayer.

Das ist falsch!
/gmx macht folgendes SendRconCommand("gmx"); das löst OnGameModeExit() aus
außerdem werden die Spieler nicht disconnectet den der Client ist ja weiterhin connectet, es gibt es server-seitiges Restart (Stop and Start)

Ap0llinaris · 8. April 2012

Das heißt:
public IsPlayerLoggedIn(playerid)
{
if(Spieler[playerid][pLoggedIn] == 1)
{
return 1;
}
return 0;
}
In meinem Gamemode
und im Filterscript muss ich dann wo CallRemoteFunction einfügen und wie?

Habe noch nie damit gearbeitet.

NicoAiko · 8. April 2012

Eigentlich ist das einfacher...

//GameMode
public SavePlayer(playerid)
{
if(!IsPlayerLoggedIn(playerid) || !IsPlayerConnected(playerid)) return 0;
//rest
return 1;
}
public SaveAll()
{
for(new i = 0; i < MAX_PLAYERS; i++) {
if(!IsPlayerLoggedIn(i) || !IsPlayerConnected(i)) return 0;
//rest
}
return 1;
}
//Filterscript
public OnRconCommand(cmd[])
{
if(!strcmp(cmd,"gmx",true))
{
CallRemoteFunction("SaveAll",""); // Denke das müsste so hin hauen... Arbeite auch nicht damit^^
}
return 0;
}

Edit: SaveAll geadded

Ap0llinaris · 8. April 2012

Hab die Funktion im Filterscript so verwendet:

public OnRconCommand(cmd[])
{
if(strcmp(cmd, "gmx", true) == 0)
{
for(new i=0;i<GetMaxPlayers();i++)
{
CallRemoteFunction("SavePlayer", "%i", i);
}
}
return 1;
}

Funktioniert trotzdem nicht.
Nur wenn ich mich das zweite mal einlogge und dann einen GMX mache. Diesen GMX mache ich halt jetzt nur, damit ich sicher gehen kann falls der Server ausfällt dass nix passiert.

NicoAiko · 8. April 2012

OnRconCommand muss 0 returnen

Außerdem nicht %i sondern nur i (-> ist wie bei sscanf)

Ap0llinaris · 8. April 2012

Nope, klappt immernoch nicht.

d30af26d · 8. April 2012

Den Namen zu escapen ist sinnlos, da dieser garnichts anrichten kann, da dieser nichtmal " ) oder ; beinhalten kann.

NicoAiko · 8. April 2012

was gibts du ein? /rcon gmx?

oder nur /gmx?

Ap0llinaris · 8. April 2012

Ich gebe in die Console gmx ein.

NicoAiko · 8. April 2012

printe es mal und guck mal was raus kommt vllt klappt es gar nicht und es funkt nur ingame...

Ap0llinaris · 8. April 2012

Nein.
Wie gesagt es funktioniert nur wenn ich mich registriere, nen GMX mache, einlogge und nochmal nen GMX mache.