PacMan-Clone wird immer langsamer

[nman]

Ich sehe hier auf Anhieb nichts falsches, aber sieh Dir uU mal den Destruktor von FTGLPixmapFont an, vielleicht "leakts" an dem.

PS: Achte auf const-Correctness und übergib alle Nicht-Primitive per (konstanter) Referenz!

[Gartenzwerg]

hi,
ok das mit der const-Correctness und den Referenzen werde ich mir zu Herzen nehmen. So nun eine Hintergrundfrage: Der Destruktor von FTGLPixmapFont sieht so aus:

FTGLPixmapFont::~FTGLPixmapFont()
{}

völlig leer und nutzlos.

Aber FTGLPixmapFont erbt public von FTFont und dessen Destruktor ist virtual und sieht so aus:

FTFont::~FTFont()
{
    delete glyphList;
}

Wenn ich das richtig verstanden hab, verursacht eine virtuelle Methode eine abstrakte Klasse und alle erbenden Klassen müssen diese virtuelle Methode ersetzen. Wird der FTFont-Destruktor nun automatisch aufgerufen oder nur der von FTGLPixmap? Die glyphList ist das einzig dynamische an der Klasse. Sie speichert die Zeichen, sie muss am Ende wieder freigegeben werden.

Tschau Gartenzwerg

[nman]

Gartenzwerg schrieb:

Wenn ich das richtig verstanden hab, verursacht eine virtuelle Methode eine abstrakte Klasse und alle erbenden Klassen müssen diese virtuelle Methode ersetzen.

Das hast Du falsch verstanden; eine Klasse ist dann abstrakt wenn sie ein oder mehrere rein virtuelle Methoden hat, diese muss dann auch für die abgeleiteten Klassen implementiert werden.
Allerdings gilt es hier den Unterschied zwischen virtuellen und rein virtuellen Methoden zu beachten!

// rein virtuelles foo, Ba kann nicht instanziiert werden (ist abstrakt)
class Base
{
    virtual void foo() = 0;
    // ...
};

// virtuelles foo, Da kann instanziiert werden
class Derived : public Base
{
    virtual void foo() {}
    // ...
};

Wird der FTFont-Destruktor nun automatisch aufgerufen oder nur der von FTGLPixmap?

~Derived ruft selbständig ~Base auf. (Nur korrekt wenn ~Base virtuell ist.)

Die glyphList ist das einzig dynamische an der Klasse. Sie speichert die Zeichen, sie muss am Ende wieder freigegeben werden.

Das klingt seltsam, ich lade mir mal Deine Sourcen herunter und schau sie mir genauer an!

[nman]

Irgendwie finde ich Deinen Code ein bisschen unüberstichtlich aber wo wird denn zB in FTGlyphContainer charMap freigegeben?

[Gartenzwerg]

hi,
ok das war einer der Hauptfehler. Das Menü verbraucht jetzt nur konstant Speicher, d.h. der Speicherverbrauch steigt nicht mehr an. Aber mit jedem neuen Level geht der Speicherverbrauch nochmal in die Höhe, d.h. beim Neustart und Einladen des Levels. Was könnte die Ursache dafür sein? Wieso ist mein Code unübersichtlich. Das er nicht super ist, ist mir schon klar. Aber was hätte ich denn anders organisieren sollen?

Tschau Gartenzwerg

[nman]

Gartenzwerg schrieb:

ok das war einer der Hauptfehler. Das Menü verbraucht jetzt nur konstant Speicher, d.h. der Speicherverbrauch steigt nicht mehr an.

*mirselbstaufdieschulterklopf*

Aber mit jedem neuen Level geht der Speicherverbrauch nochmal in die Höhe, d.h. beim Neustart und Einladen des Levels. Was könnte die Ursache dafür sein?

Naja, irgendwas was beim Entladen des alten Levels geschieht zB? Ich muss mir die Sourcen genauso anschaun wie Du; checke doch einfach mal was genau Dein Programm da macht und was für den Fehler verantwortlich sein könnte! (Ich muss mich in Deine Sourcen erst einarbeiten, Du dich selbst nicht.)
Heute habe ich keine Zeit mehr mir die Sache genauer anzusehen, vielleicht komme ich aber im Laufe des Wochenendes noch dazu.

Wieso ist mein Code unübersichtlich. Das er nicht super ist, ist mir schon klar. Aber was hätte ich denn anders organisieren sollen?

Gibt jetzt nichts was ich Dir so konkret nennen könnte aber die Kommentare zB sind teilweise echt furchtbar und verunstalten außerdem die Header.

[Gartenzwerg]

hi,
wie schon gesagt, wir haben nicht so viel Ahnung von OpenGL etc. Wir haben den Code schon mehrmals durchsucht und nichts auffälliges finden können. Manchmal ist es so, dass man die Lösung vor der Nase hat und sie nicht sieht. Doch jemand der objektiv an die Sache herangeht, findet sie sofort. Deshalb wäre es sehr nett, wenn du dir die Mühe machen würdest und dir unseren Code mal genauer ansehen würdest.

Tschau Gartenzwerg

[nman]

Werde ich machen sobald ich wieder ein bisschen mehr Zeit habe aber sag mir zumindestens welche Methode/ Funktionen aufgerufen werden wenn ein Level beendet ist! (Klar kann ich das auch selbst herausfinden, allerdings wäre das für mich unverhältnismäßig aufwändiger als für Dich!)

[Gartenzwerg]

hi,
danke für deine Hilfe! Du brauchst den Code nun nicht mehr zu durchsuchen. Ich habe den Fehler jetzt selbst gefunden. War total einfach. Ich hatte vergessen vor dem Neuladen des nächsten Levels die alten Texturen zu löschen. Naja nochmals: "Vielen Dank!"

Tschau Gartenzwerg

[nman]

Gratuliere!