Beiträge von Blowfish

@Blax's Wir nutzen Apache + PHP & jQuery in Komination mit Node.js, kein CMS oder ähnliches, alles komplett selbst entwickelt.

Woher hast du denn überhaupt diese Funktionen? Im standard release sind die soweit ich weiß nicht vorhanden.

edit: Kommando zurück! Die müssen neu sein, kannte ich noch gar nicht...

char *RowData = NULL;
Result->GetRowData(params[1], params[2], &RowData);
if(RowData == NULL sscanf(RowData, "%d", &ReturnVal) != 1) {
CLog::Get()->LogFunction(LOG_ERROR, false, "cache_get_row_int", "invalid data type");
ReturnVal = 0;
}



Um genau zu sein kommen die Fehler von diesem Code
Es wird wohl daran liegen, dass du irgendwo cache_get_row_int ausführst obwohl für die angegebene Zeile / Spalte gar kein Ergebnis existiert.

Wunderbar

if(mysql_DoubleRowExist(playerid, "db_vehicles", "pName", "vNumber", escapedName, fIDStr))
{

}


Diese if-Abfrage brauchst du nicht. In der UPDATE Abfrage hast du ein WHERE Statement:
WHERE `pName` = '%s' AND `vNumber` = '%i'

Wenn deine Datenbank keine Zeile finden kann, die die Bedingung nach dem WHERE erfüllt dann passiert nichts. Da kannst du dich da ganz auf deinen MySQL Server verlassen und dein Code wird dadurch sogar kürzer.

Du kannst aus Callbacks nichts an den Code, in dem du die Abfrage gestartet hast zurückgeben.

Das ganze funktioniert ähnlich wie mit Dialogen, nur kommen die Antworten hier meist etwas schneller
Du erstellst an einer Stelle im Code einen Dialog und an einer anderen Stelle im Code (OnDialogResponse) kommt dann das Resultat zurück.

Mit Dialogen kannst du ja auch nicht so eine Funktion erstellen:

stock IsPlayerCool(playerid) {
return ShowPlayerDialog(playerid, 123412, DIALOG_STYLE_MSGBOX, "", "Bist du cool?", "Ja", "Nein");
}

Das funktioniert prinzipiell nicht, weil Funktion IsPlayerCool in dem Moment beendet wird in dem der Dialog an den Spieler geschickt wird. Die Antwort kommt dann in OnDialogResponse zurück.
Ganz genau so ist es auch mit dem MySQL Plugin. An einer Stelle im Code startest du die Abfrage und an einer anderen, im erstellten Callback, kommt dann die Antwort wenn die Abfrage beendet ist.
Da muss man zu Anfang ein wenig umdenken.

Das musst du ja nicht

Du willst eine Zeile in der Tabelle Updaten, die folgende Bedingungen erfüllt:
WHERE `pName` = '%s' AND `vNumber` = '%i'
Wenn es keine Zeile in der Tabelle gibt, die diese Bedingung erfüllt dann wird auch nichts geupdated. Die Überprüfung ist daher nicht nötig, das macht die Datenbank schon von ganz alleine.




edit Ob du in den Callbacks etwas per return zurückgibst oder nicht ist egal. Es wird ignoriert.

Du lässt die Überprüfung einfach weg. Wenn die Zeile nicht existiert kann sie auch nicht geupdated werden, und das wird sie auch nicht.

Wenn du möchtest, dass die Zeile angelegt wird wenn sie noch nicht existiert gibt es dafür ein nützlichen Konstrukt von MySQL selbst:
http://www.yourhelpcenter.de/2…ert-record-sql-statement/

Der Grund warum ich sage, dass das nicht funktionieren kann ist der gleiche wie bei deinem mysql_GetPlayerPasswort vorhin. Das Callback ist vollkommen unabhängig von der Funktion, in der du es erzeugst. Wenn das Callback ausgeführt wird ist die Funktion, in der die Abfrage gestartet wurde schon lange beendet. Das ist gerade das gute am neuen System. Es wird nichts mehr blockiert. Der Gamemode kann problemlos weiterlaufen während die Abfrage läuft.

if(mysql_DoubleRowExist(playerid, "db_vehicles", "pName", "vNumber", escapedName, fIDStr))

Das sollte schonmal nicht mehr funktionieren schätze ich.

Ausserdem ist da bei deinem
mysql_function_query(MySQLHandle, query, false, "OnGetReturn", "", "");
etwas faul. Nach "OnGetReturn" lässt du den String leer, in dem du sagst welche Parameter übergeben werden sollen, übergibst dann aber noch einen leeren String. Lass den Letzten Parameter einfach mal weg.

Und noch ein kleiner Tipp zu mysql_function_query. Wenn du bei deiner Query keine Daten erwartest kannst du das Callback auch einfach weglassen, sprich den String dafür einfach leer lassen. Dann wird auch kein Callback ausgeführt.
Die Zeile würde also wie folgt lauten:

mysql_function_query(MySQLHandle, query, false, "", "");


Das Callback OnGetReturn kannst du dann entfernen.

Nur das was du hinzugefügt oder geändert hast nachdem du es das letzte mal kompilieren konntest, wenn du noch weißt was das alles ist

Poste bitte auch mal deine cache_get_row_int und cache_get_row_float Funktionen.

Welcher Fehler nun genau? Der mit undefined symbol "pVehicle", oder das Aufhängen des Compilers?

Bei cache_get_row kannst du es wegnehmen, da es dort ein optionaler Parameter ist. Das musst du natürlich nicht tun, macht den Code nur etwas übersichtlicher. Komm aber nicht auf die Idee es bei mysql_function_query zu entfernen Da ist der Parameter nicht optional.

cache_get_row sieht nicht vor, dass man eine maximale Größe des Strings übergibt. Es wird einfach davon ausgegangen, dass dein String groß genug ist.
Die maximale Länge setzt du ja in der Tabellenstruktur fest, daher kannst du den String auch hinreichend groß machen.

Hier die Signatur von cache_get_row.

cache_get_row( row, idx, dest[] [, connectionHandle = 1] )

Der fünfte Parameter in Zeile 12 und 15 gehört dort nicht hin.
Das Connection Handle ist bei cache_get_row übrigends auch optional. Das musst du dort nicht überall hinschreiben wenn du sowieso nur eine Datenbankverbindung hast.

Vielleicht ist es diese Zeile:

mysql_real_escape_string(PlayerName(playerid), PlayerName(playerid), 1, 18);


Änder das mal in:

new escapedName[MAX_PLAYER_NAME];
mysql_real_escape_string(PlayerName(playerid), escapedName);

Und nicht vergessen dann auch in der Abfrage escapedName statt PlayerName(playerid) zu nutzen

Da bist du auf einen der Nachteile des Modells gestoßen. Wie du weißt wird die Abfrage in einem separaten Thread ausgeführt um den Gamemode nicht zu blockieren. Das Callback wird erst aufgerufen wenn die Abfrage beendet ist. Der Funktionsaufruf von mysql_GetPlayerPassword ist zu diesem Zeitpunkt jedoch bereits beendet.

Du hast also keine Möglichkeit einfach per return im Callback den Wert an mysql_GetPlayerPasswort zu übergeben.
Sprich du musst das Passwort bereits zu einem frühreren Zeitpunkt in einer Variable gespeichert haben, damit du eine derartige Funktion umsetzen kannst.
Die andere Möglichkeit ist, dass du die nötige Überprüfung im Callback ausführst.
Ich würde hier ersteres bevorzugen, da du ja sowieso die Userdaten aus der Datenbank ausliest.
Das Plugin zwingt dich gewissermassen unnötige Abfragen zu vermeiden.

Hallo nochmal

Zu der Zeile Hier:

mysql_real_escape_string(PlayerName(playerid), PlayerName(playerid));

Das funktioniert so leider nicht. Den escapten Namen musst du anschliessend in einem neuen String speichern.
Also beispielsweise:

new escapedName[MAX_PLAYER_NAME];
mysql_real_escape_string(PlayerName(playerid), escapedName);



Die Überprüfung des Passworts kannst du auch gleich im Callback machen. Die Abfrage lautet ja:

`pName` = '%s' AND `Password` = '%s'


Sie liefert also nur ein Ergebnis wenn ein Benutzer unter dem Namen gefunden wurde und auch das richtige Passwort angegeben wurde.
Im Callback kannst du daher sehr einfach überprüfen ob der Login erfolgreich ist:

new rows, fields;
cache_get_data(rows, fields);
if(rows == 0) {
// Falsches Passwort
return 0;
} else {
// Passwort korrekt, lade Daten
}

Auch die Zeile mit dem Geld kannst du anpassen:
SetPlayerEuro(playerid, mysql_GetInt("db_accounts", "Money", "pName", PlayerName(playerid)));

Hier hast du bereits das Geld aus der Datenbank geholt, also kannst du doch einfach folgendes schreiben:
SetPlayerEuro(playerid, pInfo[playerid][Money]);


Sieht aber schon ganz gut aus was du da machst

Erstmal fällt mir auf, dass du in der Abfrage gar kein WHERE verwendest und so einfach alle Benutzer aus der Datenbank lädst. Das ist nicht sehr effizient Versuchs mal mit dem Ansatz:

new escapedName[MAX_PLAYER_NAME * 2];
new name[MAX_PLAYER_NAME];
new query[512];


GetPlayerName(playerid, name, sizeof(name));


// Verhindert mögliche SQL Injections. Beim Spielernamen zwar unwahrscheinlich, aber man weiß ja nie
mysql_real_escape_string(name, escapedName);

format(query, sizeof(query), "SELECT ... FROM ... WHERE `name` = '%s'", escapedName);


mysql_function_query( // .......


So bekommst du nur den Benutzer aus der Datenbank, der auch den entsprechenden Namen hat. Du kannst dies natürlich auch gleich mit einer Passwortabfrage verknüpfen:


... WHERE `name` = '%s' AND `password` = '%s'


Bei Daten, die in Form von Strings so an die Datenbank übergeben werden solltest du jedoch immer darauf achten, dass du sie mit mysql_real_escape_string(source[], destination[]) escapest. Dadurch werden SQL Injections verhindert. Es wäre ja schade wenn das Passwort eines Users plötzlich lautet:

'; DROP TABLE `db_bank` --

Nun zu deiner eigentlichen Frage:
Bei mysql_function_query läuft das ganz genau so wie bei SetTimerEx. Du gibst erst in einem String an was für Parameter übergeben werden sollen, bei dir eine Zahl und ein String, also:

"is"

Und darauf hin folgen dann die weiteren Parameter, also:

mysql_function_query(MySQLHandle, query, true, "OnPlayerLoad", "is", playerid, key);

Das beste Tutorial was ich dazu kenne ist immer noch das hier:http://forum.sa-mp.com/showthread.php?t=337810

edit: Ob du wirklich umsteigen möchtest bleibt dir überlassen. Wie gesagt: Auf den ersten Blick scheint es sehr kompliziert zu sein, aber man gewöhnt sich daran.

Aus deinem Code ist nicht genau ersichtlich wie deine Tabellenstruktur ist, aber ich gehe mal davon aus, dass die id aus einer anderen Tabelle kommt
als db_bank.

Soweit ich mich erinnere kann man mit dem Plugin nur eine Abfrage zugleich durchführen, da stets der gleiche Zwischenspeicher genutzt wird für mysql_store_result() und mysql_free result(). Es ist also nicht möglich verschachtelte Abfragen wie in deinem Code zu machen, mir fallen für das Problem zwei Lösungsansätze ein.

Der erste Weg ist nicht sehr elegant und führt immer noch dazu, dass du viele Abfragen machst. Du speicherst einfach in der Schleife alle IDs in ein Array und machst nach mysql_free_result() eine weitere Schleife, die über das Array geht.
Diesen Weg kann ich jedoch nicht empfehlen.

Der zweite Weg sind Joins. Durch Joins kannst du kurz gesagt Daten aus mehreren Tabellen in einer Abfrage verknüpfen. Dies macht deine Abfragen zwar etwas komplexer, aber macht deinen Code anschliessend deutlich einfacher.
Hier ein Tutorial dazu: http://wiki.selfhtml.org/wiki/…_SQL/Einfuehrung_in_Joins
Ich weiß nicht woher die ID kommt, aber ich nehme einfach mal an sie kommt aus deiner Spielertabelle:

SELECT a.`id`, b.`type`, b.`number`, b.`nickname`
FROM `db_users` a, `db_bank` b
WHERE a.`id` = b.`id`

Weiterhin kannst du durch dieses Konzept auch gleich den Spielernamen aus der db_bank Tabelle entfernen, da du ihn ja direkt aus der db_users Tabelle holen kannst.

SELECT a.`id`, b.`type`, b.`number`, a.`nickname`
FROM `db_users` a, `db_bank` b
WHERE a.`id` = b.`id`

Dadurch hast du weniger redundanzen in deiner Datenbank und letztendlich eine einfachere Tabellenstruktur.

edit: Wenn deine Queries zu lang werden kannst du sie einfach mit strcat zusammenfügen:


new teilstring[128];
new string[512];


format(teilstring, sizeof(teilstring) , ".......", ....);
strcat(string, teilstring, sizeof(string));

format(teilstring, sizeof(teilstring) , ".......", ....);
strcat(string, teilstring, sizeof(string));


Und so weiter... strcat hängt einfach teilstring an string an.