[SAMMELTHREAD] Kleine Scripting Fragen

Kaliber · 25. August 2023

Du hast da e_modelid, statt modelid stehen.

Es ist halt maximal stupid modelid als globale Variable zu haben. Fast so wie wenn man playerid o.ä. als globale Variable hätte.

Nenn es um zu gModelid (für global) oder sowas. Oder lass es ganz weg eig ist es unnötig.

#Handy

</MRX> · 26. August 2023

Wenn ich es entferne, bleibt der Error, wenn ich es modelid ist der Error weg, aber der Server startet nicht wegen

Code

[00:11:33] [debug] Run time error 19: "File or function is not found"
[00:11:33] [debug]  KillPlayerTimers
[00:11:33] [debug] Run time error 19: "File or function is not found"
[00:11:33] [debug]  KillPlayerTimers
[00:11:33] Script[gamemodes/pRPv3.0bEdited.amx]: Run time error 19: "File or function is not found"
[00:11:33] Number of vehicle models: 1

Wenn ich es in gModelid umnenne, ist der Error wieder da

DJSatrox · 26. August 2023

dann guck in welchen enum du modelid verwendest und nenne dieses um dann geht auch modelid bei dem public

zb beim autohaus mache ich immer ahmodel oder amodelcar halt nix mit modelid im enum / new

</MRX> · 26. August 2023

das habe ich auch schon versucht der fehler beleibt

Ioannis20x · 9. September 2023

Aloha miteinander,

ich habe seit kurzem ein Problem mit meinem Fahrzeugsystem und zwar speichert es die Farbe des Fahrzeuges nicht ab.

Ich habe überhaupt keine Ahnung woran das liegen könnte, da der mysql log leer ist.

Wäre nett wenn ihr nochmal drüber schauen könntet, vielleicht findet ihr ja einen Fehler.

Vielen Dank schonmal im Voraus.

Code

Code zum Speichern des Fahrzeuges
mysql_format(dbhandle,query,sizeof(query),"UPDATE autos SET f1='%i',f2='%i', x='%f', y='%f', z='%f' WHERE id='%i'",cInfo[i][farbe1], cInfo[i][farbe2], cInfo[i][c_x], cInfo[i][c_y], cInfo[i][c_z], cInfo[i][db_id]);

Code

Funktion zum Laden der Fahrzeuge
loadPlayerCars(playerid)
{
    new query[128];
    mysql_format(dbhandle,query,sizeof(query),"SELECT * FROM autos WHERE besitzer='%i'",sInfo[playerid][db_id]);
    mysql_tquery(dbhandle,query,"OnPlayerCarsLoad","d",playerid);
    return 1;
}

Code

public OnPlayerCarsLoad(playerid)
{
    new num_fields,num_rows;
    cache_get_row_count(num_rows);
    cache_get_field_count(num_fields);
    if(!num_rows)return 0;
    for(new i=0; i<num_rows; i++)
    {
new id=getFreeCarID();
cache_get_value_name_int(i,"model",cInfo[id][model]);
cache_get_value_name(i,"besitzer",cInfo[id][besitzer],128);
cache_get_value_name_float(i,"x",cInfo[id][c_x]);
cache_get_value_name_float(i,"y",cInfo[id][c_y]);
cache_get_value_name_float(i,"z",cInfo[id][c_z]);
cache_get_value_name_float(i,"r",cInfo[id][c_r]);
cache_get_value_name_int(i,"id",cInfo[id][db_id]);
cache_get_value_name_int(i,"f1+",cInfo[id][farbe1]);
cache_get_value_name_int(i,"f2",cInfo[id][farbe2]);
cache_get_value_name_float(i,"dl",cInfo[id][dl]);
cache_get_value_name(i,"Kennzeichen",cInfo[id][kennzeichen],128);
cache_get_value_name_int(i,"tank",tank[id]);
cInfo[id][id_x]=CreateVehicle(cInfo[id][model],cInfo[id][c_x],cInfo[id][c_y],cInfo[id][c_z],cInfo[id][c_r],cInfo[id][farbe1],cInfo[id][farbe2],-1);
    }
    return 1;
}

Alles anzeigen

Grüße

DJSatrox · 9. September 2023

send mal alles vom speichern der fahrzeuge sieht mir nach getvehiclecolor aus was fehlt

Ioannis20x · 9. September 2023

Zitat von DJSatrox

send mal alles vom speichern der fahrzeuge sieht mir nach getvehiclecolor aus was fehlt

Code

for(new i=0; i<sizeof(cInfo); i++)
    {
        if(cInfo[i][id_x]==0)continue;
        if(strcmp(cInfo[i][besitzer],getPlayerName(playerid))==1)continue;
        GetVehiclePos(cInfo[i][id_x],cInfo[i][c_x],cInfo[i][c_y],cInfo[i][c_z]);
        GetVehicleZAngle(cInfo[i][id_x],cInfo[i][c_r]);
        new query[128];
        mysql_format(dbhandle,query,sizeof(query),"UPDATE autos SET f1='%i',f2='%i', x='%f', y='%f', z='%f' WHERE id='%i'",cInfo[i][farbe1],cInfo[i][farbe2],cInfo[i][c_x],cInfo[i][c_y],cInfo[i][c_z], cInfo[i][db_id]);
        mysql_tquery(dbhandle,query);
        DestroyVehicle(cInfo[i][id_x]);
        cInfo[i][id_x]=0;
    }

Alles anzeigen

Kaliber · 9. September 2023

In deinem Update Query, speicherst du ja auch nicht die Fahrzeugfarben?!

Wieso sollte das auch funktionieren...

Wie sieht denn die Datenbankstruktur aus?

Ioannis20x · 9. September 2023

Zitat von Kaliber

In deinem Update Query, speicherst du ja auch nicht die Fahrzeugfarben?!

Wieso sollte das auch funktionieren...

Wie sieht denn die Datenbankstruktur aus?

Doch das sind ja farbe1 und farbe2.

DB Struktur sieht folgendermaßen aus:

Spoiler anzeigen

Kaliber · 9. September 2023

Oh sry, sind ja ganz am Anfang, das habe ich übersehen, ist schon spät.

Okay, dann haben die wohl einfach den falschen Wert?

Du kannst mit mysql_log(DEBUG); unter OnGameModeInit den MySQL Log auch kurzzeitig aktivieren, dann wird unter logs/ der MySQL Log angelegt.

Aber ich vermute du musst GetVehicleColor(cInfo[i][id_x], cInfo[i][farbe1],cInfo[i][farbe2]); evtl noch vor dem Update Query aufrufen.

Wenn die Funktion nicht definiert ist, wie stellst du dir sonst so vor an die Farben zu kommen? Setzt du die immer?

Alternativ zu dem Log, kannst auch einfach n print setzen: printf("%d|%d",cInfo[i][farbe1],cInfo[i][farbe2]); und reinschauen, was drinnen steht.

Ich vermute 0|0, weil du es nirgends updatest.

Ioannis20x · 10. September 2023

Zitat von Kaliber

Oh sry, sind ja ganz am Anfang, das habe ich übersehen, ist schon spät.

Okay, dann haben die wohl einfach den falschen Wert?

Du kannst mit mysql_log(DEBUG); unter OnGameModeInit den MySQL Log auch kurzzeitig aktivieren, dann wird unter logs/ der MySQL Log angelegt.

Aber ich vermute du musst GetVehicleColor(cInfo[i][id_x], cInfo[i][farbe1],cInfo[i][farbe2]); evtl noch vor dem Update Query aufrufen.

Wenn die Funktion nicht definiert ist, wie stellst du dir sonst so vor an die Farben zu kommen? Setzt du die immer?

Alternativ zu dem Log, kannst auch einfach n print setzen: printf("%d|%d",cInfo[i][farbe1],cInfo[i][farbe2]); und reinschauen, was drinnen steht.

Ich vermute 0|0, weil du es nirgends updatest.

Alles anzeigen

Leider habe ich im Log nichts rausbekommen.

Weder mit der einen Methode noch mit der anderen.

Bei "mysql_log" hatte ich "ALL" drin gehabt.

Außerdem werden die Fahrzeuge mittlerweile nicht mehr gelöscht.

Ich habe die Abfrage auf den Besitzer etwas angepasst, da die DB-ID des Users dort abgespeichert wird und nicht der Name.

Sieht mittlerweile so aus:

C

for(new i=0; i<sizeof(cInfo); i++)
    {
        if(cInfo[i][id_x]==0)continue;
        if(cInfo[i][besitzer] != sInfo[playerid][db_id])continue;
        GetVehiclePos(cInfo[i][id_x],cInfo[i][c_x],cInfo[i][c_y],cInfo[i][c_z]);
        GetVehicleZAngle(cInfo[i][id_x],cInfo[i][c_r]);
        new query[128];
        mysql_format(dbhandle,query,sizeof(query),"UPDATE autos SET f1='%i',f2='%i', x='%f', y='%f', z='%f' WHERE id='%i'",cInfo[i][farbe1],cInfo[i][farbe2],cInfo[i][c_x],cInfo[i][c_y],cInfo[i][c_z], cInfo[i][db_id]);
        mysql_tquery(dbhandle,query);
        DestroyVehicle(cInfo[i][id_x]);
        cInfo[i][id_x]=0;
    }

Alles anzeigen

Kaliber · 10. September 2023

Zitat von Ioannis20x

Bei "mysql_log" hatte ich "ALL" drin gehabt.

Ja, dann wird es auch geloggt. Das liegt in einer extra Datei...da musst du schon einmal in deinem Verzeichnis gründlich suchen xD

Der Code bringt uns nichts. Wir brauchen entweder die Logs oder per printf den Output...

Ioannis20x · 10. September 2023

Ich habe die Datei gefunden.

Die war echt schwer zu finden.

Ich habe die Datei durchsucht, aber habe die query zum Speichern nicht gefunden :o

Hier nochmal ein Link zur Datei: https://pastebin.com/W1JWK5cT

Kaliber · 10. September 2023

Zitat von Ioannis20x

aber habe die query zum Speichern nicht gefunden :o

Ja, das erklärt ja dann auch, wieso das Auto nicht zerstört wurde.

Einfach weil es nie aufgerufen wurde.

Zitat von Ioannis20x

cache_get_value_name(i,"besitzer",cInfo[id][besitzer],128);

Wieso? Besitzer ist doch ein Integer? Oder hast du das etwa als String behandelt? Irgendwas stimmt hier ganz und gar nicht.

Das soll doch eine DB-ID ein. Also deklariere es als Integer und nicht als String. Da steht ja kein Name drinnen, sondern nur die DB id.

Ich denke, wenn du das fixxt und prüfst, dass in der Datenbank deine DB-ID da drinnen steht, dann sollte es klappen.

Ioannis20x · 10. September 2023

Hab das alles mal nochmal angeschaut und tatsächlich klappt das mit dem Aufrufen.

Leider spuckt der mir das raus:

Code

[18:31:14] [DEBUG] mysql_format(1, 0xF5982290, 128, "UPDATE autos SET f1='%i', f2='%i', x='%f', y='%f', z='%f' WHERE id='%i'")
[18:31:14] [DEBUG] mysql_format: return value: '86'
[18:31:14] [DEBUG] mysql_tquery(1, "UPDATE autos SET f1='0', f2='0', x='0.000000', y='0.000000', z='0.000000' WHERE id='0'", "", "")

Kaliber · 10. September 2023

Ja, wie ich schon sagte, 0|0, bedeutet, die Farben sind nicht gesetzt.

Hast du es mit GetVehicleColor probiert?

Ioannis20x · 10. September 2023

Zitat von Kaliber

Ja, wie ich schon sagte, 0|0, bedeutet, die Farben sind nicht gesetzt.

Hast du es mit GetVehicleColor probiert?

GetVehicleColor existiert bei mir nicht.

Code

(1298) : error 017: undefined symbol "GetVehicleColor"

Kaliber · 10. September 2023

Ohne die Farbvariablen zu setzen, wie hast du dir vorgestellt, dass dort die Farbe des Fahrzeuges landet? 🧐

Hier die Include von Ryder damals, die das übernimmt

Ioannis20x · 10. September 2023

Zitat von Kaliber

Ohne die Farbvariablen zu setzen, wie hast du dir vorgestellt, dass dort die Farbe des Fahrzeuges landet? 🧐

Hier die Include von Ryder damals, die das übernimmt

Danke dir für den Include.

Ich werde es gleich mal testen

Die Farbe setze ich diese tatsächlich per Befehl.

C

CMD:vcolor(playerid,params[])
{
new vID,c1,c2;
if(sscanf(params,"iii",vID,c1,c2))return SendClientMessage(playerid,COLOR_GREY,"INFO: /vcolor [vID] [c1] [C2]");
ChangeVehicleColor(vID, c1, c2);
cInfo[vID][farbe1] = c1;
cInfo[vID][farbe2] = c2;
return 1;
}

Kaliber · 10. September 2023

Zitat von Ioannis20x

cInfo[vID][farbe1] = c1; cInfo[vID][farbe2] = c2;

Ah okay, ja das hier ist kompletter Quatsch.

Mich wundert es, dass du da keine Fehler bekommst.

Zumindest wenn du crashdetect laufen hättest, würdest du hier einen Overflow sehen.

(Es sei denn, dass du nur ganz wenige Fahrzeuge auf dem Server hast, bzw dein Auto eine niedrige vehicleid bisher hatte)

C

for(new i; i<sizeof(cInfo); i++)
{
    if(cInfo[i][id_x] != vID) continue;
    cInfo[i][farbe1] = c1;
    cInfo[i][farbe2] = c2;
    break;
}

Das ist es, was du da eigentlich nutzen magst.

i ist ja schließlich nicht die vehicleid, sondern der Index vom Fahrzeug.