Memoryleak oder doch nicht?

[Fritzew]

Das wird sicher nur 1+ durchlaufen?

Delphi-Quellcode:

if nReading = 0 then
  Begin
    New(FToPBuffer);
    LineStart := FToPBuffer;
    LineStart^.Nr := 0;
  end;

[EWeiss]

Zitat von Fritzew:

Das wird sicher nur 1+ durchlaufen?

Delphi-Quellcode:

if nReading = 0 then
  Begin
    New(FToPBuffer);
    LineStart := FToPBuffer;
    LineStart^.Nr := 0;
  end;

Ja wenn nReading = 0

EDIT:
Und LineStart wird ja freigegeben.
Da es der gleiche Pointer wie der von FToPBuffer ist darf ich FToPBuffer nicht nochmals freigeben das wäre ja dann doppelt gemoppelt.


gruss

[Fritzew]

Ohne mehr Code kann ich Dir nicht mehr sagen,
aber es wird ja wohl definitiv mehr erzeugt als freigeben wird.....
Mach doch mal Spasseshalber zum Debugger eine globale Variable die du hochzählst bei jedem! erzeugen und runterzählst beim freigeben. Sollte ja am Ende 0 sein

[EWeiss]

Zitat von Fritzew:

Ohne mehr Code kann ich Dir nicht mehr sagen,
aber es wird ja wohl definitiv mehr erzeugt als freigeben wird.....
Mach doch mal Spasseshalber zum Debugger eine globale Variable die du hochzählst bei jedem! erzeugen und runterzählst beim freigeben. Sollte ja am Ende 0 sein

Kein Problem muss mich da wohl durch wurschteln. Danke.

gruss

[Stevie]

Davon abgesehen kannst dir evtl den ganzen Pointerkrams/New/Dispose und ne linked list sparen und ein Array nutzen. Linked lists sind nur dann wirklich besser als arrays wenn man zwischendurch viele! Hinzufüge oder Entfernen Operationen hat, da sie die nachfolgenden Elemente in einem Array nicht verschieben müssen oder das Array resizen müssen. Und auch dann würd ich das nur nach ausgiebigen Performancetests unterschreiben, denn selbst dann spielt noch eine Rolle über was für eine Lebensspanne der liste wir reden. Lebt sie lang genug, wird sie irgendwann Opfer von Speicherfragmentierung (die ganzen Nodes liegen kreuz und quer im RAM) wogegen der CPU Cache/Prefetcher bei nem Array nur kurz zuckt.

[EWeiss]

Ich lese eine Textdatei ein und diese bzw.. die geparsten Daten davon leben solange wie die Classe lebt.
Ein Array nutzt in dem fall gar nichts.

EDIT:
Die eingelesenen Daten werden über die gesamte Classe verteilt.

gruss

[Stevie]

Zitat von EWeiss:

Ich lese eine Textdatei ein und diese bzw.. die geparsten Daten davon leben solange wie die Classe lebt.
Ein Array nutzt in dem fall gar nichts.

Ein array of TParseFile statt einer linked list nützt nichts weil?