spieleprogrammierer.de - Forum und Wiki zur Spieleprogrammierung und Spieleentwicklung

Nicht jeder Enemy hat eine. Die Textur ist statisch... was... nicht so cool ist. Auch die Include-Hierarchie sollte dringend überarbeitet werden. Da wird quasi überall Enemy.hpp inkludiert, obwohl eigentlich die SFML-Includes die einzig wichtigen wären.

Die Textur wird trotzdem immer geladen und eie alte nicht freigegeben.

Das is wohl richtig, einen gewaltigen Memory-Leak hat er.

1) Mit Smart-Pointern wäre das nicht passiert
2) Ohne static wäre das nicht passiert
3) Einen bereits mit Heap-vollgestopften Pointer neu zu allokieren ohne ihn vorher zu prüfen, führt zu Problemen
4) Behebst Du 3 und deletest ihn vorher, hast Du kaputte Sprites

Ja. Gleichzeitig lädst du aber jedes Mal die Textur neu in den Heap, ohne die alte zu löschen.

Im Grunde schon, nur erstellst du die Texture bei jeder Instanzierung eines Objekts im Konstruktor neu, während die alte noch irgendwo rumliegt. Besser: Eine Klasse, welche Texturen verwaltet und zb alle Texturen am Anfang lädt. Das Enemy kann sich dann einen Zeiger auf eine bestimmte holen und dann verwenden. Zusatzaufgabe: vermeide das dabei das schlüsselwort static sowie die Verwendung von globalen Variablen.

PS:
https://www.spieleprogrammierer.de/wiki/…insteiger-Tipps