Ganz leicht erklärlicher Fehler in C++ [solved]

Werbeanzeige

ERROR

Alter Hase

Beiträge: 417

Wohnort: Paderborn

Beruf: Informatik Student

17.11.2013, 23:01

Ganz leicht erklärlicher Fehler in C++ [solved]

Nabend,

Ich führe folgenden code aus:

C-/C++-Quelltext

void AMap::Init(Map &map)
{
    //Aussenrum alles blocken
    for(int i = 0; i < GetWidth();i++)
    {
        atiles[i][0] = ATile(blocked);
        atiles[i][GetHeight()] = ATile(blocked);
    }
    for(int i = 0; i < GetHeight();i++)
    {
        atiles[0][i] = ATile(blocked);
        atiles[GetWidth()][i] = ATile(blocked);
    }
    //normale Map in AStar Map übertragen
    for(int x = 0; x < GetWidth();x++)
    {
        for(int y  = 0; y < GetHeight();y++)
        {
            atiles[x+1][y+1] = ATile(map.tiles[x][y].GetProperty());
        }
    }
}

Der Konstruktor der für die ATiles aufgerufen wird.

	C-/C++-Quelltext
1 2 3 4 5	ATile::ATile(TileProperty p) { prop = p; list = No; }

Diese Funktion erstellt eine Map für den A*. Dafür wird die normale Map (Map &map) benutzt. Diese wird nur zum Auslesen benutzt, damit in der Pathfinding Map nicht begehbare Tiles verzeichnet sind usw.

Nun ist das problem, dass nach dem letzten Durchlauf der ersten Schleife die Variable "Nullpunkt" und nach dem letzten Durchlauf der zweiten Schleife das Tile[0][0] verändert werden.

Das Bild "Debug" zeigt diese Veränderungen nochmal im Debuggen.

Das Bild "error" zeigt die Fehlermeldung, die dann später dadurch ausgeworfen wird. (Bei späterem Zugriff auf die Tiles)

Nun ist einfach das Problem, dass ich diese Variablen niemals verändern möchte und eigentlich auch nie verändere, aber sie werden geändert und ich weiss nicht wieso.

Ich hoffe mir kann da Jemand weiterhelfen.

»ERROR« hat folgende Bilder angehängt:

Dieser Beitrag wurde bereits 1 mal editiert, zuletzt von »ERROR« (17.11.2013, 23:37)

Zum Seitenanfang

David Scherfgen

Administrator

Beiträge: 10 382

Wohnort: Hildesheim

Beruf: Wissenschaftlicher Mitarbeiter

17.11.2013, 23:12

	C-/C++-Quelltext
1 2	atiles[i][GetHeight()] = ATile(blocked); atiles[GetWidth()][i] = ATile(blocked);

Da sind deine Fehler.
Mehr sage ich nicht (und ich bitte auch sonst niemanden dies zu tun, sonst geht der Lerneffekt verloren).

David Scherfgens Website | Konzentrationstest Polizei

Zum Seitenanfang

ERROR

Alter Hase

Beiträge: 417

Wohnort: Paderborn

Beruf: Informatik Student

17.11.2013, 23:19

Also ehrlich gesagt erkenne ich da jetzt keinen Fehler.

Der Sinn dieser beiden Schleifen ist es ja, Alle äusseren Tiles als unbegehbar (blocked) zu markieren.

Da ist doch nirgendwo ein zugriff auf das Übergebene Objekt &map oder bin ich grade total blind?

Zum Seitenanfang

David Scherfgen

Administrator

Beiträge: 10 382

Wohnort: Hildesheim

Beruf: Wissenschaftlicher Mitarbeiter

17.11.2013, 23:26

Zitat von »"ERROR"«

Also ehrlich gesagt erkenne ich da jetzt keinen Fehler.

Ein Array[N] geht von 0 bis ...?

Zitat von »"ERROR"«

Da ist doch nirgendwo ein zugriff auf das Übergebene Objekt &map oder bin ich grade total blind?

Nicht direkt. Aber im Speicher liegt das alles hintereinander. Wenn du also auf eine Variable über die Grenzen hinaus zugreifst, kommst du plötzlich in die im Speicher benachbarte Variable hinein.

Genug Hinweise?

David Scherfgens Website | Konzentrationstest Polizei

Zum Seitenanfang

ERROR

Alter Hase

Beiträge: 417

Wohnort: Paderborn

Beruf: Informatik Student

17.11.2013, 23:32

Och ne oder?!

Zitat

Ein Array[N] geht von 0 bis ...?

N-1 natürlich. :dash:

Dumme Anfänger Fehler :fie:

Zitat

Aber im Speicher liegt das alles hintereinander

Also das ist ja echt dumm von mir gewesen

Ich muss ja ehrlich sein und sagen ich sass da grade über 60 Minuten dran und habe das übersehen

Danke dir David

Jetzt kann ich beruhigt schlafen

Zum Seitenanfang

dot

Supermoderator

Beiträge: 9 757

Wohnort: Graz

17.11.2013, 23:37

Happens...ich hab mir an einem ähnlich simplen Fehler mal ein Jahr lang die Zähne ausgebissen...dafür wirst du es nun garantiert nie wieder falsch machen...