[c++]Mettre une procédure de fenetre dans une class

Version imprimable

Bonjour à tous,

Je suis en train de créer une class TABLEAU dans laquelle se trouve une WNDCLASSEX appelée cTableau!
La procédure de fenetre de cTableau doit absolument manipuler des éléments private de la class TABLEAU.

Du coup, je dois obligatoirement mettre la WindowProc dans la class TABLEAU.

J'écris donc, dans le fichier tableau.h
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 #ifndef TABLEAU_H_ #define TABLEAU_H_ #include <windows.h> typedef struct _CASE{ RECT coordonnees; char contenu[250]; }CASE; class TABLEAU { private: CASE (* cases)[2]; HWND edit_temporaire; char buffer[250]; CASE *pCase; WNDCLASSEX cTableau; int nLignes,nColonnes; //Les méthodes associées LRESULT CALLBACK procedure(HWND , UINT , WPARAM , LPARAM ); public: TABLEAU(HINSTANCE hInstance,int nLignes, int nColonnes); }; #endif
Et, le constructeur de cette classe initialise les membres de la WNDCLASSEX cTableau.

Je suis donc amené à écrire
Code:

1 2 cTableau.lpfnWndProc=procedure;
Le problème c'est que mon compilateur me renvoie l'erreur suivante sur cette ligne :

Citation:

argument of type 'LRESULT(TABLEAU::)(HWND__*,UINT,WPARAM,LPARAM)'does not match 'LRESULT (*)(HWND__*,UINT,WPARAM,LPARAM)'

Voilà! J'aimerais savoir d'où provient cette erreur et comment la contourner.

Merci d'avance.

Spartan03

30/11/2005, 13h25
Aurelien.Regat-Barrel

Il faut que ta procédure soit statique.
30/11/2005, 13h27
Médinoc

IL y avait un exemple dans les "COM tutorial samples" de MSDN: Tu déclares ta procédure en static, et lors du CreateWindow[Ex], tu passes le pointeur de ta classe en paramètre. Tu le récupères ainsi lors du WM_CREATE, et tu le stocke dans GWL_USERDATA avec un petit SetWindowLong() ou SetWindowLongPtr()...

Ainsi, lorsque tu as besoin de manipuler les données de ton objet, tu récupères son adresse avec GetWindowLong() ou GetWindowLongPtr(), et tu travailles sur ce pointeur-là (normalement, puisque la fonction est membre de la classe, elle devrait avoir accès aux données privées)
30/11/2005, 13h39
Aurelien.Regat-Barrel

Quelques liens:
http://c.developpez.com/faq/cpp/?pag...onction_membre
http://www.developpez.net/forums/viewtopic.php?t=397202
http://www.developpez.net/forums/viewtopic.php?t=285423
30/11/2005, 14h31
Spartan03

Si j'ai bien compris, il faudrait que je mette ma fonction membre en static pour que ça marche. Sauf que si une méthode est en static, elle n'accepte plus "this" donc en clair, je ne peux plus modifier d'éléments de ma classe qu'ils soient private ou public. Et c'est là que j'ai pas tout compris? Je vais néanmoins essayer de me débrouiller avec ce que vous m'avez donné en le relisant bien attentivement et (ré)essayer de comprendre.

Merci à tous
30/11/2005, 14h53
Aurelien.Regat-Barrel

Citation:

Envoyé par Spartan03

Si j'ai bien compris, il faudrait que je mette ma fonction membre en static pour que ça marche. Sauf que si une méthode est en static, elle n'accepte plus "this" donc en clair, je ne peux plus modifier d'éléments de ma classe qu'ils soient private ou public.

C'est ça, et c'est pourquoi tu dois associer le pointeur this au HWND, afin de pouvoir récupérer une instance de ta classe à partir d'un HWND. Y'a plusieurs méthodes, tu as eu un rapide apperçu.

Désolé mais je ne suis pas très sur d'avoir compris. Pour que ça marche, il faut que ma classe soit en static mais reste-t-il encore un moyen de modifier des membres de cette dernière. Car je ne peux pas modifier les arguments d'une WindowProc. Là, j'avoue que je suis un peu perdu. L'autre alternative consisterai donc à ne pas mettre la procédure en static auquel cas il y'a une erreur à la ligne suivante :

Code:

cTableau.lpfnWndProc=procedure;

Je vous donne également le fichier tableau.cpp si cela peut vous aider. (il est incomplet pour l'instant)

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
#include "tableau.h"

TABLEAU&#58;&#58;TABLEAU&#40;HINSTANCE hInstance,int noLignes, int noColonnes&#41;
&#123;
     //On crée et enregistre la WndClassEx cTableau
     cTableau.cbSize=sizeof&#40;WNDCLASSEX&#41;;
    cTableau.style=CS_HREDRAW|CS_VREDRAW|CS_DBLCLKS;
    cTableau.lpfnWndProc=procedure;
    cTableau.cbClsExtra=0;
    cTableau.cbWndExtra=0;
    cTableau.hInstance=hInstance;
    cTableau.hIcon=LoadIcon&#40;NULL, IDI_APPLICATION&#41;;
    cTableau.hCursor = LoadCursor &#40;NULL, IDC_IBEAM&#41;;
    cTableau.hbrBackground = &#40;HBRUSH&#41; CreateSolidBrush&#40;RGB&#40;200,200,200&#41;&#41;;
    cTableau.lpszMenuName = NULL;
    cTableau.lpszClassName = "TABLEAU";
    cTableau.hIconSm = 0;
    
    RegisterClassEx&#40;&cTableau&#41;;
     
     
     
     //On crée un TABLEAU au nombre de lignes/colonnes demandées
     cases = new CASE&#91;noLignes&#93;&#91;2&#93;;
     //On enregistre les valeurs du nombre de lignes et de colonnes
     nLignes = noLignes; nColonnes = noColonnes;
&#125;

LRESULT CALLBACK TABLEAU&#58;&#58;procedure&#40;HWND fenetre, UINT message, WPARAM wParam, LPARAM lParam&#41;
&#123;
    switch&#40;message&#41;
    &#123;
        case WM_PAINT&#58;
             //On récupère les dimensions de la zone client dans un RECT
             RECT client;
             GetClientRect&#40;fenetre,&client&#41;;
             //-----------------------------------
             
             //On calcule les coordonnées des futurs cases
             int a;
             for&#40;a=0;a<=nLignes;a++&#41;
             &#123;
                //On calcule celles de la case de droite courante!
                this.cases&#91;a&#93;&#91;1&#93;.coordonnees.left = client.left;
                this.cases&#91;a&#93;&#91;1&#93;.coordonnees.top = a*20;
                this.cases&#91;a&#93;&#91;1&#93;.coordonnees.right = client.right/2;
                this.cases&#91;a&#93;&#91;1&#93;.coordonnees.bottom = &#40;a+1&#41;*20;
                
                //On calcule celles de la case de gauche courante
             &#125;
             
             
        break;
        //----------------------------------------------------------------------
        case WM_DESTROY&#58;
             PostQuitMessage&#40;0&#41;;
        break;
        //----------------------------------------------------------------------
        default&#58;
          return DefWindowProc&#40;fenetre,message,wParam,lParam&#41;;
    &#125;
&#125;

Merci d'avance de m'éclairer sur ce point la

Citation:

Envoyé par Spartan03

Désolé mais je ne suis pas très sur d'avoir compris. Pour que ça marche, il faut que ma classe soit en static mais reste-t-il encore un moyen de modifier des membres de cette dernière. Car je ne peux pas modifier les arguments d'une WindowProc. Là, j'avoue que je suis un peu perdu. L'autre alternative consisterai donc à ne pas mettre la procédure en static auquel cas il y'a une erreur à la ligne suivante :

Code:

cTableau.lpfnWndProc=procedure;

sauf que ça n'est pas possible, car tu as un paramètre caché this en plus si elle est pas static. Windows ne peut pas gérer ce pointeur caché, car il est lié à ton programme C++. C'est donc à toi de le gérer, et de l'associer à ton HWND.
L'idée, c'est d'avoir des fonctions SetPtr/GetPtr qui te permette d'associer une isntance de ta classe à un HWND:
Code:

1 2 3 4 5 6 7 8 9 10 11 void SetPtr( HWND hWnd, TABLEAU *Ptr ) { SetWindowLongPtr( hWnd, GWL_USERDATA, reinterpret_cast<DWORD>( Ptr ) ); } TABLEAU* GetPtr( HWND hWnd ) { return reinterpret_cast<TABLEAU*>( GetWindowLongPtr( hWnd, GWL_USERDATA ) ); }
Tu utilises ainsi:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 case WM_NCCREATE: SetPtr( hWnd, new Tableau ); case WM_PAINT: TABLEAU *ptr = GetPtr( hWnd ); if ( ptr != 0 ) { ptr->cases[a][1].coordonnees.left = ... } case WM_NCDESTROY: delete GetPtr( hWnd );
Dans cet exemple j'utilise GWL_USERDATA, mais en principe il ne faudrait pas acr c'est réservé à celui qui crée la fenêtre, il serait préférable d'utiliser des extra bytes dans ta classe.

30/11/2005, 18h27
Médinoc

Mais dans les cas où tu ne peux régler les Window Bytes toi-même (Ex: boîte de dialogue), tu peux également utiliser les Window Properties, qui bien que lentes, sont une bonne solution de rechange (SetProp()/GetProp())
30/11/2005, 18h31
Spartan03

Ok, je vais essayer de me débrouiller, je ne pense pas que je vais utiliser les Windows Bytes car tout ce que je sais faire avec, c'est les allouer.

Merci pour tout!
01/12/2005, 10h56
Aurelien.Regat-Barrel

Ca fonctionne pareil que GWL_USERDATA, qui est en quelque sorte un extra byte automatique. GetWindowLong( 0 ) te renvoie le premier "extra long"...
Avec les props, faut pas oublier de libérer ensuite.