ich komme aus dem Java-Lager und bin nicht so mit der Speicherverwaltung in c++ so vertraut.

In einer Methode lege ich 3 temporäre Vektoren an:

vector<Icon*> iconsInFrontLayer;
vector<Icon*> iconsInMiddleLayer;
vector<Icon*> iconsInBackLayer;

Muss ich am Ende dieser Methode die Vektoren wieder löschen, da sie danach nicht mehr gebraucht werden? Wenn ja, wie mache ich das?

Grüße
Alex

allerdings müssten evtl. die Icon Objekte auf die pointer (Icon*) verweisen gelöscht werden.

allerdings müssten evtl. die Icon Objekte auf die pointer (Icon*) verweisen gelöscht werden.

Meinst Du damit die Objekte in den Vektoren? Wenn ja, die werden noch gebraucht und sollten nicht gelöscht werden

Dein, musst Du nicht. Vektoren rufen beim Verlassen des Scopes in dem sie sich befinden automatisch die Destruktoren der gespeicherten Objekte auf.

Also sollten meine Icon-Objekte sich im eignen Destructor löschen?

Vielen Dank!

allerdings müssten evtl. die Icon Objekte auf die pointer (Icon*) verweisen gelöscht werden.

Meinst Du damit die Objekte in den Vektoren? Wenn ja, die werden noch gebraucht und sollten nicht gelöscht werden

Dein, musst Du nicht. Vektoren rufen beim Verlassen des Scopes in dem sie sich befinden automatisch die Destruktoren der gespeicherten Objekte auf.

Also sollten meine Icon-Objekte sich im eignen Destructor löschen?

Ganz einfache Regel:
Alles, was Du mit new angelegt hast, musst Du auch mit delete wieder freigeben.
Du speicherst keine Icon-Objekte im Vektor, sondern Zeiger auf Iconobjekte. Also werden auch nie die Icon-Objekte zerstört, sondern nur die Zeiger. Wenn die Icons mit new angelegt wurden, und die Zeiger darauf zerstört wurden, bevor Du die referenzierten Objekte mit delete wieder freiegegben hast, hast Du ein Speicherleck.

Der destructor sieht momentan so aus:

// Free memory when closing
Icon::~Icon() {
	pTextureLoader->FreeTexture(&m_MyWallTexture);
}

Was passiert mit dem Speicher, wenn ich die Anwendung schließe und die Objekte nicht gelöscht habe? Bleibt der Speicher belegt?

Die Icon Objekte müssen beim Verlassen der Anwendung gelöscht werden.

Der destructor sieht momentan so aus:

// Free memory when closing
Icon::~Icon() {
	pTextureLoader->FreeTexture(&m_MyWallTexture);
}

Was passiert mit dem Speicher, wenn ich die Anwendung schließe und die Objekte nicht gelöscht habe? Bleibt der Speicher belegt?

Das hängt von der Plattform ab. Bei Windows werden Image/Icon-Handles nicht wieder freigegeben.
Man sollte grundsätzlich angeforderte Ressourcen immer frei geben. Auch, wenn man weiss, dass einem das Betriebssystem hinterher räumt (z.B. bei Speicher).
Wenn Du die Icon-Objekte mit new angelegt hast, und kein delete machst, wird der Icon-Destructor nie aufgerufen!

Wie wird denn Dein Vektor befüllt?

Wie wird denn Dein Vektor befüllt?

Das mache ich so:

iconPointer = new Icon("DIRECTORY", _x, _y, _z, wfd.cFileName, objectIndex);
iconVector.push_back(iconPointer);

Ok verstehe! Die Plattform ist in meinem Fall Windows.

Wie wird denn Dein Vektor befüllt?

Das mache ich so:

iconPointer = new Icon("DIRECTORY", _x, _y, _z, wfd.cFileName, objectIndex);
iconVector.push_back(iconPointer);

Dann ist doch eigentlich klar, was zu tun ist:

tachyon schrieb:

Alles, was Du mit new angelegt hast, musst Du auch mit delete wieder freigeben.

Ich bin etwas verwirrt.

Meine Struktur ist folgendermaßen:

in der main erzeuge ich einen

IconGetter myIcons;

der wiederum verwaltet diesen Vektor, von dem Elemente vom Typ Icon (eigene) Klasse drin stehen.

Muss ich dann in der "exit" - Methode der main myIcons deleten oder die einzelnen Icon-Objekte im Vektor?

...

Alles, was Du mit new angelegt hast, musst Du auch mit delete wieder freigeben.

Was ist daran so schwer zu verstehen?

IconGetter myIcons; //wurde NICHT mit new angelegt, also auch kein delete

Wenn du dein Objekt iconVector in einer Klasse hast, dann musst du dafür sorgen, dass, wenn das Objekt gelöscht wird auch wieder aller Speicher, den du in iconVector gesammelt hast wieder freigibst. Und das kannst du genau im Destruktor machen, dem egal ist, wie das Objekt zerstört wird (exception, am Ender der main, oder irgendwann mit einem delete). Der Destruktor wird immer aufgerufen, also musst du genau dort alle Elemente, die du per new angefordert hast und in iconVector gespeichert hast auch wieder mit delete freigeben. (ich sage micht Absicht, dass du die mit new angelegt haben mussts, da du auch ohne weiteres ein automatisches Objekt haben kannst und in iconVector schieben, was aber strengstens vermieden werden sollte. Da du im dtor ja nicht mehr weisst, welches das war und ein delete auf ein bereits gelöschtes Objekt kann fatale Folgen habe..)