Array index out of bounds

Padarom · 29. April 2012, 17:21

Hallo,
ich habe gerade einige Fehler, die ich beim besten Willen nicht wegbekomme:

Quellcode

error 032: array index out of bounds (variable "haus") // 13x - Überall wo haus[i][...] verwendet wird

Hier der relevante Teil des Codes. Hauptsächlich sind die Fehler im Callback LoadHaus.
[pwn]#define MAX_HAUS 350

enum hInfo {
Float:X,
Float:Y,
Float:Z,
Preis,
Gebiet[5],
Besitzer[24],
KeyID,
pickup,
bool:loaded = false
};

new haus[MAX_HAUS][hInfo];

public LoadHaus()
{
new text[128];
for(new i=0; i<MAX_HAUS; i++)
{
if(haus[loaded] == false)
{
format(mQuery, 128, "SELECT * FROM haus WHERE ID = %i", i);
mysql_query(mQuery);
mysql_store_result();
if(mysql_num_rows())
{
mysql_free_result();
haus[i][X] = mysql_GetFloat("haus", "X", "ID", i);
haus[i][Y] = mysql_GetFloat("haus", "Y", "ID", i); // error 032
haus[i][Z] = mysql_GetFloat("haus", "Z", "ID", i); // error 032
haus[i][Besitzer] = mysql_GetString("haus", "Besitzer", "ID", i); // error 032
haus[i][Gebiet] = mysql_GetString("haus", "Gebiet", "ID", i); // error 032
haus[i][KeyID] = mysql_GetInt("haus", "KeyID", "ID", i); // error 032
haus[i][Preis] = mysql_GetInt("haus", "Preis", "ID", i); // error 032
haus[i][loaded] = true;
if(strlen(haus[i][Besitzer]) > 2) // error 032
{ // Bewohnt
format(text, 128, "- Besitzer: %s -\n- Gebiet: %s -", haus[i][Besitzer], haus[i][Gebiet]); // error 032
haus[i][pickup] = CreatePickup(1272, 1, haus[i][X], haus[i][Y], haus[i][Z], -1); // error 032
Create3DTextLabel(text, cAQUA, haus[i][X], haus[i][Y], haus[i][Z]+1, 5, 0, 0); // error 032
}
else
{ // Verkäuflich
format(text, 128, "- Verkäuflich -\n- Preis: $%i -\n- Gebiet: %s -", haus[i][Preis], haus[i][Gebiet]); // error 032
haus[i][pickup] = CreatePickup(1273, 1, haus[i][X], haus[i][Y], haus[i][Z], -1); // error 032
Create3DTextLabel(text, cDONE, haus[i][X], haus[i][Y], haus[i][Z]+1, 5, 0, 0); // error 032
}
}
}
mysql_free_result();
}
return 1;
}[/pwn]
Ich hoffe mir kann jemand helfen.

Mit freundl. Grüßen
Padarom

Vincent Petritz · 29. April 2012, 17:40

Kannst du uns bitte noch sagen wo konkret nun die Errorzeile ist dann können wir daraus Schlüsse ziehen.

Padarom · 29. April 2012, 17:44

Ist oben editiert.

Vincent Petritz · 29. April 2012, 17:46

Ich kenne das.
Meine Lösung ist nicht die einfachste, dennoch funktioniert sie:
[pwn]new str1[5],
schleife;
while(schleife < MAX_HAUS) {
format(str1, sizeof(str1), "%d", schleife);
haus[X] = mysql_GetFloat("haus", "X", "ID", str1);
haus[i][Y] = mysql_GetFloat("haus", "Y", "ID", str1);
haus[i][Z] = mysql_GetFloat("haus", "Z", "ID", str1);
//...
}[/pwn]

Dann bei "..." eben noch den Rest einsetzen und anpassen - klappt eigtl. ohne Probleme! ;-)

Padarom · 29. April 2012, 17:50

So bleiben die Fehler leider trotzdem bestehen, es ändert sich nichts

Bubelbub · 29. April 2012, 18:53

[pwn]format(mQuery, 128, "SELECT * FROM haus WHERE ID = %i LIMIT 1;", i);[/pwn]

Wüsste sonst nicht, wieso das kommt.

Padarom · 29. April 2012, 18:58

Klappt leider immer noch nicht.
Würde euch vielleicht eine andere Möglichkeit einfallen wie man das durchführen kann?

Mit freundl. Grüßen
Padarom

Bubelbub · 29. April 2012, 19:00

Padarom schrieb:

Hallo,
ich habe gerade einige Fehler, die ich beim besten Willen nicht wegbekomme:

Quellcode

error 032: array index out of bounds (variable "haus") // 13x - Überall wo haus[i][...] verwendet wird

Hier der relevante Teil des Codes. Hauptsächlich sind die Fehler im Callback LoadHaus.
[pwn]#define MAX_HAUS 350

enum hInfo {
Float:X,
Float:Y,
Float:Z,
Preis,
Gebiet[5],
Besitzer[24],
KeyID,
pickup,
bool:loaded = false
};

new haus[MAX_HAUS][hInfo];

public LoadHaus()
{
new text[128];
for(new i=0; i<MAX_HAUS; i++)
{
if(haus[loaded] == false)
{
format(mQuery, 128, "SELECT * FROM haus WHERE ID = %i", i);
mysql_query(mQuery);
mysql_store_result();
if(mysql_num_rows())
{
mysql_free_result();
if(i > MAX_HAUS) continue;
haus[i][X] = mysql_GetFloat("haus", "X", "ID", i);
haus[i][Y] = mysql_GetFloat("haus", "Y", "ID", i); // error 032
haus[i][Z] = mysql_GetFloat("haus", "Z", "ID", i); // error 032
haus[i][Besitzer] = mysql_GetString("haus", "Besitzer", "ID", i); // error 032
haus[i][Gebiet] = mysql_GetString("haus", "Gebiet", "ID", i); // error 032
haus[i][KeyID] = mysql_GetInt("haus", "KeyID", "ID", i); // error 032
haus[i][Preis] = mysql_GetInt("haus", "Preis", "ID", i); // error 032
haus[i][loaded] = true;
if(strlen(haus[i][Besitzer]) > 2) // error 032
{ // Bewohnt
format(text, 128, "- Besitzer: %s -\n- Gebiet: %s -", haus[i][Besitzer], haus[i][Gebiet]); // error 032
haus[i][pickup] = CreatePickup(1272, 1, haus[i][X], haus[i][Y], haus[i][Z], -1); // error 032
Create3DTextLabel(text, cAQUA, haus[i][X], haus[i][Y], haus[i][Z]+1, 5, 0, 0); // error 032
}
else
{ // Verkäuflich
format(text, 128, "- Verkäuflich -\n- Preis: $%i -\n- Gebiet: %s -", haus[i][Preis], haus[i][Gebiet]); // error 032
haus[i][pickup] = CreatePickup(1273, 1, haus[i][X], haus[i][Y], haus[i][Z], -1); // error 032
Create3DTextLabel(text, cDONE, haus[i][X], haus[i][Y], haus[i][Z]+1, 5, 0, 0); // error 032
}
}
}
mysql_free_result();
}
return 1;
}[/pwn]
Ich hoffe mir kann jemand helfen.

Mit freundl. Grüßen
Padarom

so?

Padarom · 29. April 2012, 19:07

Klappt nicht, ist aber eigentlich auch unnötig, da die Schleife ja garnicht über MAX_HAUS hinauskommt. Folglich müsste man den Fall, dass i > MAX_HAUS ist auch nicht abfragen.

Vincent Petritz · 29. April 2012, 19:11

[pwn]#define MAX_HAUS 350

enum hInfo {
Float:X,
Float:Y,
Float:Z,
Preis,
Gebiet[5],
Besitzer[24],
KeyID,
pickup,
bool:loaded = false
};

new haus[MAX_HAUS][hInfo];

stock LoadHaus()
{
new str[10];
for(new var; var < MAX_HAUS; var++) {
format(str, sizeof(str), "%d", var);
haus[var][X] = mysql_GetFloat("haus", "X", "ID", str);
haus[var][Y] = mysql_GetFloat("haus", "Y", "ID", str); // error 032
haus[var][Z] = mysql_GetFloat("haus", "Z", "ID", str); // error 032
haus[var][Besitzer] = mysql_GetString("haus", "Besitzer", "ID", str); // error 032
haus[var][Gebiet] = mysql_GetString("haus", "Gebiet", "ID", str); // error 032
haus[var][KeyID] = mysql_GetInt("haus", "KeyID", "ID", str); // error 032
haus[var][Preis] = mysql_GetInt("haus", "Preis", "ID", str); // error 032
haus[var][loaded] = true;
if(strlen(haus[Besitzer]) > 2) // error 032
{ // Bewohnt
format(text, 128, "- Besitzer: %s -\n- Gebiet: %s -", haus[i][Besitzer], haus[i][Gebiet]); // error 032
haus[i][pickup] = CreatePickup(1272, 1, haus[i][X], haus[i][Y], haus[i][Z], -1); // error 032
Create3DTextLabel(text, cAQUA, haus[i][X], haus[i][Y], haus[i][Z]+1, 5, 0, 0); // error 032
}
else
{ // Verkäuflich
format(text, 128, "- Verkäuflich -\n- Preis: $%i -\n- Gebiet: %s -", haus[i][Preis], haus[i][Gebiet]); // error 032
haus[i][pickup] = CreatePickup(1273, 1, haus[i][X], haus[i][Y], haus[i][Z], -1); // error 032
Create3DTextLabel(text, cDONE, haus[i][X], haus[i][Y], haus[i][Z]+1, 5, 0, 0); // error 032
}
}
return 1;
}[/pwn]

Padarom · 29. April 2012, 19:16

Danke für die Zeit die ihr aufbringt, aber es geht leider immernoch nicht

Ich verzweifel langsam. Ich habe es mal wie folgt getestet:
[pwn]new Float:hX[MAX_HAUS], Float:hY[MAX_HAUS], Float:hZ[MAX_HAUS];[/pwn]
So klappt es, allerdings ist das ziemlich unordentlich bei ca. 10 verschiedenen Variablen und auch später mit dem Abfragen.

Vincent Petritz · 29. April 2012, 19:17

Ich weiß nicht recht, ich würde fast behaupten dann hat dein Script 'ne Makke, ohne dir was unterstellen zu wollen - aber das muss so eigtl. gehen.
Kurze Frage: Du nutzt doch G-Stylezzz Plugin oder?

Padarom · 29. April 2012, 19:20

Genau, die R5 aber, da die R7 irgendwie nicht läuft (Komische Errors z.B. dass mysql_query nicht definiert wäre).
Ich wüsste aber auch nicht wo großartige Fehler in meinem Script sein sollten. Ohne den LoadHaus-Part gibt es keine Fehler, und ansonsten hat das Script auch nur 450 Zeilen bisher.

Vincent Petritz · 29. April 2012, 19:25

[pwn]haus[var][X] = mysql_GetFloat("haus", "X", "ID", strval(str));[/pwn]
Was sagt der Compiler dazu?

Padarom · 29. April 2012, 19:36

Erstaunlicherweise garnichts. Das geht erst ab dem zweiten los. Kann es vielleicht sein, dass etwas am mysql_GetFloat nicht stimmt, sodass ab dem zweiten Mal ein Fehler stattfindet?
[pwn]stock Float:mySQL_GetFloat(table[], row[], whereIs[], what[])
{
new query[128], Float:Float;
mysql_real_escape_string(whereIs, whereIs);
mysql_real_escape_string(what, what);
format(query, 128, "SELECT `%s` FROM `%s` WHERE `%s` = '%s'", row, table, whereIs, what);
mysql_query(query);
mysql_store_result();
mysql_fetch_float(Float);
mysql_free_result();
return Float;
}[/pwn]

Vincent Petritz · 29. April 2012, 19:40

Sofern es das von Maddin's Tutorial ist geht es.
Nimm die stock's von Maddin's TuT, die gehen prima!

Padarom · 29. April 2012, 19:42

Das ist der von Maddin's Tutorial

Ich habe eben testweise woanders im Script Werte in die Variable zugewiesen. Ebenfalls Fehler:
[pwn] haus[3][X] = 0.57636;
haus[3][Y] = 0.57636; // Out of bounds
haus[3][Z] = 0.57636; // Out of bounds[/pwn]
Vielleicht stimmt also etwas mit der Enum(eration) nicht.

Vincent Petritz · 29. April 2012, 19:44

Das stimmt meines Erachtens soweit! :-o

Padarom · 29. April 2012, 19:44

Wooaahhh... Ich habs!
Dadurch dass ich den Wert der bool (loaded) zugewiesen habe, hat er erwartet, dass ich jedem anderen auch schon einen Standardwert zuweise. Da Bool's standardweise false sind, ist die Zuweisung so oder so schwachsinn. Diese also rausgemacht und bumm... Es klappt!
Dennoch danke!

edit: Ich meine, dass ich so einen Fehler schonmal hatte, als das = false alles durcheinandergeschmissen hatte, konnte mich daran aber eben nicht mehr erinnern.

Vincent Petritz · 29. April 2012, 19:45

Mh, hätt' ich auch drauf kommen können - viel Spaß weiterhin beim scripten, sorry das ich nicht drauf kam! ;-s

Quellcode

Quellcode

Teilen