Einzelnen Beitrag anzeigen

Benutzerbild von sx2008
sx2008

Registriert seit: 15. Feb 2008
Ort: Baden-Württemberg
2.332 Beiträge
 
Delphi 2007 Professional
 
#5

AW: Embeddedform und Prozessbar

  Alt 5. Apr 2014, 16:11
Deine Klasse cDownloadStatusCallback greift bisher fest auf die globale Variable Form1 zu.
Das ist schlecht denn wenn du das Formular dynamisch erzeugst und in ein anderes Formular einbettest ist Form1 nicht korrekt belegt.
Oder stell dir vor du hättest 2 Downloads gleichzeitig laufen.
Also muss die globale Variable Form1 sterben.

cDownloadStatusCallback benötigt ein Label und ein Gauge. Die könntest du schon im Konstruktor übergeben:
Delphi-Quellcode:
type
  cDownloadStatusCallback = class(TObject,IUnknown,IBindStatusCallback)
  private
    FLabel: TLabel; // neu
    FGauge: TGauge; // neu
    function QueryInterface(const IID: TGUID; out Obj): HResult; stdcall;
    ...
  public
    constructor Create(ALabel:TLabel; AGauge:TGauge);


constructor cDownloadStatusCallback.Create(ALabel:TLabel; AGauge:TGauge);
begin
  inherrited Create;
  FLabel := ALabel;
  FGauge := AGauge;
  Assert(Assigned(FLabel)); // Sicherheitsprüfung
  Assert(Assigned(FGauge));
end;
Statt Form1.Label1.Caption := 'Der Download wurde gestartet...'; schreibst du dann
Delphi-Quellcode:
  FLabel.Caption := 'Der Download wurde gestartet...';
  FLabel.Refresh;
fork me on Github

Geändert von sx2008 ( 5. Apr 2014 um 16:21 Uhr)
  Mit Zitat antworten Zitat