Discussion :

[hologram]

Je code sous Code::Blocks.

Je lance la boîte de dialogue principale de mon programme grâce à l 'instruction suivante :SendMessage(hInstance,WM_SETICON,(WPARAM)ICON_SMALL,(LPARAM)500).

Tout fonctionne parfaitement mais j'ai un warning de CB : warning: passing arg 1 of `SendMessageA' from incompatible pointer type. J'aimerais corriger mon texte pour éviter ce warning...

Je n'ai rien trouvé sur Google, quelqu'un peut-il m'aider ?

Merci d'avance,

hologram
.oO°^°Oo.

[Melem]

Bin s'il y a un warning c'est que tout ne fonctionne pas parfaitement .

1. SendMessage sert à envoyer un message à une fenêtre. Le premier argument requis par cette fonction est le handle de la fenêtre à lquelle on veut envoyer un message. Qu'est-ce donc ce hInstance ... ?

2. WM_SETICON est le message qu'il faut envoyer pour changer l'icône associée à une fenêtre. Lorsqu'on on envoie ce message, il faut mettre ICON_BIG ou ICON_SMALL dans wParam selon qu'on veut changer le "BIG ICON" ou le "SMALL ICON". BIG ICON est l'icône affichée par Windows lorsqu'on appuie sur ALT + TAB et SMALL ICON celle que Windows affiche dans la barre de titre de la fenêtre. Puis dans lParam, il faut mettre le handle de l'icône. Donc le (LPARAM)500, c'est pas bien du tout. 500 est le handle d'une icône ?

3. Je sais tout cela car j'ai cherché sur Google .

WM_SETICON Message

[Melem]

Je lance la boîte de dialogue principale de mon programme grâce à l'instruction suivante : SendMessage(hInstance,WM_SETICON,(WPARAM)ICON_SMALL,(LPARAM)500).

Euh ... non c'est pas cette instruction qui lance la boîte de dialogue. C'est DialogBox ou CreateDialog. Utilise DialogBox.

[hologram]

Bon, visiblement mon morceau de code n'était pas suffisant et j'ai dit une connerie c'est pas cette instruction qui lance la dialogbox . Cette instruction vient en complément et génère le placement de l'icône sur la boîte de dialogue.

Voici l'instruction qui lance la dialogbox :
DialogBox(hInstance,"DIALOG1",NULL,(DLGPROC)DlgProc);

Je programme ensuite l'instruction qui génère l'icône et qui donne le message d'erreur :
SendMessage(hInstance,WM_SETICON,(WPARAM)ICON_SMALL,(LPARAM)500);

Le paramètre (LPARAM)500 correspond à un define du resource.h. Ceci dit tout fonctionne correctement, ... , je cherche à améliorer le texte et à comprendre les erreurs.

Le restant je l'ai aussi trouvé sur google.

hologram

[hologram]

Bon j'ai déplacé l'initialisation des icônes dans le case WM_INITDIALOG , avec les instructions suivantes et je n'ai plus mon message d'erreur. Et ça fonctionne. Est-ce plus propre comme ça ?

hIcon=LoadIcon(GetModuleHandle(NULL), MAKEINTRESOURCE(DLG_ICON));
SendMessage(hDlg, WM_SETICON, ICON_SMALL, (LPARAM)hIcon);
SendMessage(hDlg, WM_SETICON, ICON_BIG, (LPARAM)hIcon);

hDlg = le handle de la Dlgbox.

hologram

[Melem]

Bon j'ai déplacé l'initialisation des icônes dans le case WM_INITDIALOG , avec les instructions suivantes et je n'ai plus mon message d'erreur. Et ça fonctionne. Est-ce plus propre comme ça ?

hIcon=LoadIcon(GetModuleHandle(NULL), MAKEINTRESOURCE(DLG_ICON));
SendMessage(hDlg, WM_SETICON, ICON_SMALL, (LPARAM)hIcon);
SendMessage(hDlg, WM_SETICON, ICON_BIG, (LPARAM)hIcon);

hDlg = le handle de la Dlgbox.

Pas que c'est plus propre (ce qui insinuerait que l'autre version était propre d'une quelconque manière ), c'est ENFIN propre. hInstance n'est pas un handle d'une boîte de dialogue, tu espérais quel résultat ? 500 (DLG_ICON je suppose) n'est pas le handle d'une icône, mais l'id d'une icône en ressource. Il faut appeler LoadIcon par exemple pour avoir un handle vers cette icône. Bonne continuation .

[Médinoc]

La seule chose peu propre ici, c'est le fait que tu utilises le même handle d'icône pour régler les deux versions de l'icône de la boîte de dialogue (pour ça, il faudrait deux appels à LoadImage() en spécifiant les deux tailles). Ah, et le fait que tu ne mémorises pas l'ancien handle pour restaurer à la fin et supprimer l'icône.

Par contre, il y a une partie très sale dans la création de ta boîte de dialogue: Tu fais un cast explicite sur le pointeur de fonction. Supprime ce cast et corrige l'erreur en corrigeant le prototype de la fonction.

[hologram]

Je ne comprend le cast explicit ? Je n'ai fait que suivre le tutoriel de CGI (http://chgi.developpez.com/windows/dialogue)

Voici l'appel qui y renseigné :
DialogBox(hinst, "DIALOG1" , hwnd, (DLGPROC)Dialog1Proc);
Voici le mien :
DialogBox(hInstance,"DIALOG1",NULL,(DLGPROC)DlgProc);
le 2ème handle est mis à NULL parce qu'il n'y a pas de parent.

La déclaration de fonction du tuto :
BOOL APIENTRY Dialog1Proc(HWND, UINT, WPARAM, LPARAM);
Ma déclaration :
BOOL APIENTRY DlgProc(HWND hDlg,UINT uMsg,WPARAM wParam,LPARAM lParam);

Ceci dit si on peut faire mieux ...

hologram

[hologram]

Bon ben j'ai lu les tutoriaux de melem et j'y ai vu qu'au lieu de prendre le type BOOL APIENTRY, les DlgBx prennent un type INT_PTR...

Je suppose que c'est de ça que tu voulais parler Médinoc ? Ceci dit j'y avais pensé en lisant la doc msdn, mais là ils déclarent les DlgBx sous les 2 types en question...

Et quelle est la différence entre les deux, le retour est le même (TRUE, FALSE) ?

Concernant l'association des icônes, voici ce que j'ai mis dans le WM_INITDIALOG :

hIconB=LoadIcon(GetModuleHandle(NULL), MAKEINTRESOURCE(DLG_ICON));
hIconS=LoadIcon(GetModuleHandle(NULL), MAKEINTRESOURCE(DLG_ICON));
SendMessage(hDlg, WM_SETICON, ICON_SMALL, (LPARAM)hIconS);
SendMessage(hDlg, WM_SETICON, ICON_BIG, (LPARAM)hIconB);

hologram

[Melem]

BOOL était utilisé dans les anciennes versions de Windows mais maintenant c'est le type INT_PTR qui l'est (demande à MS pourquoi ). La règle est la même, on retourne quand TRUE on a traité un message et FALSE dans le cas contraire. Concernant l'association des icônes, le code est bon.

[Médinoc]

(demande à MS pourquoi ).

Je connais la réponse, mais elle mène sur une autre question: Certains messages attendent de la boîte de dialogue qu'elle retourne directement son résultat casté en BOOL, plutôt que passer par SetWindowLongPtr(DWL_MSGRESULT) (notamment les WM_CTLCOLORxxx). Et comme ces valeurs de retour peuvent être des handles, il fallait un type pouvant contenir un pointeur sous Win64, d'où INT_PTR.
La question à laquelle ça mène, c'est bien sur: Pourquoi une telle entorse au traitement normal des messages?

PS: Comme indiqué dans le lien de ma signature, si tu castes un pointeur de fonction, c'est que ça cache une autre erreur. Donc, autant que possible, il faut éviter de tels casts.
En C++, la règle est un peu plus relaxée, due à la présence des casts C++ plus restrictifs.

PPS: Il va falloir prévenir CGi que sont tutoriel comporte une erreur...

[hologram]

Petit complément d'information je code sous Code::Blocks avec MINGW (GNU GCC Compiler)

Je reviens sur la question de la fonction DlgBx

Si je lance comme ceci :
DialogBox(hInstance,"DIALOG1",NULL,DlgProc);
Le compilateur donne un warning :
warning: passing arg 4 of `DialogBoxParamA' from incompatible pointer type.

Par contre DialogBox(hInstance,"DIALOG1",NULL,(DLGPROC)DlgProc);
passe sans encombre ??

hologram

[Melem]

Je connais la réponse ...

Tu bosses chez MS ? Ah je savais pas qu'on pouvait qu'on pouvait retourner autre chose que TRUE / FALSE dans une DlgProc. Bin c'est de là que vient le problème alors. Mais personnellement, j'aime bien le modèle BOOL (TRUE ou FALSE seulement) comme type de retour et DWL_MSGRESULT pour les informations additionnels s'il y en a suffisant. Je ne vois pas non plus pourquoi avoir compliqué les choses.

Si je lance comme ceci :

DialogBox(hInstance, "DIALOG1", NULL, DlgProc);

Le compilateur donne un warning :

warning: passing arg 4 of `DialogBoxParamA' from incompatible pointer type.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

INT_PTR WINAPI DlgProc(...);

Si t'as un problème avec cela, c'est que ton compilateur utilise encore l'ancien prototype. Tu utilises MinGW 1.0 ? Change alors le prototype en (ancien prototype ...) :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

BOOL WINAPI DlgProc(...);

Mais en aucun cas on caste un pointeur de fonction. De tous les cats possibles, c'est celui-là qui est le plus mauvais.

[hologram]

Je ne sais pas quelle version de mingw est utilisée mais c'est code::blocks 8.02.

Avec INT_PTR CALLBACK, comme dans le tuto de Melem ça fonctionne. Avec INT_PTR WINAPI aussi. Y a-t'il un meilleur choix que l'autre ?

Par contre j'avais pris la déclaration indiquée sur MSDN INT_PTR, et là j'étais obligé de caster.

Merci à tous les deux,


hologram

[Médinoc]

Une DialogProc est ce qu'on appelle "une fonction callback", donc tu sais quel est le meilleur choix...

Par contre j'avais pris la déclaration indiquée sur MSDN INT_PTR, et là j'étais obligé de caster.

http://msdn.microsoft.com/en-us/libr...69(VS.85).aspx

[Melem]

CALLBACK est la convention d'appel utilisée par les fonctions "callbacks" (fonctions que l'on spécifie dans le programme mais que l'on n'appelle jamais soi-même (WndProcs, DlgProcs, etc.)) sous Windows. A l'heure actuelle, c'est la même que celle qui est utilisée par les fonctions de l'API Windows qui n'est autre que WINAPI.