TInterfacedObject Subclass

[Uwe Raabe]

Zitat von EWeiss:

Ich habe aber bemerkt das die ListBox kurz nach dem erstellen wieder zerstört wird

Dann zeig doch mal den Code, der die ListBox erzeugt und wie die Variable deklariert ist, der die erzeugte ListBox zugewiesen wird.

[EWeiss]

Zitat von Uwe Raabe:

Zitat von EWeiss:

Ich habe aber bemerkt das die ListBox kurz nach dem erstellen wieder zerstört wird

Dann zeig doch mal den Code, der die ListBox erzeugt und wie die Variable deklariert ist, der die erzeugte ListBox zugewiesen wird.

Kein Problem..

Delphi-Quellcode:

type
  ISkinListBox = interface
    ['{38EF3B4F-86A1-45D0-A7F3-4E45E125979D}']
    function GetHandle: hWnd;
    property Handle: hWnd read GetHandle;
    procedure SetFont(nPointSize: Integer; FontName: PAnsiChar; AktForecolor: COLORREF;
      InAktForecolor: COLORREF; Shadow: Boolean; SOffset: Integer; ShadowColor: COLORREF);
  end;

  TSkinListBox = class(TInterfacedObject, ISkinListBox)
  private
    LStyle:  DWORD;
    FHOwner: HWND;
    FHandle: HWND;
    FEnumProcInst: Pointer;
    ImgIcon:  Cardinal;
    ImgIconH: Cardinal;
    ImgIconW: Cardinal;
    function MakeProcInstance(M: TMethod): Pointer;
    procedure SetCTLFont(hCtL: HWND; Font: hFont);
    procedure  SubClass(WinHandle: HWND);
    procedure UnSubClass(WinHandle: HWND);
    function GetHandle: hWnd;
  public
    property Handle: HWND Read FHandle;
    procedure SetFont(nPointSize: Integer; FontName: PAnsiChar; AktForecolor: COLORREF;
      InAktForecolor: COLORREF; Shadow: Boolean; SOffset: Integer; ShadowColor: COLORREF);
    function ListBoxProc(WinHandle: HWND; Msg: UINT; wP: WParam; lP: LParam): LRESULT; stdcall;

    procedure DrawItem(WinHandle: HWND; Dc: Hdc; Index: Integer;Rect: TRect; Selected: Bool);
    procedure InitTrackbar;
    procedure ListSetTopIndex(hList: HWND; nTopIndex: Integer);
    function ListGetTopIndex(hList: HWND): Integer;
    function ListGetSel(hList: HWND; nSelected: Integer): Bool;
    function GetItemHeight(hList: HWND; ItemHeigh: Integer): Integer;
    function ListCount(hList: HWND): Integer;
    procedure ListDeleteAll(hList: HWND);
    procedure ListDelete(hList: HWND; nIndex: Integer);
    procedure ListSelectPlus(hList: HWND; nSelected: Integer);
    function ListFindString(hList: HWND; Tmp: string): Integer;
    function ListGetCurSel(hList: HWND): Integer;
    function ListAdd(hList: HWND; Tmp: string): Integer;
    function ListGetText(hList: HWND; Item: Integer): PAnsiChar;
    constructor Create(hOwner: HWND; FullpathImageName: string; x, y, xW, yH,
      ListID: integer; Visible: Boolean; ItemHeight: Integer; BackColor: COLORREF);
    destructor Destroy; override;
  end;

type
  LBTYPE = Record
    AktForecolor      : COLORREF;
    InAktForecolor    : COLORREF;
    Backcolor         : COLORREF;
    Shadow            : Boolean;
    ShadowColor       : COLORREF;
    ShadowOffset      : Integer;
    ForeColorSelected : COLORREF;
    BackColorSelected : COLORREF;
    PointSize         : Integer;
    DrawStyle         : Integer;
    BorderStyle       : Integer;
    Icon              : string;
    ItemHeight        : Integer;
    Handle            : HWND;
    Left              : Integer;
    Top               : Integer;
    Width             : Integer;
    Height            : Integer;
    Font              : HFONT;
  end;

Delphi-Quellcode:

constructor TSkinListBox.Create(hOwner: HWND; FullpathImageName: string; x, y, xW, yH,
  ListID: integer; Visible: Boolean; ItemHeight: Integer; BackColor: COLORREF);

begin

  with SkinEngine do
  begin
        // LBS_NOTIFY übergeben ohne wird kein Event
        // auf LBN_DBLCLK ausgelößt
        if Visible = True then
        begin
          LStyle := LBS_HASSTRINGS or LBS_OWNERDRAWFIXED or
                    LBS_NOINTEGRALHEIGHT or LBS_NOTIFY or WS_CHILD or WS_VISIBLE;
        end else
        LStyle := LBS_HASSTRINGS or LBS_OWNERDRAWFIXED or
                  LBS_NOINTEGRALHEIGHT or LBS_NOTIFY or WS_CHILD;

        // Propertys der Listbox festlegen
        ListBoxType.Backcolor := BackColor;
        ListBoxType.ForeColorSelected := GetSysColor(COLOR_HIGHLIGHTTEXT);
        ListBoxType.BackColorSelected := GetSysColor(COLOR_HIGHLIGHT);
        ListBoxType.BorderStyle := EDGE_RAISED;
        ListBoxType.Left := x;
        ListBoxType.Top := y;
        ListBoxType.Width := xW;
        ListBoxType.Height := yH;
        ListBoxType.Icon := FullpathImageName;
        ListBoxType.ShadowColor := (RGB(0,0,0));
        ListBoxType.Shadow:= FALSE;
        ListBoxType.ShadowOffset := 3;
        ListBoxType.ItemHeight := ItemHeight;

        // ListBox erstellen
        FHandle := CreateWindowEx(WS_EX_TRANSPARENT,
          SKLISTBOX, nil, LStyle, ListBoxType.Left, ListBoxType.Top, ListBoxType.Width, ListBoxType.Height,
          hOwner, ListID, skInstance, nil);

        if FHandle <> 0 then
        begin
          ListBoxType.Handle := FHandle;
          SendMessage(Handle, LB_SETITEMHEIGHT, 0, ListBoxType.ItemHeight);

          ListBoxType.DrawStyle := CreateSolidBrush(ListBoxType.Backcolor);
          FHOwner := hOwner;
          SubClass(FHOwner);
        end;
  end;
end;

gruss

[Uwe Raabe]

Und wo wird hier TSkinListBox.Create aufgerufen? Welche Variable hält die erzeugte Instanz? Was für einen Typ hat diese Variable?

Was da im Create passiert, ist wahrscheinlich völlig irrelevant. Die Tatsache, daß du das Window-Handle abspeicherst, hat mit der Lebensdauer der Instanz nichts zu tun.

Ich hätte jetzt hier so etwas erwartet:

Delphi-Quellcode:

var
  SkinListBox: ISkinListBox; // Wichtig! Hier muss eine Interface-Variable stehen. IInterface ginge auch, nicht aber TSkinListBox oder TObject.
...
  SkinListBox := TSkinListBox.Create(...);

[EWeiss]

Zitat von Uwe Raabe:

Und wo wird hier TSkinListBox.Create aufgerufen? Welche Variable hält die erzeugte Instanz? Was für einen Typ hat diese Variable?

Was da im Create passiert, ist wahrscheinlich völlig irrelevant. Die Tatsache, daß du das Window-Handle abspeicherst, hat mit der Lebensdauer der Instanz nichts zu tun.

Ich hätte jetzt hier so etwas erwartet:

Delphi-Quellcode:

var
  SkinListBox: ISkinListBox; // Wichtig! Hier muss eine Interface-Variable stehen. IInterface ginge auch, nicht aber TSkinListBox oder TObject.
...
  SkinListBox := TSkinListBox.Create(...);

Wird es auch
InstrumentList: ISkinListBox; Aber in meiner Anwendung nicht in der TSkinList Classe.


Wird doch von außerhalb aufgerufen...

Delphi-Quellcode:

    InstrumentList := CTRL_ListBoxCreate(hMain, PAnsiChar(SKAERO_FOLDER + 'Sound.png'),
      145, 62, 610, 268, ID_INSTRUMENTLIST, True, 18, SKAERO_INACTIVECAPTION);
    SKAERO_SetAnchorMode(InstrumentList.Handle, ANCHOR_RIGHT);
    SKAERO_SetZorder(InstrumentList.Handle, ANCHOR_RIGHT);
    InstrumentList.SetFont(SKAERO_CAPTIONFONTHEIGHT, PAnsiChar(SKAERO_TEXTFONT),
      SKAERO_ACTIVECAPTION, SKAERO_INACTIVECAPTION, TRUE, 2, 0);

In der DLL..

Delphi-Quellcode:

function CTRL_ListBoxCreate(hOwner: HWND; FullpathImageName: string; x, y, xW, yH,
  ListID: integer; Visible: Boolean; ItemHeight: Integer; BackColor: COLORREF): ISkinListBox; stdcall;
begin

  result := TSkinListBox.Create(hOwner, FullpathImageName, x, y, xW, yH,
    ListID, Visible, ItemHeight, BackColor);
end;

Zurückgegeben wird ein handle mit dem ich in der Anwendung
in verbindung mit der übergebenen ID der ListBox arbeite.

Zitat:

Was da im Create passiert, ist wahrscheinlich völlig irrelevant.

In wie fern ?
Du kennst doch die Zusammenhänge nicht wie ich mit den Daten im weiteren verlauf arbeite.
Alles was deklariert wurde wird auch im späteren verlauf verwendet.
Und funktioniert in meiner anderen Anwendung "OHNE" Dll.

gruss

[Uwe Raabe]

Zitat von EWeiss:

Zitat:

Was da im Create passiert, ist wahrscheinlich völlig irrelevant.

In wie fern ?
Du kennst doch die Zusammenhänge nicht wie ich mit den Daten im weiteren verlauf arbeite.
Alles was deklariert wurde wird auch im späteren verlauf verwendet.
Und funktioniert in meiner anderen Anwendung "OHNE" Dll.

Mit irrelevant meinte ich "für die Lebensdauer der Instanz irrelevant". Du erwähntest, daß die Listbox kurz nach der Erstellung wieder freigegeben wird. Wenn nicht an irgendeiner Stelle die Instanz explizit freigegeben wird, kann das eigentlich nur geschehen, wenn sämtliche Interface-Referenzen auf die Instanz (auch implizit) auf nil gesetzt werden. Also sollte solange der Variablen InstrumentList nichts anderes zugewiesen wird und diese Variable nicht aus dem Scope verschwindet auch die TSkinListBox-Instanz intakt bleiben.

Kannst du nicht einen Breakpoint in das Destroy setzen und schauen, wodurch es ausgelöst wurde?

BTW, es macht zwar bislang hier keinen Unterschied, aber man sollte doch immer inherited Create aufrufen. Man weiß ja nie, was in zukünftigen Delphi-Versionen in TObject.Create so alles noch eingebaut wird.

[EWeiss]

Habs schon versucht.. mit Breakpoint
Das problem ist das der Compiler nur den ASM Bildschirm öffnet und da kann ich nur lesen

Zitat:

TObject.InitInstance:

Als wenn bei der Initialisierung irgend etwas schief läuft.

Zitat:

aber man sollte doch immer inherited Create aufrufen

Ja das kann ich machen.


gruss

[Uwe Raabe]

InitInstance ist auch in Assembler, da wird man nicht viel mehr sehen können. Aber vielleicht kannst du im Aufrufstack etwas erkennen?

In der DLL wird nach dem Create dreimal auf InstrumentList zugegriffen. Kommt es überhaupt dazu oder knallt es schon beim Create?

Ich weiß, daß man in einer DLL nicht immer alles das machen kann, was in einer Exe problemlos möglich ist. Ich habe aber noch keine Ahnung, was das hier sein könnte. Zumindest würde man mit diesem Wissen die Suche in eine andere Richtung bringen.