Danke für die ausführliche Erklärung, jetzt ist mir das Vorgehen und das Problem klar.
Ändere mal beim /addhouse
new query[200];
zu:
new query[500];
Vermutlich schneidet es die WHERE Bedingung auf Grund der Größe genau passend ab, ohne dass ein Sytax Error entsteht.
Nope, immernoch das gleiche Problem... liegts eventuell an meiner Datenbank?
Kannst du bitte den aktuellen Code nochmal posten und das Problem nochmal etwas detaillierter erklären?
Bitte mit Screenshot aus der Datenbank wo das Problem erkennbar ist (vorher / nachher) und dem Server Log dazu, an dem du das Problem erkennst.
Hab aber eigentlich die Datenbank schon mit sizeof(HouseInfo) gefüllt & die IDs haben sich ja sowieso automatisch inkrementiert.
Dann versuche eventuell mal, in diesem UPDATE Query ganz am Ende bei der WHERE Bedingung anstatt der hID des Arrays nur "i" zu nehmen. Der Vermutung nach ist die hID zu dem Zeitpunkt leer, also 0.
Möchte keinen fertig geschrieben Code von dir, oder sonst wem, sondern nur ein paar Hinweise, wie ich am besten Vorgehen sollte.
Du erstellst das Textdraw mit CreatePlayerTextDraw, zeigst es mit PlayerTextDrawShow an und die Interaktion mit dem Textdraw machst du mit OnPlayerClickPlayerTextDraw sowie SelectTextDraw.
Ein Beispiel findest du hier: http://wiki.sa-mp.com/wiki/OnPlayerClickPlayerTextDraw
Einen Dialog für die Passwort Eingabe benötigst du aber trotzdem.
if(!strcmp(PIN[playerid], PlayerInfo[playerid][pPIN], true))
Zu
if(PIN[playerid] == PlayerInfo[playerid][pPIN])
strcmp ist nicht mehr notwendig, da es kein String mehr ist.
PIN[playerid] = strval(inputtext);
Dann aber auch nur
new PIN[MAX_PLAYERS];
Wenn es nur Zahlen sind (max. 9 Stellen), dann auf jeden Fall so.
Hast du es mal mit format probiert?
format(PIN[playerid], sizeof(PIN[]), inputtext);
Falls das auch nicht geht, wird der Code überhaupt aufgerufen? Wenn ja, woran siehst du, dass der PIN nicht oder falsch gespeichert wird? Werden nur Zahlen gespeichert, dann kannst du es auch als Integer speichern.
strmid(PIN[playerid], inputtext, 0, strlen(inputtext));
Falls das nicht hilft, wie hast du PIN deklariert (new...)?
Du meintest ja:
Achso, jetzt verstehe ich was du meinst. Ich dachte du wolltest alle Häuser ohnehin speichern.
Beim Anlegen des Hauses darfst du aber kein "UPDATE" Query nehmen, sondern musst ein INSERT nehmen (wie beim Registrieren hier im Tutorial).
Das INSERT Query gibt dir dann über das aufgerufene Callback die ID des Hauses zurück, für HouseInfo[i][hID].
Mit dem Laden der Daten hats ja nichts zu tun oder? Unter OnGameModeInit hab ich:
Hast du das return 1 entfernt?
Wenn ja, was ist jetzt das Problem? Wenn alle Häuser gespeichert werden, ist das doch richtig, oder?
Total requirements:64032232 bytes
Die Meldung besagt schon, dass du ungewöhnlich viel Speicher verbrauchst.
Ich vermute, du hast ein mehrdimensionales Array eingebaut, welches dazu führt, dass so viel Speicher reserviert werden muss. Vermutlich unabsichtlich.
Ohne Code kann ich dir aber nicht sagen, an was es liegt.
Entfernen kannst du die Meldung (falls der Debug Modus nicht an ist [pawn.cfg mit "-d3"]) mit folgendem Code:
#pragma dynamic 100000
Die Zahl so lange erhöhen, bis die Meldung weg ist.
Schneller kompiliert der Code dadurch aber nicht.
dann hatte ich glaub das Faktionsvehicle system geschrieben und dannach braucht er länger :o
Möglicherweise hast du dort ein großes Array eingebaut, eventuell unabsichtlich.
Beachte, dass mehrdimensionale Array sich natürlich aufmultiplizieren.
Wie groß ist deine AMX Datei?
Mein Pawno Compiler brauch ungewöhnlich lange zu compilieren bei knapp 1600 Zeilen woran kann dies Liegen?
Ein möglicher Grund sind viele Variablen oder sehr große Arrays, oder beides zusammen.
Die Anzahl der Zeilen ist eher zweitrangig.
Was heißt ungewöhnlich lange? Der Code meines ehemaligen Gamemodes hat ca. 1 Minute kompiliert.
Wir haben das in diesem Thread mal ziemlich ausführlich durchgearbeitet: Tankvolumen und Treibstoff an Fahrzeugen zuordnen
Gehe mal durch den Thread und schaue, ob dir das hilft.
dabei wird nur 1x in der Konsole geprintet:
Nimm das return 1; aus der Schleife (nach dem print) raus. Das beendet die Schleife sofort.
Dass die Häuser trotzdem gespeichert werden, dürfte an einem anderen Code liegen, der die Daten in die Datenbank schreibt.
Somit läuft der Timer nicht permanent (?).
Da würde ich trotzdem einen Timer nehmen, der permanent läuft und in der Schleife prüft, ob der Spieler im DM ist.
Das ist bei mehreren Spielern besser, da nur ein Timer läuft.
for(new i = 0; i < MAX_PLAYERS; i++)
{
if(PlayerInDM[i])
{
//Mache etwas
}
}
SetTimerEx("OnSomethingToDo", 1000, "i", playerid);
Wiederholend oder einmalig? Das fehlt hier nämlich.
Einmalig wäre so:
SetTimerEx("OnSomethingToDo", 1000, false, "i", playerid);
Das ist von der Performance überhaupt kein Problem, da es ja nur ein Aufruf ist, das spielt keine Rolle wann und wo du den machst.
Wiederholend wäre so:
SetTimerEx("OnSomethingToDo", 1000, true, "i", playerid);
Da würde allerdings ein einziger Timer mehr Sinn machen, mit einem Loop durch alle Spieler.
Bei der genannten Anforderung gehe ich aber davon aus, dass es sich um einen einmaligen Aufruf handelt, sprich die erste Variante.
Und das speichern? Ist wahrscheinlich auch nicht so effizient oder? Wüsste aber nicht wie ich´s nur 1x abschicken könnte.
Grundsätzlich passt das.
Man könnte es zwar alles (für alle Häuser) in ein großes Query packen und mit INSERT INTO ... ON DUPLICATE KEY UPDATE ... absenden, das macht aber relativ wenig Sinn.
Wobei es die bessere Variante wäre, ein Haus immer dann direkt zu speichern, wenn eine Änderung passiert, so oft ändert sich an den Häusern ja nichts.
Dann hättest du auch kein Problem, wenn der Server mittendrin abstürzt, ohne OnGameModeExit aufzurufen.
Sprich, nicht einmal "SaveAllHouses", sondern z.B. dort wo sich der Besitzer des Hauses ändert (Kauf, Verkauf, etc...) eine Funktion "SaveHouse(id)" aufrufen.
Sowas sollte aber natürlich nicht in einem Timer stehen, der jede Sekunde aufgerufen wird, im ungünstigsten Fall noch für alle Häuser.
Welche IDs waren das nochmal?
Wetter ID 2 funktioniert. Die anderen müsstest du ausprobieren.
Du musst anstatt der 0 im ersten Parameter das h eintragen.
Für den Index i musst du den Cache Wert deiner Key Spalte (id) auslesen, vor allen anderen, und diesen Wert in i wegschreiben.
Du solltest aus Performance Gründen nur ein Query absenden, ohne die WHERE Bedingung. Das Ergebnis kannst du dann im Callback mit einem Loop durchgehen. Dann wird die Datenbank nur einmal anstatt x mal angesprochen.
kann nur schlecht einschätzen wie groß die query sein muss... schlimm wenn die bisschen überdimensional groß is?^^
Das Query (SELECT...) sind bei dir keine 100 Zeichen. 2000 ist da etwas überdimensioniert.
