Problem mit Haussystem

inZeyne44 · 28. April 2022

Guten Abend meine Freunde,

ich habe ein Problem mit meinem Haussystem.

Vorerst: ich habe mich im Forum umfangreich umgeschaut aber bin nicht fündig geworden.

Folgendes Problem habe ich ingame:

Wenn ich ein Haus erstelle, dann passiert erstmal alles wie gewollt, Pickup und 3DText werden geladen und das Haus legt

sich auch direkt in der Datenbank an, sobald ich das haus lösche, verschwindet es auch aus der Datenbank, weshalb ich einen ladefehler ausschließe.

nun zum Problem: sobald ich ein zweites Haus erstelle, so verschwindet das erste Hausicon samt Text und Pickup, jedoch ist es weiterhin in der Datenbank vorhanden.

Auch an der stelle kann ich das haus nicht löschen.

Ich vermute mal es liegt an meinem /hcreate befehl, jedoch finde ich nichts was auf den Fehler hindeuten könnte...

Code

ocmd:chouse(playerid,params[])
{
    if(!isAdmin(playerid,3))return SendClientMessage(playerid,RED,"You dont have permissions to do this.");
    new Float:xc, Float: yc, Float: zc;
    GetPlayerPos(playerid, xc, yc, zc);
    new id=getFreeHausID();
    hInfo[id][h_x]=xc;
    hInfo[id][h_y]=yc;
    hInfo[id][h_z]=zc;
    hInfo[id][ih_x]=0.0;
    hInfo[id][ih_y]=0.0;
    hInfo[id][ih_z]=0.0;
    hInfo[id][h_interior]=0;
    strmid(hInfo[id][h_besitzer], "", 0, MAX_PLAYER_NAME, MAX_PLAYER_NAME);
    hInfo[id][h_preis]=1;
    updateHaus(id);
    new query[256];
    format(query, sizeof(query), "INSERT INTO haus (h_x, h_y, h_z, ih_x, ih_y, ih_z, h_interior, h_preis) VALUES ('%f', '%f', '%f', '0.0', '0.0', '0.0', '0', '1')", xc, yc, zc);
    mysql_function_query(dbhandle, query, true, "OnHausCreated", "i", id);
    return 1;
}

Alles anzeigen

So lade ich meine Häuser

Code

public OnHousesLoad()
{
    new num_fields,num_rows;
    cache_get_data(num_rows,num_fields,dbhandle);
    if(!num_rows)return 1;
    for(new i=0; i<num_rows; i++)
    {
        new id=getFreeHausID();
        hInfo[id][h_x]=cache_get_field_content_float(i,"h_x", dbhandle);
        hInfo[id][h_y]=cache_get_field_content_float(i,"h_y", dbhandle);
        hInfo[id][h_z]=cache_get_field_content_float(i,"h_z", dbhandle);
        hInfo[id][ih_x]=cache_get_field_content_float(i,"ih_x", dbhandle);
        hInfo[id][ih_y]=cache_get_field_content_float(i,"ih_y", dbhandle);
        hInfo[id][ih_z]=cache_get_field_content_float(i,"ih_z", dbhandle);
        hInfo[id][h_interior]=cache_get_field_content_int(i,"h_interior", dbhandle);
        new tmp_name[MAX_PLAYER_NAME];
        cache_get_field_content(i, "besitzer", tmp_name, dbhandle);
        strmid(hInfo[id][h_besitzer], tmp_name, 0, sizeof(tmp_name), sizeof(tmp_name));
        hInfo[id][h_id]=cache_get_field_content_int(i,"id", dbhandle);
        hInfo[id][h_preis]=cache_get_field_content_int(i,"h_preis", dbhandle);
        updateHaus(id);
    }
    return 1;
}

Alles anzeigen

Und noch ein paar Infos zu den Icons/Texten, da diese immer verschwinden sobald ich ein neues Haus erstelle.

Code

updateHaus(id)
{
    new string[128];
    if(hInfo[id][h_pickup])
    {
        DestroyPickup(hInfo[id][h_pickup]);
    }
    if(hInfo[id][h_text])
    {
        Delete3DTextLabel(hInfo[id][h_text]);
    }
    if(!strlen(hInfo[id][h_besitzer]))
     {
          hInfo[id][h_pickup]=CreatePickup(1273, 1, hInfo[id][h_x], hInfo[id][h_y], hInfo[id][h_z], -1);
        format(string,sizeof(string), "House for sale\nPrice: %i$\nUse: /buyhouse", hInfo[id][h_preis]);
         hInfo[id][h_text]=Create3DTextLabel(string, RED, hInfo[id][h_x], hInfo[id][h_y], hInfo[id][h_z], 10, 0, 1);
    }
    else
    {
          hInfo[id][h_pickup]=CreatePickup(19522, 1, hInfo[id][h_x], hInfo[id][h_y], hInfo[id][h_z], -1);
          format(string,sizeof(string), "House owner: %s\nUse: /enter", hInfo[id][h_besitzer]);
          hInfo[id][h_text]=Create3DTextLabel(string, WHITE, hInfo[id][h_x], hInfo[id][h_y], hInfo[id][h_z], 10, 0, 1);
    }
    return 1;
}

Alles anzeigen

Das System ist auf MrMonat angelehnt, da ich seine Arbeit sehr sauber finde.

Ich hoffe, dass mir jemand helfen kann, bis dahin mache ich mich weiter auf die Fehlersuche.

MFG

Kaliber · 28. April 2022

Zitat von inZeyne44

jedoch ist es weiterhin in der Datenbank vorhanden.

Dann liegt das Problem vermutlich im Lösch Befehl des Hauses

Also diesen gerne mal posten gerade das Query.

Vermutlich schlägt auch einfach das Query fehl, über den mysql_log können solche Fehler sehr gut erkannt werden

inZeyne44 · 28. April 2022

Code

ocmd:dhouse(playerid,params[])
{
    if(!isAdmin(playerid,3))return SendClientMessage(playerid,RED,"You dont have permissions to do this.");
    for(new i=0; i<sizeof(hInfo); i++)
    {
        if(!hInfo[i][h_id])continue;
        if(!IsPlayerInRangeOfPoint(playerid, 5, hInfo[i][h_x], hInfo[i][h_y], hInfo[i][h_z]))continue;
        new query[128];
        format(query,sizeof(query), "DELETE FROM haus WHERE id='%i'", hInfo[i][h_id]);
        mysql_function_query(dbhandle, query, false, "", "");
        hInfo[i][h_x]=0.0;
        hInfo[i][h_y]=0.0;
        hInfo[i][h_z]=0.0;
        hInfo[i][ih_x]=0.0;
        hInfo[i][ih_y]=0.0;
        hInfo[i][ih_z]=0.0;
        hInfo[i][h_id]=0;
        hInfo[i][h_preis]=0;
        hInfo[i][h_interior]=0;
        if(hInfo[i][h_pickup])
        {
            DestroyPickup(hInfo[i][h_pickup]);
        }
        if(hInfo[i][h_text])
        {
            Delete3DTextLabel(hInfo[i][h_text]);
        }
        return 1;
    }
    return 1;
}

Alles anzeigen

Das ist mein Delete Befehl.

Nur wie gesagt kommt der nicht zum Einsatz.

Mit /chouse erstelle ich ein haus, erstelle ich ein weiteres so verschwindet das erste komplett (Icon,Text) aber es ist weiterhin in der DB vorhanden.

Ansonsten läuft alles fehlerfrei.

Wenn ich nur eins erstelle und das lösche, so passiert alles nach vorschrift.

//Es ist praktisch der /chouse befehl der das erste Haus zum löschen bringt sobald ein weiteres erstellt wird.

Kaliber · 28. April 2022

Zitat von inZeyne44

Nur wie gesagt kommt der nicht zum Einsatz.

Gesagt wurde das nicht, aber ok.

Ja, ich glaube ich verstehe was du sagen wolltest.

Geh mal unter OnGameModeInit und füge das hier da ganz oben ein:

C

CreatePickup(1239,23, 0,0,-30,99);
Create3DTextLabel("T",-1,0,0,-30,0,99,0);

Das Problem ist nämlich, dass die IDs von CreatePickup und Create3DTextLabel bei 0 starten und das der Standard Wert von den Icons ist.

Damit umgehst du quasi diesen "ID 0 BUG"

**Andosius** · 28. April 2022

Das Problem wird mit einer sehr großen Sicherheit in getFreeHausID liegen. Das würde erklären, warum das Pickup dann an einer neuen Stelle steht.

Vielleicht findest du den Fehler ja selbst, kannst den Code - wenn du nicht mehr weiter weißt - gerne noch teilen.

LG

inZeyne44 · 28. April 2022

Zitat von Kaliber
Gesagt wurde das nicht, aber ok.

Ja, ich glaube ich verstehe was du sagen wolltest.

Geh mal unter OnGameModeInit und füge das hier da ganz oben ein:
C
CreatePickup(1239,23, 0,0,-30,99);
Create3DTextLabel("T",-1,0,0,-30,0,99,0);
Das Problem ist nämlich, dass die IDs von CreatePickup und Create3DTextLabel bei 0 starten und das der Standard Wert von den Icons ist.
Damit umgehst du quasi diesen "ID 0 BUG"
Alles anzeigen

Leider klappt dies nicht, sieht jedoch sehr interessant aus.

Damit du nochmal ein genaueres Bild hast:

Houseicon bug

streamable.com

(Alle erstellten Häuser bleiben ordnungsgerecht in der DB gespeichert, es verschwindet lediglich das Icon + der Text und an der stelle kann man auch

kein /buyhouse mehr machen. Somit existiert immer nur ein Haus auf dem Server, aber nicht in der DB)

Zitat von Andosius

Das Problem wird mit einer sehr großen Sicherheit in getFreeHausID liegen. Das würde erklären, warum das Pickup dann an einer neuen Stelle steht.

Vielleicht findest du den Fehler ja selbst, kannst den Code - wenn du nicht mehr weiter weißt - gerne noch teilen.

LG

Ich werde mal schauen, jedoch könnte ich es mir nicht vorstellen, da dies eigentlich dazu dient, es richtig in die DB einzuordnen

Code

getFreeHausID()
{
    for(new i=0; i<sizeof(hInfo); i++)
    {
        if(hInfo[i][h_id]==0)return 1;
    }
    return 0;
}

Da ich aber kein Profi bin werde ich mir den Tipp zu Herzen nehmen und direkt mal schauen was mir so auffällt, vielen vielen Dank euch beiden, ist auch nicht

selbstverständlich anderen bei deren Problemen zu helfen :b ich schätze es wirklich sehr

Kaliber · 28. April 2022

Ah, wie sieht denn OnHausCreated aus?

inZeyne44 · 28. April 2022

C

//Oben im Script

forward OnHausCreated(id);

//Die letzen 4 Zeilen vom command /chouse (createhouse)

    new query[256];
    format(query, sizeof(query), "INSERT INTO haus (h_x, h_y, h_z, ih_x, ih_y, ih_z, h_interior, h_preis) VALUES ('%f', '%f', '%f', '0.0', '0.0', '0.0', '0', '1')", xc, yc, zc);
    mysql_function_query(dbhandle, query, true, "OnHausCreated", "i", id);
    return 1;
}

public OnHausCreated(id)
{
    hInfo[id][h_id]=cache_insert_id();
}

Alles anzeigen

Kaliber · 28. April 2022

Hast du in der MySQL Datenbank die Spalte id als Primary Key und Auto_Increment eingestellt?

Gerne mal die Tabellenstruktur zeigen

inZeyne44 · 28. April 2022

https://i.imgur.com/Bimsvb4.png

so sieht es aus ich komme einfach nicht weiter, da wie gesagt mit der DB alles in ordnung ist. Häuser werden direkt angelegt und das zuletzt erstelle auch gelöscht, sofern ich den /dhouse (deletehouse) command nutze

**Andosius** · 28. April 2022

Du gibst den falschen Wert in getFreeHausID zurück. Dort gibt es bei dir nur 1 und 0, daran liegt es.

C

getFreeHausID()
{
    for(new i=0; i<sizeof(hInfo); i++)
    {
        if(hInfo[i][h_id]==0)return 1;
    }
    return 0;
}

// hiermit ersetzen
getFreeHausID() {
    for(new i = 0; i < sizeof(hInfo); i++) {
        if(hInfo[i][h_id] == 0) return i;
    }
    return -1;
}

Alles anzeigen

inZeyne44 · 28. April 2022

Zitat von Andosius

Du gibst den falschen Wert in getFreeHausID zurück. Dort gibt es bei dir nur 1 und 0, daran liegt es.

C

getFreeHausID()
{
    for(new i=0; i<sizeof(hInfo); i++)
    {
        if(hInfo[i][h_id]==0)return 1;
    }
    return 0;
}

// hiermit ersetzen
getFreeHausID() {
    for(new i = 0; i < sizeof(hInfo); i++) {
        if(hInfo[i][h_id] == 0) return i;
    }
    return -1;
}

Alles anzeigen

OMG du bist der König...... ehrlich gesagt wäre ich da nicht mal als letztes drauf gekommen, es funzt nun einwandfrei wie gewollt =)

**Andosius** · 28. April 2022

Kleine Anmerkung noch. Überprüfe beim laden der Häuser und dessen Erstellung auf jeden Fall immer, ob getFreeHausID -1 zurück gibt. Dann kannst du nämlich keine neuen Häuser mehr erstellen, da das Array nicht mehr hergibt.

So ersparst du dir auch Crashes da -1 ein ungültiger Index ist.

inZeyne44 · 28. April 2022

Zitat von Andosius

Kleine Anmerkung noch. Überprüfe beim laden der Häuser und dessen Erstellung auf jeden Fall immer, ob getFreeHausID -1 zurück gibt. Dann kannst du nämlich keine neuen Häuser mehr erstellen, da das Array nicht mehr hergibt.

So ersparst du dir auch Crashes da -1 ein ungültiger Index ist.

Vielen Dank 🙏

Problem mit Haussystem

Beitrag von Kaliber (28. April 2022)

breadfish.de 29. Mai 2022

Ähnliche Themen

Wir suchen einfach alles, Von Gangs über Meds,Cops und Normalos :)

🔰Lost-V.de🔰 | 🔰Roleplay🔰 | 🔰SaltyChat🔰 | 💊Labore💊 | 🔰Häuser🔰 | 🔰Autohändler🔰