[Error] Memory could not be read

[Doktor.]

Hey, hab jetzt schon alle Möglichkeiten aus Google versucht, aber nichts hat geholfen.
Hoffe ihr könnt weiterhelfen, der Code:

Object *object1;
Object *object2;
Object *object3;
Object *object4;

int main()
{

while(1)
{
//Hier kommt es irgendwann zum Memory Fehler
object1 = NTMap->getAll(hprog, mobs);
object2 = NTMap->getAll(hprog, players);
object3 = NTMap->getAll(hprog, items);
object4 = NTMap->getAll(hprog, npcs);

//Augabe.....{}
//
//
//


delete[] object1 ;
delete[] object2;
delete[] object3;
delete[] object4 ;
}
return 0;
}

Das ganze Programm läuft einwandfrei, außer eben das es nach einer bestimmten Zeit crasht und der Fehler kommt "Memory could not be read".

Weis zufällig jemand wie man diesen Fehler umgehen kann, bzw. wodurch der ausgelöst wird?

[Schlüsselbein]

Was sagt dee Debugger? Was machen die Funktionen etc.?

[Doktor.]

Der Debugger sagt nichts dazu, das Programm startet ganz normal.

object1 = NTMap->getAll(hprog, mobs);
object2.....
enthalten danach die Mobs etc. Namen auf der momentanen Map von einem Game, die dann ausgegeben werden.

[MrSm!th]

Jemand sollte ein Tutorial für ordentliches Debuggen veröffentlichen..

[Doktor.]

Dann mach du doch so'n Tutorial, anstatt Posts zu posten die einem nicht mal wirklich weiter helfen.

[Padmak]

Deine Fehler helfen aber auch nicht wirklich weiter, du hast eigentlich keinen Fehlerrelevaten Code gepostet. Gibt NTMap::getAll immer valide werte zurück oder können die auch mal 0 sein?
Ich weiß nicht was du für Antworten haben willst, aber das ist nunmal Sache des Debuggens

Übrigens scheint der Code nicht allzu hübsch zu sein, solltest dir vllt mal Gedanken machen

Padmak

[Doktor.]

Ja, ist eine Library, enthält alle nötigen Funktion und klassen und der Wert kann (bin mir grad nicht ganz sicher, muss es noch testen) 0 sein, wenn ich grad die Map wechsel.

Das der Code nicht so ganz schön ist, ist ja auch klar, da dies oben nicht der richtige Code ist, sondern nur der Ausschnitt wo der Absturz stattfindet.

[Delinquenz]

Das der Code nicht so ganz schön ist, ist ja auch klar, da dies oben nicht der richtige Code ist, sondern nur der Ausschnitt wo der Absturz stattfindet.

Sofern du uns nicht die "Object" Klasse und die Klasse der Variable NTMap zeigst, können und werden wir dir nicht helfen.

Wir sind keine Hellseher.

[Padmak]

Originally Posted by ~Wystan

und der Wert kann [...] 0 sein, wenn ich grad die Map wechsel.

Teste jeden Pointer vorm delete, ob der auch wirklich nicht null ist. Außerdem: sicher, dass es ein Array ist? Wenn's nur ein Objekt ist, das per new alloziiert wurde: delete statt delete[]

Padmak

[Nightblizard]

Originally Posted by Padmak

Teste jeden Pointer vorm delete, ob der auch wirklich nicht null ist.

Das ist unsinn. delete nullptr; ist wohldefiniert und verursacht keine Probleme.
Siehe:


@TE:
Du solltest dir RAII zu Gemüte führen (bzw. die Finger gleich von Pointer lassen), dann passieren solche Zugriffsfehler nicht.

[Tasiro]

Siehe [new.delete.single]/14:

Default behavior: If ptr is null, does nothing. Otherwise, reclaims the storage allocated by the earlier call to operator new.

[MrSm!th]

Das Problem ist eher, einen bereits gelöschten Pointer (oder einen anderweitig ungültigen Adresswert) zu löschen.

[Nightblizard]

Was auch ganz interessant ist, ist dass er die einzelnen Objekte mit delete[] löscht. Aufgrund der Benennung schaut das ersteinmal falsch aus, aber er kann auch einfach nur mit den Namen geschlampt haben (das getAll lässt dann nämlich doch eher auf ein Array schließen..).
Mehr spuckt die Glaskugel im Moment nicht aus, muss sie wohl mal wieder neu aufladen. Alternativ solltest du uns mehr Code zeigen, bzw. wie die anderen sagten mit nem Debugger durch das Programm laufen.