WndProc in eine Klasse bekommen...

KPC

Ich habe letztens in der Cpp der Datei der Klasse eine globale Variable auf die Klasse gesetzt und die WndProc als friend in der Klasse deklariert. Ist eine sehr unelegante Lösung und hat das Problem, dass man nur ein Objekt der Klasse haben kann (wegen der globalen Variable).
Klappt allerdings auch...

Mis2com

Also ich denke, dass ich alles statisch mache.
Maximal möchte ich für meine Zwecke sowieso nur eine Instanz, da ich mit DirectX arbeite und einfach keine Lust habe, den ganzen WinAPI-Code immer abzutippen.

Ansonsten:
Welche Lösung wäre denn jetzt mal 'elegant'?

Shade Of Mine

Original erstellt von Mis2com:
Welche Lösung wäre denn jetzt mal 'elegant'?

Ich persoenlich halte folgende fuer sehr elegant:

WndProc ist static, beim WM_CREATE wird allerdings mit SetWindowLong() der this Zeiger, der bei CreateWindow als letzter parameter uebergeben wird, gespeichert.

WndProc fragt dann jedesmal diesen this Zeiger wieder ab.

KPC

Die Lösung gefällt mir, da brauche ich meine globale Variable gar nicht mehr

WebFritzi

Hey was soll das? Werde ich hier einfach ignoriert, oder was?

WebFritzi, deine Lösung ist völliger quatsch!

WebFritzi

*LOL* Ok. Ich geb mich geschlagen. Aber dann würde ich doch noch gerne wissen, warum. Bzw. möchte ich gerne wissen, was ich an dem Problem falsch verstanden habe. Ich dachte nur, er will ein Objekt haben, dem man eine WindowProc zuordnen kann. Klärt mich bitte auf. Will doch was lernen.

WebFritzi

OK, ich habe jetzt verstanden, um was es geht. Ich habe mir jetzt eine kleine Klasse namens CWindow gemacht, für die jede Instanz eine eigene WindowProc haben soll, die NICHT außerhalb der Klasse zu finden sein soll. Ich denke, das war genau Mis2coms Problem. Jetzt verstehe ich aber Shades Vorschlag nicht. Wie macht man das? Hier mal meine Header-Datei der Klasse:

//---------------------------------------------------------------------------
#ifndef CWindowH
#define CWindowH
//---------------------------------------------------------------------------
#include <windows.h>
//---------------------------------------------------------------------------
typedef long(__stdcall* WNDPROC)(void*,unsigned int,unsigned int,long);

class CWindow
{
   private:
   LPCTSTR     fClassName;
   HINSTANCE   fInstance;
   HWND        fHandle;
   HWND        fParentHandle;
   DWORD       fStyle;
   DWORD       fExStyle;
   HICON       fIcon;
   HCURSOR     fCursor;
   HMENU       fMenu;
   WORD        fID;
   int         fLeft;
   int         fTop;
   int         fWidth;
   int         fHeight;
   LPCTSTR     fCaption;
   WNDPROC     fWndProc;

   BOOL        fRegistered;
   BOOL        fCreated;

   protected:
   VOID Initialize();

   public:
   CWindow(LPCTSTR classname, HINSTANCE);
   CWindow(LPCTSTR classname, HINSTANCE, HWND parent);
   ~CWindow(void);

   ATOM    Register(WNDPROC);
   BOOL    Create(void);

   BOOL    SetStyle(DWORD);
   BOOL    SetExStyle(DWORD);
   VOID    SetIcon(HICON);
   VOID    SetCursor(HCURSOR);
   VOID    SetMenu(HMENU);
   VOID    SetID(WORD);
   VOID    SetLeft(int);
   VOID    SetTop(int);
   VOID    SetWidth(int);
   VOID    SetHeight(int);
   VOID    SetBounds(int left, int top, int width, int height);
   VOID    SetRect(RECT);
   VOID    SetCaption(LPCTSTR);
   //VOID    SetWindowProc(WNDPROC);

   HWND    Handle();
   HWND    ParentHandle();
   /*DWORD   Style();
   DWORD   ExStyle();
   HICON   Icon();
   HCURSOR Cursor();
   HMENU   Menu();
   WORD    ID();
   int     Left();
   int     Top();
   int     Width();
   int     Height();
   RECT    Rect();
   LPCTSTR Caption();
   WNDPROC WndProc();*/

   LRESULT Perform(UINT, WPARAM, LPARAM);
   /*BOOL    AddStyle(DWORD);
   BOOL    DeleteStyle(DWORD);
   BOOL    AddExStyle(DWORD);
   BOOL    DeleteExStyle(DWORD);*/
};
//---------------------------------------------------------------------------

LRESULT CALLBACK ThisWndProc(HWND hwnd, UINT iMsg, WPARAM wParam, LPARAM lParam)
{
  CWindow*  This;

  switch(iMsg)
  {
     case WM_CREATE:
        break;

     case WM_COMMAND:
        MessageBox(NULL, "HALLO", "HALLO", MB_OK);
        break;

     case WM_PAINT:
        break;

     case WM_DESTROY:
        PostQuitMessage(0);
        break;
  }
  return DefWindowProc(hwnd, iMsg, wParam, lParam);
}
//---------------------------------------------------------------------------
#endif

KPC

Shade meint glaube ich :

class Window
{
    static LRESULT CALLBACK ThisWndProc(HWND hwnd, UINT iMsg, WPARAM wParam, LPARAM lParam);
};

LRESULT CALLBACK Window::ThisWndProc(HWND hwnd, UINT iMsg, WPARAM wParam, LPARAM lParam)
{
    CWindow*  This;

    switch(iMsg)
    {
    case WM_CREATE:
        This = (Window *) (((LPCREATESTRUCT) lParam)->lpCreateParams);
        break;

    }
}

Beim erstellen musst du jetzt den Window Pointer übergeben.

WebFritzi

Jo, so ähnlich hatte ich mir das dann auch gedacht. Aber ich verstehe das nicht. Warum sollte in WM_CREATE in den CreateParams genau der Zeiger auf das CWindow-Objekt sein, das das Handle hwnd hat? Also, der "momentane" Zeiger? Und warum nimmt er WndProc ohne Angabe von static nicht, und so schon? Fragen über Fragen - aber ich hoffe, ihr könnt sie beantworten. Danke.

lol, den zeiger auf die fensterklasse übergibt man bei CreateWindow, also CreateWindow(..., ..., this); Deshalb kann man dann bei den CreateParams darauf zugreifen

Ich hab das so gemeint wie KPC erklärt hat...

Nur hat KPC vergessen im WM_CREATE den This auch mit SetWindowLong bzw. SetWindowLongPtr zu speichern und die erste Zeile von ThisWndProc sollte This mit GetWindowLong/Ptr füllen...

@Webfritzi:
Die wndproc MUSS static sein, anders kann sie nicht in der klasse sein, was sie aber sein muss, sonst ist die kapselung im *****...

deshalb machen wir sie static.

den this gibt man bei CreateWindow als letzten Parameter an, der wird beim WM_CREATE dann mitgeliefert in der CREATESTRUCT. somit können wir ihn speichern...

bisher habe ich noch nie eine andere vernüftige möglichkeit gesehen, das anders zu lösen. ich lasse mich aber gene eines besseren belehren.

WebFritzi

Ich habe das jetzt so gemacht, dass ich den Pointer in der USERDATA den Windows speichere. Dann brauche ich diesen CREATESTRUCT auch nicht. Außerdem steht in meiner SDK-Hilfe:

lpCreateParams

Points to data to be used for creating the window.
Windows NT: This member is the address of a SHORT (16_bit) value that specifies the size, in bytes, of the window creation data. The value is immediately followed by the creation data. For more information, see the following Remarks section.

WebFritzi

Original erstellt von Shade@work:
**@Webfritzi:
Die wndproc MUSS static sein, anders kann sie nicht in der klasse sein, was sie aber sein muss, sonst ist die kapselung im *****...

deshalb machen wir sie static.
**

Das habe ich schon gemerkt, dass sie static sein MUSS. Aber wieso? Ich meine, ich wüsste gerne die Bedeutung des Schlüsselwortes static in diesem Zusammenhang. Und warum geht's nicht ohne static, was wieder Mis2coms Frage vom Anfang wäre.

Shade Of Mine

die frage wurde doch schon beantwortet!

Jeder Methode wird der this zeiger uebergeben.
A a;
a.b() == b(&a)
von der funktionsweise her!

eine static methode hat aber keinen this zeiger, deshalb ist
a.b() nicht moeglich wenn b static ist.
A::b() ist dann aber sehr wohl moeglich!
A::b() == b()
und somit sind wir das problem losgeworden, dass der compiler der wndproc den this pointer mitgeben wollte...

Denn die wndproc darf keinen this zeiger bekommen, denn sie ist callback, was soviel heisst wie: windows ruft die funktion auf! und windows hat keine ahnung welchen this zeiger es mitgeben muss.

Endy

Wenn du ne Wrapperklasse schreiben willst kannst du bei jedem aufruf der funktion einfach eine funktion deiner Wrapperklasse starten, du legst in der funktion ne statische map an und öffnest immer jeweils die funktion zum HWND das ist dann fast das selbe... du musst halt noch den zwischenschritt machen...

erm... schau mal in meinem Tut unter Entwicklung eines Steuerelements!
ciaoy

> Endy <<

5thElement

da gibts ne viel einfachere lösung....

class CWindow
{
static LRESULT CALLBACK BaseWndProc(HWND hWnd, UINT uMsg, WPARAM wParam, LPARAM lParam);

LRESULT CALLBACK WindowProc(HWND hWnd, UINT uMsg, WPARAM wParam, LPARAM lParam);
};

LRESULT CALLBACK BaseWndProc(HWND hWnd, UINT uMsg, WPARAM wParam, LPARAM lParam)
{
return CWindow::WindowProc(hWnd, uMsg, wParam, lParam);
}

LRESULT CALLBACK CWindow::WindowProc(HWND hWnd, UINT uMsg, WPARAM wParam, LPARAM lParam)
{
//hier kannste jetzt alles machen...
}

und bei der wndclass kannste jetzt die basewndproc als wndproc angeben.....

mfg 5th

[ Dieser Beitrag wurde am 16.06.2002 um 11:22 Uhr von 5thElement editiert. ]

Toll, das bringt's!!!!

Christoph

Das geht doch gar nicht! :p
Schonmal versucht das zu kompilieren?!

<eigenwerbung>Hier gibts meine WinAPIWrapper inklusive Quelltext.</eigenwerbung>

Bigor

eins vorweg:ich steige gerade um von c auf c++!
es ist nur eine idee aber vielleicht kann man das problem als lösung verwenden!!
vielleicht geht es das man die WndProc mit drei parametern deklariert(den indirekten this noch dazu sind es 4!)
windows übergibt ja auch genau 4 parameter,ich hoffe das alle 4Byte lang sind(bei windows weis ichs, in der eigens definierten müssen dann alle 4Byte lang sein)
,wenn man den this dann in HWND castet müsste es gehen(falls der this der 1.param ist)
,den this muss man dann im konstruktor in einer "this" variable speichern(falls man ihn braucht).
ich weis nich ob das geht aber ich bin mir fast sicher mit rumgecaste müsste es gehen.was sagt ihr dazu ich bin ja in c++ nich so bewandert,könnte es gehen?gebt mal euren senf dazu ab! :p