Fehler bei multidimensionalen Array

Drasko · 22. Juni 2013

Hallo.
Undzwar hab' ich das Problem, dass mein multidimensionales Array nicht will.

Oben im Script definiere ich es so:

new TicketID = 1;
new TicketText [][128];

Und bei meinem Support Befehl mach ich folgendes:

ocmd:support(playerid, params[]) {
new text[128];
if(pData[playerid][pEingeloggt] == 0) return SendClientMessage(playerid, COLOR_ERROR, "[Fehler]{FFFFFF} Du musst eingeloggt sein, um Befehle auszuführen!");
if(sscanf(params, "s[128]", text)) return SendClientMessage(playerid, COLOR_USING, "[Benutzung]{FFFFFF} /support [Text]");
if(pData[playerid][pTicket] == 1) {
SendClientMessage(playerid, COLOR_SUPPORT, "[Support]{FFFFFF} Du hast dein Supportticket gelöscht!");
TicketID = TicketID-1;
pData[playerid][pTicket] = 0;
} else {
new msg[64], msg2[160];

TicketText[TicketID][128] = { text };

pData[playerid][pTicketID] = TicketID;
format(msg, sizeof(msg), "[Support]{FFFFFF} Du hast ein Supportticket eröffnet (ID: %d)", TicketID);
format(msg2, sizeof(msg2), "[Support]{FFFFFF} %s", text);
SendClientMessage(playerid, COLOR_SUPPORT, msg);
SendClientMessage(playerid, COLOR_SUPPORT, "[Support]{FFFFFF} Deine Frage:");
SendClientMessage(playerid, COLOR_SUPPORT, msg2);
TicketID++;
}
return 1;
}

Error ist folgender:

Code

C:\Users\Timo\Desktop\Pawn\gammodes\blueberry.pwn(316) : error 032: array index out of bounds (variable "TicketText")
C:\Users\Timo\Desktop\Pawn\gammodes\blueberry.pwn(316) : error 006: must be assigned to an array
Pawn compiler 3.2.3664	 	 	Copyright (c) 1997-2006, ITB CompuPhase


2 Errors.

Ich bitte um Hilfe
LG
Maydo

**theTV** · 22. Juni 2013

Welche Zeile genau? Denn 316 gibts ja nicht.

Drasko · 22. Juni 2013

Achso, fail:

ocmd:support(playerid, params[]) {
new text[128];
if(pData[playerid][pEingeloggt] == 0) return SendClientMessage(playerid, COLOR_ERROR, "[Fehler]{FFFFFF} Du musst eingeloggt sein, um Befehle auszuführen!");
if(sscanf(params, "s[128]", text)) return SendClientMessage(playerid, COLOR_USING, "[Benutzung]{FFFFFF} /support [Text]");
if(pData[playerid][pTicket] == 1) {
SendClientMessage(playerid, COLOR_SUPPORT, "[Support]{FFFFFF} Du hast dein Supportticket gelöscht!");
TicketID = TicketID-1;
pData[playerid][pTicket] = 0;
} else {
new msg[64], msg2[160];

TicketText[TicketID][128] = { text }; // <--- Das da

pData[playerid][pTicketID] = TicketID;
format(msg, sizeof(msg), "[Support]{FFFFFF} Du hast ein Supportticket eröffnet (ID: %d)", TicketID);
format(msg2, sizeof(msg2), "[Support]{FFFFFF} %s", text);
SendClientMessage(playerid, COLOR_SUPPORT, msg);
SendClientMessage(playerid, COLOR_SUPPORT, "[Support]{FFFFFF} Deine Frage:");
SendClientMessage(playerid, COLOR_SUPPORT, msg2);
TicketID++;
}
return 1;
}

[EGC] selEcT · 22. Juni 2013

So vielleicht?

TicketText[TicketID] = text;

oder halt

strmid(TicketText[TicketID], text, 0, strlen(text),255);

Drasko · 22. Juni 2013

Zitat

TicketText[TicketID] = text;

Errors habe ich so keine. Allerdings sendet er trotzdem nicht mein erstelltes Ticket:

Code

[18:05:27] [Benutzung]{FFFFFF} /support [Text]
[18:05:36] [Support]{FFFFFF} Du hast ein Supportticket eröffnet (ID: 1)
[18:05:36] [Support]{FFFFFF} Deine Frage:
[18:05:36] [Support]{FFFFFF} Ich brauche Hilfe, weil ich dumm bin
[18:05:44] |------ Offene Tickets ------|
[18:05:51] [AdminCMD] Ryan hat den Dienst als Administrator begonnen!
[18:05:53] |------ Offene Tickets ------|

ocmd:support(playerid, params[]) {
new text[128];
if(pData[playerid][pEingeloggt] == 0) return SendClientMessage(playerid, COLOR_ERROR, "[Fehler]{FFFFFF} Du musst eingeloggt sein, um Befehle auszuführen!");
if(sscanf(params, "s[128]", text)) return SendClientMessage(playerid, COLOR_USING, "[Benutzung]{FFFFFF} /support [Text]");
if(pData[playerid][pTicket] == 1) {
SendClientMessage(playerid, COLOR_SUPPORT, "[Support]{FFFFFF} Du hast dein Supportticket gelöscht!");
pData[playerid][pTicket] = 0;
} else {
new msg[64], msg2[160];

TicketText[TicketID] = text;

pData[playerid][pTicketID] = TicketID;
format(msg, sizeof(msg), "[Support]{FFFFFF} Du hast ein Supportticket eröffnet (ID: %d)", TicketID);
format(msg2, sizeof(msg2), "[Support]{FFFFFF} %s", text);
SendClientMessage(playerid, COLOR_SUPPORT, msg);
SendClientMessage(playerid, COLOR_SUPPORT, "[Support]{FFFFFF} Deine Frage:");
SendClientMessage(playerid, COLOR_SUPPORT, msg2);
TicketID++;
}
return 1;
}

ocmd:tickets(playerid, params[]) {
if(pData[playerid][pAdmin] > 0){
SendClientMessage(playerid, COLOR_SUPPORT, "|------ Offene Tickets ------|");
for(new i;i<MAX_PLAYERS;i++){
if(pData[i][pTicket] == 1) {
new zeile[128], status[24];

switch(pData[i][pTicketOffen])
{
case 0: status = "Wird bearbeitet";
case 1: status = "Offen";
}
format(zeile, sizeof(zeile), "(%d) %s (%s), Frage: %s", pData[i][pTicketID], GetName(i), status, TicketText[pData[i][pTicketID]]);
SendClientMessage(playerid, 0xFFFFFFFF, zeile);
}
}
return 1;
} else {
return SendClientMessage(playerid, COLOR_ERROR, "[Fehler]{FFFFFF} Du hast nicht die nötigen Berechtigungen dazu!");
}
}

maddin · 22. Juni 2013

new TicketText [][128];Seit wann kann man arrays so deklarieren?

Die Längenangabe kann nur dann weg gelassen werden, wenn du der variable direkt beim erstellen einen wert zuweist. Bspw.
new foo[] = "bar";

Versuch mal die Größe anzugeben, bspw per MAX_PLAYERS (da du ja offensichtlich für jeden Spieler eine Dimension brauchst).

Zu Zeile 316 siehe selEcT's 2. Vorschlag. Wobei die längenangaben da auch nicht ganz richtig sind:
strmid(TicketText[TicketID], text, 0, 128);
Bitte bedenke, das man Strings nicht per = kopieren sollte.

Drasko · 22. Juni 2013

Mir wird weiterhin bei /tickets nix angezeigt.
Woran kann das liegen? (Code steht 2 Beiträge über mir)

//EDIT:

ocmd:support(playerid, params[]) {
new text[128];
if(pData[playerid][pEingeloggt] == 0) return SendClientMessage(playerid, COLOR_ERROR, "[Fehler]{FFFFFF} Du musst eingeloggt sein, um Befehle auszuführen!");
if(sscanf(params, "s[128]", text)) return SendClientMessage(playerid, COLOR_USING, "[Benutzung]{FFFFFF} /support [Text]");
if(pData[playerid][pTicket] == 1) {
SendClientMessage(playerid, COLOR_SUPPORT, "[Support]{FFFFFF} Du hast dein Supportticket gelöscht!");
pData[playerid][pTicket] = 0;
} else {
new msg[64], msg2[160];

strmid(TicketText[TicketID], text, 0, 128);

pData[playerid][pTicketID] = TicketID;
format(msg, sizeof(msg), "[Support]{FFFFFF} Du hast ein Supportticket eröffnet (ID: %d). Deine Frage:", TicketID);
format(msg2, sizeof(msg2), "[Support]{FFFFFF} %s", text);
SendClientMessage(playerid, COLOR_SUPPORT, msg);
SendClientMessage(playerid, COLOR_SUPPORT, msg2);

TicketID++;
}
return 1;
}

ocmd:tickets(playerid, params[]) {
if(pData[playerid][pAdmin] > 0){
SendClientMessage(playerid, COLOR_SUPPORT, "|------ Offene Tickets ------|");
for(new i;i<MAX_PLAYERS;i++){
if(pData[i][pTicket] == 1) {
new zeile[128], status[24];

switch(pData[i][pTicketOffen])
{
case 0: status = "Wird bearbeitet";
case 1: status = "Offen";
}
format(zeile, sizeof(zeile), "(%d) %s (%s), Frage: %s", pData[i][pTicketID], GetName(i), status, TicketText[pData[i][pTicketID]]);
SendClientMessage(playerid, 0xFFFFFFFF, zeile);
}
}
return 1;
} else {
return SendClientMessage(playerid, COLOR_ERROR, "[Fehler]{FFFFFF} Du hast nicht die nötigen Berechtigungen dazu!");
}
}

Keine Errors

**theTV** · 22. Juni 2013

bei ocmd:tickets fragst du ab:
if(pData[i][pTicket] == 1)

aber du setzt sie bei ocmd:support nicht auf 1

maddin · 22. Juni 2013

Ich glaube du hast da einen kleinen Denkfehler drin.
Die Maximalgröße von Arrays in SA-MP ist 1024 (jedenfalls laut Wiki). Das würde bedeuten, das selbst wenn du das Array so groß machst, du nicht mehr als 1024 Tickets verarbeiten kannst bevor du theoretisch deinen Server neustarten müsstest.
Da du ja (das entnehmen ich jedenfalls so deinem Code) jeden Spieler nur ein Ticket auf einmal zur Verfügung stellen willst, mach das ganze doch einfach auch per Enum. Da du ja sowieso schon pData[i][pTicket] zum abfragen benutzt, kannst du das gleich alles damit machen.

Wenn du pTicket als string mit einer länge von 128 deklarierst, kannst du damit gleich alles machen: Abfragen ob ein Ticket existiert und ein Ticket drin speichern.

Abfragen:if( strlen(pData[playerid][pTicket]) == 0 )
{
//Der spieler hat kein Ticket. Kann also ein neues erstellen.
}
else
{
//Der Spieler hat bereits ein ticket.
}
Zuweisen:strmid(pData[playerid][pTicket], text, 0, 128);
Löschen:strdel(pData[playerid][pTicket], 0, 128);

So würde ich es jedenfalls machen.

Warum dein Code nicht geht kann mehrere Ursachen haben.
Du weist pData[i][pTicketOffen] beispielsweise nie einen wert zu, du kopierst strings mit = , und du solltest definitiv noch eine Abfrage einbauen ob der Spieler überhaupt Online ist.

IPrototypeI · 22. Juni 2013

Zitat von maddin

Ich glaube du hast da einen kleinen Denkfehler drin.
Die Maximalgröße von Arrays in SA-MP ist 1024 (jedenfalls laut Wiki). Das würde bedeuten, das selbst wenn du das Array so groß machst, du nicht mehr als 1024 Tickets verarbeiten kannst bevor du theoretisch deinen Server neustarten müsstest.

Die Aussage stimmt nicht du kannst den string so groß machen wie du willst musst jedoch auf die zugesicherten memory für stack und heap achten. Soweit wie mir mal gesagt wurde nimmt sich pawno auch trotz #pragma dynamic den memory was es braucht an memory.

Ich denk mal eher das die Leute nicht in der Lage sind bei String Length auf die Überschrift zu achten und da steht textdraws das bedeutet im zusammenhang es ist nur möglich 1024 Zeichen
auf einem textdraw anzuzeigen. Daher hat das nix mit deklarierten arrays zu tun man sollte nur aufpassen bei string-funktionen wie format was bei 4028 Zeichen crashen soll nach double0seven.

Drasko · 22. Juni 2013

Zitat von mastho

bei ocmd:tickets fragst du ab:
if(pData[pTicket] == 1)

aber du setzt sie bei ocmd:support nicht auf 1

Zitat von maddin

Ich glaube du hast da einen kleinen Denkfehler drin.
Die Maximalgröße von Arrays in SA-MP ist 1024 (jedenfalls laut Wiki). Das würde bedeuten, das selbst wenn du das Array so groß machst, du nicht mehr als 1024 Tickets verarbeiten kannst bevor du theoretisch deinen Server neustarten müsstest.
Da du ja (das entnehmen ich jedenfalls so deinem Code) jeden Spieler nur ein Ticket auf einmal zur Verfügung stellen willst, mach das ganze doch einfach auch per Enum. Da du ja sowieso schon pData[i][pTicket] zum abfragen benutzt, kannst du das gleich alles damit machen.

Wenn du pTicket als string mit einer länge von 128 deklarierst, kannst du damit gleich alles machen: Abfragen ob ein Ticket existiert und ein Ticket drin speichern.

Abfragen:if( strlen(pData[playerid][pTicket]) == 0 )
{
//Der spieler hat kein Ticket. Kann also ein neues erstellen.
}
else
{
//Der Spieler hat bereits ein ticket.
}
Zuweisen:strmid(pData[playerid][pTicket], text, 0, 128);
Löschen:strdel(pData[playerid][pTicket], 0, 128);

So würde ich es jedenfalls machen.

Warum dein Code nicht geht kann mehrere Ursachen haben.
Du weist pData[i][pTicketOffen] beispielsweise nie einen wert zu, du kopierst strings mit = , und du solltest definitiv noch eine Abfrage einbauen ob der Spieler überhaupt Online ist.

Alles anzeigen

Danke euch Beiden.
Grade auch dir maddin. Ich habe mit der Zeit den Überblick verloren und hab es mir schwieriger vorgestellt, als es ist. Eine Frage hätte ich noch:

Warum sollte man Variablen mit "strmid"und nicht einfach per "=" Operator zuweisen?

breadfish.de 12. März 2022