Jump'n'Run Performance Probleme

Ich pflichte da Darth bei, du forderst in den Schleifen Speicher an (mit new) den du nicht wieder freigibst (mit delete)... Check mal die Schleifen durch.

Aber was anderes am Rande:

int main()
{
  while(GameRunning)
  { 
      if (inGame)
      {
      }
      if (inMenu)
      {
      }
  }
}

Ich würds eher so designen. Ich finde das generell besser, nur eine while-Schleife in der Main zu haben und nicht geschachtelte... aber jedem das seine :D.
rya.

hustbaer

Variablen ala so:

// ...
while (...)
{
    int x;
    Rect r;
}

sind egal.
Was zu Leaks führen kann sind dinge die dynamisch angefordert werden. Also wenn du z.B. sowas hast:

// ...
while (...)
{
    Foo f = LoadFooFromFile("...");
}

Bei sowas kommt es darauf an was "Foo" ist. Ist "Foo" ein "Handle", dann muss es normalerweise wieder freigegeben werden:

while (...)
{
    Foo f = LoadFooFromFile("...");
    // ...
    ReleaseFoo(f);
}

Bei solchen Dinge wäre es meist auch schlauer die 1x vor der Spiel-Schleife anzufordern und erst danach wieder freizugeben:

Foo f = LoadFooFromFile("...");
while (...)
{
    // ...
}
ReleaseFoo(f);

schirrmie

Falls du Visual Studio verwendest kannst du mal den Visual Leak Detector (http://dmoulding.googlepages.com/vld) ausprobieren.

schirrmie

Andi001

Hallo!

Ich denke auch dass Du ein Speicherleck hast, Du kannst neben dem Tool das schirrmie grad gezeigt hat auch mal den AMD CodeAnalyzer drüberlaufen lassen. Dann weisst Du wo dein Programm am meisten Rechenzeit reinsteckt.

@schirrmie

Danke für den Link, habe sowas gesucht. Habe grad mal über meine Engine laufen lassen, die ich schon seit ner weile entwickle und freue mich über die Meldung:

No memory leaks detected.

Gruss, Andi

schirrmie

Ja finde den Satz auch immer wieder toll

schirrmie

Nexus

Das mit dem new und delete könnte durchaus sein. Aber was du mit ReleaseFoo meinst, begreife ich nicht ganz... Ich hab zwar Dateiladungs- und Speicherungsfunktionen, bin mir aber nicht sicher, ob du das gemeint hast... Oder beziehst du dich z.B. auf SDL_LoadBMP("..."), eine Bitmap-Ladungsfunktion? (sry für meine "Unahnung" :))
Trotzdem vielen Dank für die schnellen Antworten!

LG, Nexus

darthdespotism

SDL_Surface *SDL_LoadBMP(const char *file);

?

Wenn du das Surface nicht mehr benötigst musst du es los werden. Warscheinlich per delete

Nexus

Aha ok, vielen Dank... Nein, das wäre wohl eher folgende Funktion:

SDL_FreeSurface(SDL_Surface *Surface);

hustbaer

Ich meinte mit "ReleaseFoo" einfach "die passende Freigebe-Funktion"
"ReleaseFoo" steht in dem Fall für SDL_FreeSurface:

void foo()
{
    // Bild laden (erzeugt eine neue Surface, einen Zeiger darauf speichern wir in "image")
    SDL_Surface* image = SDL_LoadBMP("c:\\test.bmp");

    // ...
    // image verwenden
    // ...

    SDL_FreeSurface(image); // image freigeben
}

Wenn die SDL_FreeSurface Zeile fehlt wird das Bild nicht freigegeben, und du hast ein Speicher-Leak.

BTW: SDL Duko: http://docs.mandragor.org/files/Common_libs_documentation/SDL/SDL_Documentation_project_en/

schirrmie schrieb:

Falls du Visual Studio verwendest kannst du mal den Visual Leak Detector (http://dmoulding.googlepages.com/vld) ausprobieren.

schirrmie

Und noch ein Danke! Seit bei mir Visual C++ komischerweise mit _crtSetDbgFlag(..) nicht mehr die Leaks reported, hab ich mir schon Gedanken gemacht :D. Das Programm ist richtig gut...
Sorry für Offtopic
rya.