Beiträge von Kaliber

Halte mich für altmodisch, aber unter einem Tutorial verstehe ich etwas anderes, als wenn ich jemandem wortlos zuschaue wie er Code schreibt.

Das erinnert eher an ein ASMR Video 😅

Es würde ja einfach reichen ein Github-Repo zu machen, wo du deinen Code hochlädst, wenn du diesen nicht einmal erklärst.

Es gibt ja nicht einmal Kommentare und keinen Link zum Source-Code.

Also meiner Meinung nach, sollte man in Tutorials reden und den Code erklären, wieso man es so macht und nicht anders, was man sich dabei denkt und wie das alles zusammenhängt, sonst schreibt man das blind ab und dann ist ja auch keinem geholfen.

Wie hast denn File definiert?

Der Befehl ergibt ja vorne und hinten auch keinen Sinn.

item ist immer leer hahaha xD

menge ist immer 0

und params ist verfälscht durch die "items" eingabe

...Jetzt mal nichts für ungut, aber schau dir doch ein paar Basics mal an oder lass dir etwas Nachhilfe geben...

Darauf warten, dass jetzt jemand den Befehl für dich macht und das dann beim Nächsten auch usw bringt doch niemandem wirklich was.

Du musst -= und += verwenden

Kenne den nicht, war aber da schon aktiv

Versuche es mal mit Code-Logik:

ocmd:mlift(playerid, params[])
{
    if(!isPlayerInFraktion(playerid, 2))return SCM(playerid, COLOR_DARKRED, "[ERROR]: Du gehörst nicht zum San Andreas Medical Center!");
    if(IsPlayerInRangeOfPoint(playerid, 5, 1146.9622,-1315.8383,13.6594))
    {
        SetPlayerPos(playerid, 1160.9453,-1328.1024,31.5022);
        SCM(playerid, COLOR_YELLOW, "Du bist nun auf dem Dach des Los Santos Medical Centers!");
    }
    else if(IsPlayerInRangeOfPoint(playerid, 5, 1160.9453,-1328.1024,31.5022))
    {
        SetPlayerPos(playerid, 1146.9622,-1315.8383,13.6594);
        SCM(playerid, COLOR_YELLOW, "Du bist nun vor dem Los Santos Medical Center!");
    }
    else SCM(playerid, COLOR_DARKRED,"[ERROR]: Du bist nicht in der nähe vom Aufzug!");
    return 1;
}

Und wo, bzw wann rufst du das auf?

Naja, da scheint ja alles richtig zu sein, da ist der Wert ja auf 1.

Wie rufst du denn saveCar auf?

Das wäre mal interessant, in welchem Kontext die Funktion aufgerufen wird.

Ja, weil besitzer immernoch auf 0 ist.

Da wird der falsche Wert genommen.

Wie setzt du denn von PlayerInfo die db_id?

Beim laden dieser id machst du irgendwas falsch...

Zitat von RaPTiiC

Was soll ich den als Verschlüsselung nutzen hab das halt von Jeffry genommen und wie würde ich das denn jetzt noch ändern (ist es einfach und fix oder recht schwer)?

1. Ist das keine Verschlüsselung, lies dir meinen Beitrag (den ich doch eben verlinkt habe) dazu mal durch.

2. Jeffry solltest das Tutorial mal in dieser Hinsicht updaten, das wirklich keine gute Idee, das so zu gestalten.

Zitat von RaPTiiC

Was genau soll ich jetzt in der Tabelle aus der DB löschen (die beiden Fahrzeuge)?

Exakt.

1. Bitte nutze kein MD5 beim Passwort, da kannst du es eig schon gleich bleiben lassen...Zudem steht dein Passwort jetzt hier in Klartext...(Hash Verfahren genauer erklärt)
2. Das Select query ist nun richtig, aber ich sehe kein Insert query. Lösche nochmal die Sachen aus der Tabelle und mache das von Neu.
3. Lösche jetzt die Log Datei, damit sie gleich frisch erstellt wird.

Zeige dann nochmal den Log.

Da musst du den Fehler suchen.

Entweder bei der Zuweisung oder bei der Speicherung...printe es halt mal oder zeig uns mal deinen MySQL Log.

(Den aktivierst du indem du unter OnGameModeInit mysql_log(ALL); eingibst)

Zitat von RaPTiiC

aber besitzer wurder doch als INT gespeichert

Ja, aber 1 wäre glaube ich richtig und nicht 0.

besitzer = 0 in der Datenbank sieht irgendwie falsch aus, meinst du nicht?

Da ist wohl beim Speichern oder der ID Zuweisung was schief gelaufen...

Zitat von RaPTiiC

bei einem gmx sind diese leider komplett weg

Super.

...wie sieht die Datenbank aus?

Was sagen die Querys?

Hast du irgendwas geprintet?

Du solltest immer Informationen liefern, einfach nur sagen ja, bla bla geht nicht, da kann man dir meist nicht helfen.

Zitat von RaPTiiC

aber jetzt sollte alles ohne Probleme funktionieren?

Ohhh du mächtige Kristallkugel🔮

Da ist der Fehler, wieso hast du db_id im Car-Enum stehen?

Das muss in das PlayerInfo enum.

Wie ist denn db_id im enum definiert?

Zitat von RaPTiiC

loadPlayerCars(playerid,carid)
{
new query[200];
mysql_format(handle, query, sizeof(query), "SELECT * FROM autos WHERE besitzer=%d, model='%d', x='%f', y='%f', z='%f', r='%f'",PlayerInfo[playerid][p_id],cInfo[carid][model],cInfo[carid][c_x],cInfo[carid][c_y],cInfo[carid][c_z],cInfo[carid][c_r]); //966
mysql_tquery(handle, query);
return 1;
}

Alles anzeigen

Fast...

loadPlayerCars(playerid) 
{ 
new query[200]; 
mysql_format(handle, query, sizeof(query), "SELECT * FROM autos WHERE besitzer=%d",PlayerInfo[playerid][p_id]); //966 
mysql_tquery(handle, query, "OnPlayerCarsLoad","i",playerid);
return 1; 
}

Zitat von RaPTiiC

Hab es mehr oder weniger copy&pasted (selbst eingegeben etc.)

Jetzt nichts für ungut oder so, aber du hast das komplett falsch abgetippt und benannt, wtf

Du kannst doch sogar in der Beschreibung dir den Quellcode downloaden und anschauen/nutzen.

Was bei dir loadPlayerCars heißt (was ein absolut Quatsch-Name für die Funktion ist), sieht bei ihm so aus:

saveCarToDB(playerid,carid)
{
    new query[128];
    format(query,sizeof(query),"INSERT INTO autos (besitzer,model,x,y,z,r) VALUES ('%i','%i','%f','%f','%f','%f')",sInfo[playerid][db_id],cInfo[carid][model],cInfo[carid][c_x],cInfo[carid][c_y],cInfo[carid][c_z],cInfo[carid][c_r]);
    mysql_function_query(dbhandle,query,true,"carSavedToDB","i",carid);
    return 1;
}

Fallen dir Unterschiede im Query und im Namen auf?

//Edit: Jetzt hab ich erstmal den Anfang des Threads gelesen, die Syntax hat Maho vorgegeben. Das ist super verwirrend für Neulinge von der Standard-Syntax abzuweichen, generell ist das bad practice, wenn es keinen krassen Vorteil bietet.

Schau dir nochmal in Ruhe seinen Quellcode an (hier für dich mal der Direktlink https://pastebin.com/wvdygCFN)