Class funktionen und Result (MemoryLeak)

[himitsu]

Wenn das Result ein Singleton ist, dann muß man es nicht extra freigeben. (spätestens im Class Destructor oder Finalization).

Ansonsten muß jedes neu erstellte Objekt vom Aufrufer natürlich freigegeben werden.
Man könnte hier aber auch Interfaces oder Records als Result verwenden, inkl. deren automatischer Speicherverwaltung.

[hoika]

Hallo,

Zitat:

dass in einigen dieser Funktionen der Result ein Objekt

etwas so?

Delphi-Quellcode:

class function TClass1.RetClass2_1: TClass2;
var
  Class2: TClass2;
begin
  Class2:= TClass2.Create;
  Result:= Class2;
end;

class function TClass1.RetClass2_2: TClass2;
begin
  Result:= TClass2.Create;
end;

In beiden Fällen muss TClass2 vom Aufrufer freigegeben werden.

[Maliko]

Ja ich glaub für mein Problem sind die Klassenkonstruktoren und -Destruktoren die beste lösung. Ich übergebe im Result einfach nicht mehr direkt das objekt, sondern speichere dieses in einer Klassenvariable zwischen und gebe dann die Variable zurück. Im Anschluss muss ich einfach nur noch die Klassenvariable im Klassendestruktor wieder freigeben. Dürfte glaube ich die Methode sein, die am wenigsten Arbeit verursacht und in diesem Fall auch am saubersten ist.

Und ich meinte übrigens so was:

Delphi-Quellcode:

class function Unit1.TuIrgendwas(blubb : string) : TStringList;
begin
  Result := TStringList.Create;
end;

[hoika]

Hallo,
ja, und in deinem Beispiel muss der Aufrufer die TStringList freigeben.

Anders könnte auch der Aufrufer die TStringList ja auch übergeben.

[Maliko]

Auf jedem Fall schon mal Danke für eure Infos und Beispiele. Das hat mir beim Verständnis des Speichermanagements schon mal sehr gut helfen können.

[himitsu]

"StringList als Parameter reingeben und in Prozedur nur füllen"

Hier aber bitte TStrings als Parameter verwenden (so lange in der Prozedur nicht spezielle Property/Funktionen von TStringList benötigt werden)
-> dann man z.B. auch Memo.Lines oder ListBox.Items reingeben