MemoryLeak bei Frame und TComboBox.Items.AddObject()

[Poelser]

Moin,

ja, ich kann mit Delphi 10.2 das Speicherleck nachvollziehen.

Ich habe dann mal den Create mit Application statt Self ausgeführt, also
  FMainFrame := TFrame1.Create(Application); und es gibt kein Speicherleck mehr.

LG aus dem hohen Norden, Edmund

[Aviator]

Zitat von Poelser:

Moin,

ja, ich kann mit Delphi 10.2 das Speicherleck nachvollziehen.

Ich habe dann mal den Create mit Application statt Self ausgeführt, also
  FMainFrame := TFrame1.Create(Application); und es gibt kein Speicherleck mehr.

LG aus dem hohen Norden, Edmund

Danke für den Test.

Aber das sollte ja denke ich nicht die Lösung sein. Es müsste ja möglich sein, dass ich eine Form oder gar ein Panel oder whatever als Owner für das Frame angeben kann, ohne das ich hunderte Memory Leaks bekomme.

[Poelser]

Als Lösung würde ich das auch nicht bezeichnen, eher als Würgaround.

Klar sollte der Eigentümer des Frames ein Form oder ein Control á la TPanel sein und nicht die Applikation selbst. Ich hab' schon seit einiger Zeit nichts mehr mit Frames gemacht, kann mich aber aus D7-Zeiten noch daran erinnern, dass die Dinger sich oft irgendwie komisch verhalten haben

LG aus dem hohen Norden, Edmund

[himitsu]

Zitat:

Ein OnCreate() und OnDestroy() gibt es bei Frames ja leider nicht. Ich muss das Frame aber dynamisch erzeugen und es mit Werten füllen.

Dann aber mit override.
Am Speicherleck ändert es aber nichts. Beim Aufruf des Create über einen Vorfahrentypen würde dann die ComboBox einfach leer sein weil dem Vorfahren dein Create nicht bekannt ist über dessen "virtual". (z.B. wenn der Frame im Formdesigner auf die Form gepappt und in der DFM gespeichert wurde)

Haltepunkt in TFrame1.Destroy und schauen ob das Event aufgerufen wird und wenn ja, ob dort die Items der Combobox "noch" gefüllt sind.

Alternative:

Delphi-Quellcode:

  TTestObject = class(TComponent)
  ...

  TestObject := TTestObject.Create(cbb1);  // oder self oder wo immer es passt
  TestObject.TestProperty := 'Item 1';
  cbb1.Items.AddObject('Item 1', TestObject);

  // und das Free im Destroy kann entfallen

[Aviator]

Danke für deine Anteilnahme himitsu, aber du solltest meinen ersten Beitrag und auch die nachfolgenden Posts lesen. Dann wirst du merken, dass deine Vorschläge so alle nicht umsetzbar sind bzw. alles schon so durchgeführt wurde.

[himitsu]

Wie gesagt:

Zitat:

Haltepunkt in TFrame1.Destroy und schauen ob das Event aufgerufen wird und wenn ja, ob dort die Items der Combobox "noch" gefüllt sind.

[Aviator]

Zitat von himitsu:

Wie gesagt:

Zitat:

Haltepunkt in TFrame1.Destroy und schauen ob das Event aufgerufen wird und wenn ja, ob dort die Items der Combobox "noch" gefüllt sind.

Wie gesagt ... lies bitte meinen Post komplett und auch die der anderen Mitglieder ...
Post.jpg