4. Speicher wird nicht freigegeben

Speicher wird nicht freigegeben

  • P

    Dein Code sieht übrigens kopiert aus.
    Versuch dich erstmal an leichte Sachen.

    0
  • Mod

    Vor alem verstehe ich nicht den Sinn, der hier ein durch new allokierter Buffer hat. Wenn es um einen Pfad geht, häte sein Zeiger auf einen Buffer mit der Größe _MAX_PATH auch genügt.

    0
  • P

    Wie MFK schon gesagt hat,
    das muss New sein, nicht delete -.-"
    Ist doch auch eig. verständlich^^
    Aber trotzdem wünsch ich dir viel Erfolg bei deinem Projekt 😉

    0
  • V

    pivke schrieb:

    Dein Code sieht übrigens kopiert aus.
    Versuch dich erstmal an leichte Sachen.

    Nein, ist alles auf meinem Mist gewachsen. Ist ja auch nicht so, als wäre ich Anfänger, nur wie schwer zu erkennen ist, habe ich vom Umgang mit Speicher kaum Ahnung.

    pivke schrieb:

    Wie MFK schon gesagt hat,
    das muss New sein, nicht delete -.-"
    Ist doch auch eig. verständlich^^
    Aber trotzdem wünsch ich dir viel Erfolg bei deinem Projekt 😉

    Na was denn jetzt? 😉

    Ich dachte nur, der Speicher wird automatisch wieder freigegeben, weil er durch das beenden der Funktion seine Gültigkeit verliert... 😕 Wie soll man das auch anders machen? Ich kann die Variable ja schlecht zurückgeben, wenn ich sie vorher gelöscht habe 😉

    0
  • P

    Ich spiel jetzt mal linker.
    Hier findest du bestimmt auh was über Binren Speicher.
    www.robsite.de

    0
  • ?

    Also: Der Rückgabewert von FILEHANDLER::GetPathFromJobIni ist mit

    delete [] mein_wert;

    freizugeben.

    Wenn du kein Bock auf diese Speicherfreigabe hast, dann solltest du

    TCHAR * cBuffer= new TCHAR[MAX_PATH];

    umwandeln in

    static TCHAR cBuffer[MAX_PATH] = {0};

    😉

    0
  • V

    Okay danke 🙂

    An welcher Stelle muss ich denn das delete anbringen? Nach dem return macht ja keinen Sinn.

    0
  • Mod

    Wenn Du einen Zeiger so returnierst in der aufrufenden Funktion, wo sonst?
    Deshlab mach wie ich es gesagt habe. Übergib einen entsprechenden Zeiger auf einen TCHAR Array der Größe _MAX_PATH und gut ists!

    0
  • ?

    Du rufst GetPathFromJobIni auf und wenn du den Rückgabewert nicht mehr brauchst, löschst du ihn; bspw.:

    TCHAR *pszValue = GetPathFromJobIni (1, TRUE);
...
// Mache irgendetwas mit pszValue
...
delete [] pszValue;

    Martins Lösung ist typischer für die WinApi:

    // Deklaration
void GetPathFromJobIni (int i, BOOL bGetFromDone, TCHAR *pBuffer, int nMaxBuffer);

// Aufrufen:
TCHAR buffer[MAX_PATH] = {0};

GetPathFromJobIni (1, TRUE, buffer, MAX_PATH);
...
// Mache irgendetwas mit buffer
...

    Dann entfernst du die Definition von "cBuffer" aus der Methode "GetPathFromJobIni" und ersetzt die restlichen "cBuffer" durch "pBuffer" (=> s. neue Übergabeargumente)

    0
  • V

    Okay vielen Dank soweit, ich werde morgen mal schauen, wie ich damit zurechtkomme! 🙂

    0
  • V

    Gut, das funktioniert soweit 🙂 Aber eine Frage habe ich noch: Wie kann ich ganze Instanzen von Klassen wieder freigeben?

    0
  • V

    Keine Idee?

    Ich habe eine Klasse erstellt, die diverse Pointer enthält, die wieder freigegeben müssen. Ich würde aber gerne die gesamte Klasse freigeben, anstatt jeden Wert einzeln.

    0
  • T

    geht nicht

    0
  • Mod

    veryxRV schrieb:

    Keine Idee?

    Ich habe eine Klasse erstellt, die diverse Pointer enthält, die wieder freigegeben müssen. Ich würde aber gerne die gesamte Klasse freigeben, anstatt jeden Wert einzeln.

    Schon mal was von einem Destruktor gehört?
    Ansonsten muss man sehen was und wie Du das mit den Zeigern machst.
    Also zeige und Code.

    0

  • @veryxRV: std::auto_ptr, boost::scoped_ptr, boost::shared_ptr, ...

    0
Anmelden zum Antworten