AGB  ·  Datenschutz  ·  Impressum  







Anmelden
Nützliche Links
Registrieren
Thema durchsuchen
Ansicht
Themen-Optionen

Dynamische Packages - Forms anzeigen - Leak

Offene Frage von "Bernhard Geyer"
Ein Thema von Sequitar · begonnen am 12. Jan 2016 · letzter Beitrag vom 13. Jan 2016
Antwort Antwort
nahpets
(Gast)

n/a Beiträge
 
#1

AW: Dynamische Packages - Forms anzeigen - Leak

  Alt 12. Jan 2016, 13:51
Result wurde am anfang auf false gesetzt.
Wenn die form erfolgreich geladen wird, soll der Rückgabewert ja "true" sein. Die form soll ja meist nur einmal geladen werden (und modal angezeigt).


OK, ich seh grad: Result müsste am Anfang auf True initialisiert werden,um sinn zu machen.


Result := Not(Result);
Lustig.
nach mehmaligen Aufrufen weiß keiner mehr, was Result nun ist.
Das glaube ich jetzt nicht wirklich.

Result := Not(Result) , was theoretisch den Wert Result umkehren könnte, steht immer hinter dem Werfen einer Exception und wird von daher im realen Leben vermutlich nie ausgeführt. (oder hab' ich da was übersehen?)

Dashier verstehe ich nicht:
Delphi-Quellcode:
 With TComponentClass(FormClass).Create(Nil) As TCustomForm Do
            Try
              If Modal Then
                Showmodal
              Else
                Show;
            Finally
              Free;
            End;
Du erstellst ein Formular, dann entscheidest Du, ob es Modal oder nicht angezeigt werden soll.
Bei ShowModal dürfte das auch funktionieren, wenn im Finally das Free aufgerufen wird.

Wie ist das denn beim Show? Bleibt das Programm da quasi im Quelltext stehen und wartet, bis das Formular geschlossen wird, bevor im Finally das Free aufgerufen wird?

Hab's gerade mal unter Delphi 7 ausprobiert: Das auf diese Art und Weise erstellte Formular bekomme ich nicht wirklich zu sehen, ein kurzes Flimmern, bei dem man erahnen kann, dass da irgendwo ein Formular angezeigt werden könnte, ja, aber ein benutztbares Formular? Fehlanzeige.

Automatisch "zerstört" werden könnte es nur, wenn im Create statt Nil ein Parent angegeben wird. Wenn niemand weiß, wem das Formular gehört, wer soll denn dann wissen, wann es "beseitigt" werden soll?

Geändert von nahpets (12. Jan 2016 um 13:52 Uhr) Grund: Schreibfehler, mal wieder :-(
  Mit Zitat antworten Zitat
Sequitar

Registriert seit: 8. Jan 2016
74 Beiträge
 
Delphi 10.4 Sydney
 
#2

AW: Dynamische Packages - Forms anzeigen - Leak

  Alt 12. Jan 2016, 14:00
Zitat:
Wie ist das denn beim Show? Bleibt das Programm da quasi im Quelltext stehen und wartet, bis das Formular geschlossen wird, bevor im Finally das Free aufgerufen wird?

Hab's gerade mal unter Delphi 7 ausprobiert: Das auf diese Art und Weise erstellte Formular bekomme ich nicht wirklich zu sehen, ein kurzes Flimmern, bei dem man erahnen kann, dass da irgendwo ein Formular angezeigt werden könnte, ja, aber ein benutztbares Formular? Fehlanzeige.
Das ist mir auch schon aufgefallen, daher habe ich bisher alle Formulare modal erstellt. Wusste nicht wie man das ändert (ausser das formular nicht direkt wieder zu zerstören, was aber auch nichts daran ändert, was nahpets beschreibt: Kurzes Flackern und sonst nichts)


Das Formular könnte theoretisch von unterschiedlichen Anwendungen (oder anderen Formularen) geladen werden. Habe daher jetzt einen "parent" parameter eingefügt, der dann für die Freigabe zuständig sein muss und dem ich als Result die erstellte form zur verfügung stelle:


Delphi-Quellcode:
Function Tplgfrm.Loadform(Parent: Tcomponent; Form: String; Modal: Boolean)
    : TCustomForm;
  Var
    FormClass: Tformclass;
  Begin
    Try
      Form := 'T' + Form;
      FormClass := TFormClass(GetClass(Form));
      If FormClass = Nil Then
        Raise Exception.Create('The form "' + Form + '" is not available.')
      Else
        Begin
          Result := TComponentClass(FormClass).Create(Parent) As TCustomForm;
          With Result Do
            Try
              If Modal Then
                Showmodal
              Else
                Show;
            Finally
              // Free;
            End;
        End;
    Except
      Raise Exception.Create('Form could not be loaded.');
    End;
  End;


//Aufruf

Form := Loadform(Application, 'Formname', True);
Form.Free;

Geändert von Sequitar (12. Jan 2016 um 14:34 Uhr)
  Mit Zitat antworten Zitat
nahpets
(Gast)

n/a Beiträge
 
#3

AW: Dynamische Packages - Forms anzeigen - Leak

  Alt 12. Jan 2016, 14:35
Ich versuchs mal naiv:

Du hast da Deine FormClass.

Wie wäre es, wenn Du die Funktion von diesem Typ machst?

Sowas in der Art:
Delphi-Quellcode:
Function Tplgfrm.Loadform(Parent: Tcomponent; Form: String; Modal: Boolean) : Tformclass;
  Begin
    Try
      Form := 'T' + Form;
      Result := TFormClass(GetClass(Form));
      If not Assigned(Result) Then
        Begin
          Raise Exception.Create('The form "' + Form + '" is not available.');
        End
      Else
        Begin
          With TComponentClass(Result).Create(Parent) As TCustomForm Do
            Try
              If Modal Then
                Showmodal
              Else
                Show;
            Finally
              // Free;
            End;
        End;
    Except
      Raise Exception.Create('Form could not be loaded.');
    End;
  End;
Im Programm wäre dann eventuell sowas möglich:
Delphi-Quellcode:
Procedure irgendwas;
Var
  FormClass : TFormClass;
begin
  FormClass := plgfrm.Loadform(Self,'FromWieAuchImmer', True);

  if Assigned(FormClass) then begin
    ...
    FormClass.Close;
    FormClass.Free;
  end;
end;
Wobei, das Anzeigen des Formulares würd' ich aus der Funktion rausnehmen und im aufrufenden Programm machen, dann kannst Du besser entscheiden, ob das Formular direkt nach dem ShowModal freigegeben wird oder nicht.

Bei 'nem Show kannst Du die Freigabe dann auch beim Beenden des Programmes machen (oder an anderer, sinnvoller Stelle), die variabel FormClass muss dann nur an entsprechender Stelle deklariert werden und nicht innerhalb der Prozedur, die das Formular "anfordert".
  Mit Zitat antworten Zitat
Sequitar

Registriert seit: 8. Jan 2016
74 Beiträge
 
Delphi 10.4 Sydney
 
#4

AW: Dynamische Packages - Forms anzeigen - Leak

  Alt 12. Jan 2016, 14:59
Das mit dem ausgelagerten Anzeigen ist eine gute Idee.
Danke.
Hilft allerdings noch nicht gegen das problem, dass non-modale Forms nicht korrekt angezeigt werden.

Und das eigentliche Problem, dass durch die Klassenregistration Speicher geleakt zu werden scheint.

Delphi-Quellcode:
Initialization

  Registerclass(TTestform); // hier

Finalization

  Unregisterclass(TTestform);
Ich versuchs mal naiv:

Du hast da Deine FormClass.

Wie wäre es, wenn Du die Funktion von diesem Typ machst?

Sowas in der Art:
Delphi-Quellcode:
Function Tplgfrm.Loadform(Parent: Tcomponent; Form: String; Modal: Boolean) : Tformclass;
  Begin
    Try
      Form := 'T' + Form;
      Result := TFormClass(GetClass(Form));
      If not Assigned(Result) Then
        Begin
          Raise Exception.Create('The form "' + Form + '" is not available.');
        End
      Else
        Begin
          With TComponentClass(Result).Create(Parent) As TCustomForm Do
            Try
              If Modal Then
                Showmodal
              Else
                Show;
            Finally
              // Free;
            End;
        End;
    Except
      Raise Exception.Create('Form could not be loaded.');
    End;
  End;
Im Programm wäre dann eventuell sowas möglich:
Delphi-Quellcode:
Procedure irgendwas;
Var
  FormClass : TFormClass;
begin
  FormClass := plgfrm.Loadform(Self,'FromWieAuchImmer', True);

  if Assigned(FormClass) then begin
    ...
    FormClass.Close;
    FormClass.Free;
  end;
end;
Wobei, das Anzeigen des Formulares würd' ich aus der Funktion rausnehmen und im aufrufenden Programm machen, dann kannst Du besser entscheiden, ob das Formular direkt nach dem ShowModal freigegeben wird oder nicht.

Bei 'nem Show kannst Du die Freigabe dann auch beim Beenden des Programmes machen (oder an anderer, sinnvoller Stelle), die variabel FormClass muss dann nur an entsprechender Stelle deklariert werden und nicht innerhalb der Prozedur, die das Formular "anfordert".

Geändert von Sequitar (12. Jan 2016 um 15:06 Uhr)
  Mit Zitat antworten Zitat
bepe

Registriert seit: 17. Okt 2006
119 Beiträge
 
#5

AW: Dynamische Packages - Forms anzeigen - Leak

  Alt 12. Jan 2016, 19:59
Das mit dem RegisterClass ist seltsam. Wie hast du deine Methode denn getestet wenn das Formular gar nicht registriert ist?

Aber mach mal im OnClose von deiner BPL Form Action := caFree; rein. Dann wird das Form beim Schließen freigegeben und das mit dem Leak sollte sich erledigt haben.

mfg,
bp
  Mit Zitat antworten Zitat
Sequitar

Registriert seit: 8. Jan 2016
74 Beiträge
 
Delphi 10.4 Sydney
 
#6

AW: Dynamische Packages - Forms anzeigen - Leak

  Alt 12. Jan 2016, 20:36
Das mit dem RegisterClass ist seltsam. Wie hast du deine Methode denn getestet wenn das Formular gar nicht registriert ist?

Aber mach mal im OnClose von deiner BPL Form Action := caFree; rein. Dann wird das Form beim Schließen freigegeben und das mit dem Leak sollte sich erledigt haben.
Naja, ich kann natürlich die Form nur anzeigen lassen, wenn sie auch registriert ist. Ansonsten
wird ja hier
Delphi-Quellcode:
    Form := Loadform(Application, 'Testform', True);
                  If Form <> Nil Then
                    With Form Do
                      Begin
                        Showmodal; // Leaks 176 bytes
                        Free;
                      End;
auch an entsprechender Stelle abgebrochen.


Da es sich bei einer Testform um ein komplett leeres (neu erstelltes) Formular ohne jeglichen code handelt, das nichts ausführt und keine komponenten hat, außer der Form selbst, dachte ich dass es nur an der Registierung hängen kann...Denkfehler?

Anmerkungen:

- Auch wenn ich bei bestehender Registrierung den gesamten Teil (..Loadform...) auslasse (i.e check ob loadform NIL ergibt) und die Form nicht lade, gibts ein Leck.

- Wenn die Form mehrmals nacheinander aufgerufen und wieder geschlossen wird, bleibt die Leckgröße konstant


Die Action:=cafree festzulegen, hat leider keinen Effekt gezeigt.

Geändert von Sequitar (12. Jan 2016 um 20:44 Uhr)
  Mit Zitat antworten Zitat
Benutzerbild von himitsu
himitsu

Registriert seit: 11. Okt 2003
Ort: Elbflorenz
44.553 Beiträge
 
Delphi 12 Athens
 
#7

AW: Dynamische Packages - Forms anzeigen - Leak

  Alt 12. Jan 2016, 21:48
Und wieviel "Leak" wird es, wenn du die Form 100 Mal anzeigen lässt?
Ein Therapeut entspricht 1024 Gigapeut.
  Mit Zitat antworten Zitat
Sequitar

Registriert seit: 8. Jan 2016
74 Beiträge
 
Delphi 10.4 Sydney
 
#8

AW: Dynamische Packages - Forms anzeigen - Leak

  Alt 12. Jan 2016, 14:45
Übrigens verringert sich das Leak bei Weglassen von Registerclass in der Unit der einzelnen Forms. (Natürlich ist das notwendig um die Klassen zu identifizieren, wenn ich dynamische Packages nutze). Irgendein Workaround / Tip hierfür?
  Mit Zitat antworten Zitat
Antwort Antwort


Forumregeln

Es ist dir nicht erlaubt, neue Themen zu verfassen.
Es ist dir nicht erlaubt, auf Beiträge zu antworten.
Es ist dir nicht erlaubt, Anhänge hochzuladen.
Es ist dir nicht erlaubt, deine Beiträge zu bearbeiten.

BB-Code ist an.
Smileys sind an.
[IMG] Code ist an.
HTML-Code ist aus.
Trackbacks are an
Pingbacks are an
Refbacks are aus

Gehe zu:

Impressum · AGB · Datenschutz · Nach oben
Alle Zeitangaben in WEZ +1. Es ist jetzt 01:24 Uhr.
Powered by vBulletin® Copyright ©2000 - 2025, Jelsoft Enterprises Ltd.
LinkBacks Enabled by vBSEO © 2011, Crawlability, Inc.
Delphi-PRAXiS (c) 2002 - 2023 by Daniel R. Wolf, 2024-2025 by Thomas Breitkreuz