Win Api : redessiner une fenêtreRésolu

Question

Bonjour déjà 

Bon voilà mon problème et je seche depuis quelques heures dessus. Ce serait sympa de m'aider .

Je fais un petit programme en API windows . J'ai dessiné une fenêtre, une image de fond, quelques boutons... J'aimerais pouvoir redessiner cette fenêtre quand je presse sur un bouton.

Les notifications des boutons sont testées dans "case WM_COMMAND" par une fonction de mon cru  dans laquelle une structure de deux termes(  BOOL)  est mise à jour et qui est retournée par cette fonction..... ok ? ( En fait leurs valeurs sont inversée. )

J'ai vérifié, sous WM_COMMAND la structure est bien modifiée en passant pas la fonction en question.
Alors j' utilise la fonction InvalidateRect comme ceci pour forcer le message WM_PAINT.

if (temoins.liste) InvalidateRect(hwnd, 0, TRUE);

Passons au code de case WM_PAINT : 

 case WM_PAINT:
            {
                HDC hdc =NULL;
                PAINTSTRUCT ps;
                hdc = BeginPaint(hwnd,&ps);
                if(temoins.couv) dessinerCouverture (hwnd,hdc,ps, "Carnet                                              d'adresses",hBouton); // la fonction 
                if(temoins.liste) exit(0);  //pour voir si ça passe ...et non ! :( 
                EndPaint(hwnd, &ps);
                return 0;
            }
        return 0;
Comme vous l'avez compris,  ma fenêtre ne se ferme pas quand temoins.liste =TRUE. Pourquoi ??
J'aimerais bien pouvoir dessiner ma fenêtre avec une autre fonction quand temoins.liste =TRUE.

Merci pour votre réponse.

vecchio56 · Accepted Answer

Erreur bête qu'on aurait jamais trouvée sans que tu donne tout le code.
Dans la fonction Procedure, tu as la ligne temoins.liste=FALSE; qui est exécutée à chaque fois que la fonction est appelée. Donc à chaque Wm_PAINT (entre autre), temoins.liste est mis a FALSE. Normal donc que la fenêtre ne se ferme pas

_____________________________________
Un éditeur de ressources gratuit pour Windows

vecchio56 · Answer

if (temoins.liste)
{
  InvalidateRect(hwnd, 0, TRUE);
  UpdateWindow(hwnd);
}

Comme ca tu es sur que la valeur de temoins.liste ne changera pas entre temps

_____________________________________
Un éditeur de ressources gratuit pour Windows

racpp · Answer

Salut,
Comment est déclarée ta structure? en global ou en static dans la procédure de la fenêtre?

acryline · Answer

Je viens d'essayer ta solution mais la fenêtre reste ouverte . 
Bon ça m'évite déjà de prendre le risque que le message change effectivement.

Voici ma fonction au cazou...

TEMOINS messagesBoutonsCouv( HWND hwnd,HWND *hBouton, UINT uMsg , 
                            WPARAM wParam, LPARAM lParam, TEMOINS temoins)
{
      static long i=0,j=0;
        for(i=0;i<27;i++)
           {
            if(HIWORD(wParam)== BN_CLICKED)
            {
              if(lParam== (LPARAM)hBouton[i])
               {
                   for(j=0;j<27;j++) {DestroyWindow(hBouton[j]); } 
                   //les boutons s'effacent pour ça pas de problème
                   temoins.couv=FALSE;
                   temoins.liste=TRUE;
               }
            }
           }
           return temoins;
  }

vecchio56 · Answer

La structure TEMOINS est passée par valeur, donc les changement sont pas pris en compte dans la fonction originale. Passe un pointeur de la structure, ou mets la en variable globale

_____________________________________
Un éditeur de ressources gratuit pour Windows

acryline · Answer

La structure est  en static et ça marche pas ...

acryline · Answer

Je vais esayer mais ça va me prendre un peu de temps... moi les pointeurs j'aime pas ça lol et le variables globales j'essaie d'éviter...

à plus pour les nouvelles :!

et surtout merci pour vos réponses... c'est hyper rapide !

acryline · Answer

J'ai essayé en globale et en pointeur et ni l'un ni l'autre ne fonctionnent !!
Quel casse tête !

vecchio56 · Answer

void messagesBoutonsCouv( HWND hwnd,HWND *hBouton, UINT uMsg , 
                            WPARAM wParam, LPARAM lParam, TEMOINS* temoins)
{
  remplacer tous les temoins. par des temoins->
  rien a retourner
}

Mais j'ai un doute, je vois que ta fonction retourne une structure TEMOINS donc si tu l'affectais à la variable d'origine, ce devrait peut être marcher. Par contre utiliser un pointeur est plus performant

_____________________________________
Un éditeur de ressources gratuit pour Windows

racpp · Answer

Pour les variables globales, on doit les éviter tant que possible. Les pointeurs sont au contraire indispensables en programmation C/C++. Ta fonction messagesBoutonsCouv() prend un pointeur comme 2ème paramètre.
Tu sais ce n'est pas toujours facile de trouver une erreur à partir d'un morceau de code. Il faut voir la totalité du code si c'est possible.

acryline · Answer

Oui oui je sais que les pointeurs sont indispensables... ça commence à venir mais c'est pas tjrs évident.... j'ai peut être ré^pondu un peu vite tout à l'heure, je reprends mon code ..(avec les pointeurs ) à++ 
et encore merci !

acryline · Answer

Bon désolée mais ça ne marche pas... J'ai bien transformé le code pour utiliser les pointeurs. Le message WM_COMMAND : case WM_COMMAND: //Traiter les messages du menu messagesMenu (hwnd, hBouton,WM_COMMAND, wParam,lParam); //Traiter les messages des boutons des lettres de la couverture messagesBoutonsCouv (hwnd, hBouton,WM_COMMAND, wParam,lParam,&temoins); if (temoins.liste) { InvalidateRect(hwnd, 0, TRUE); UpdateWindow(hwnd); // exit(0); // ici la fenêtre se ferme pour TRUE } return 0; Le message WM_PAINT : case WM_PAINT: { HDC hdc = NULL; PAINTSTRUCT ps; hdc = BeginPaint(hwnd,&ps); if(temoins.couv) dessinerCouverture (hwnd,hdc,ps, "Carnet d'adresses",hBouton); if(temoins.liste) exit(0); // ça ne marche pas ! EndPaint(hwnd, &ps); } return 0; La fonction concernée : void messagesBoutonsCouv( HWND hwnd,HWND *hBouton, UINT uMsg , WPARAM wParam, LPARAM lParam, TEMOINS* temoins) { static long i =0,j=0; for(i=0;i<27;i++) { if(HIWORD(wParam)== BN_CLICKED) { if(lParam== (LPARAM)hBouton[i]) { for(j=0;j<27;j++) {DestroyWindow(hBouton[j]); } //les boutons s'effacent pour ça pas de problème temoins->couv=FALSE; temoins->liste=TRUE; } } } } Voilà je pense que tout y est. J'ai déclaré la structure une fois en static une fois sans rien... ça ne change pas.

vecchio56 · Answer

Je ne vois aucune erreur, le problème se situe peut être ailleurs. Comme le dit racpp, un code complet sera plus utile (avec le debugger on trouve l'erreur en deux minutes)

_____________________________________
Un éditeur de ressources gratuit pour Windows

acryline · Answer

Je l'envoie ? ... et comment ?

vecchio56 · Answer

Tu peux coller le code ici, ou bien mettre un lien vers l'endroit de ton choix (http://www.nomorepasting.com/paste.php par exemple)

_____________________________________
Un éditeur de ressources gratuit pour Windows

acryline · Answer

http://mapage.noos.fr/site1/carnet.rar



Voilà ce n'est que le code, je n'ai pas mis les images !

acryline · Answer

J'vaoue il y a pas mal de globales ...parce que j'aimerais les mettre plus tard dans un fichier de configuration.... 
Je vais essayer de changer ça plus tard !

vecchio56 · Answer

Autre chose, j'ai vu une chose bizarre dans ton code:
MessageBox(hwnd,(LPCWSTR)"Ce fichier n'existe pas !  ", (LPCWSTR)"Attention !", MB_ICONWARNING|MB_OK);

Si tu pense faire de l'unicode avec ca, tu te trompes. Si tu veux le faire (en l'occurence ca ne sert a rien puisque ca marche en ANSI), voila comme il faut faire:
MessageBoxW(hwnd,L"Ce fichier n'existe pas !  ", L"Attention !", MB_ICONWARNING|MB_OK);
Les chaines unicode sont à précéder d'un L, ne par caster directement en WCHAR*

_____________________________________
Un éditeur de ressources gratuit pour Windows

acryline · Answer

Ah ! ok, je me demandais justement si le programme passait toujours par là ? 

Mais je fais comment pour mettre  temoins.couv  à  TRUE et temoins.liste à FALSE au début ?

vecchio56 · Answer

Tu le fais dans le WM_CREATE par exemple.
temoins est a mettre en static.

Tu peux aussi faire ca:
TEMOINS temoins={TRUE};
ce qui va mettre couv a TRUE et liste à FALSE

_____________________________________
Un éditeur de ressources gratuit pour Windows

acryline · Answer

Voilà j'ai écrit :  static TEMOINS temoins={TRUE,FALSE};
et maintenant ça marche !!!

Merci mille fois !!!

acryline · Answer

Ok je le fais dans CREATE plutôt alors...

vecchio56 · Answer

Tu peux enlever le FALSE, il est implicite

_____________________________________
Un éditeur de ressources gratuit pour Windows

acryline · Answer

mais je compte ajouter d'autres temoins ...

A quoi ça sert les "réponse acceptée !" ?

vecchio56 · Answer

Ca sert a dire que le message a résolu ton problème, pour que les gens qui ont le même problème puisse aller direct à la solution

_____________________________________
Un éditeur de ressources gratuit pour Windows

acryline · Answer

Ok alors super c'est vrai mon problème est résolu. C'est vraiment très sympa de ta part d'y avoir passé autant de temps.

J'avais une idée sur une sujet :je j'ai rien trouvé de bien clair pour une débutant concernant la création de messages sur internet... peut être que tu sais où je pourrais trouver ça, où alors c'estune idée de tuto ? 

Encore un grand merci  vecchio56  !

vecchio56 · Answer

Qu'est ce que tu entends par "création de messages sur internet"

_____________________________________
Un éditeur de ressources gratuit pour Windows

acryline · Answer

Nan , je me suis mal exprimée lol
Je voulais dire que je n'ai rien trouvé sur internet sur la création de messages perso style (à ce que j'ai pu comprendre) WM_USER+x

La déclaration : où , comment ?
L'histoire de la place prise par d'autres application et registerMessageWindow etc...
A quoi ça sert, comment les utiliser. 

J'ai lu pas mal de choses mais ça reste très flou.

vecchio56 · Answer

Quand j'ai besoin de définir un message perso, je fais juste un #define WM_MESSAGEPERSO (WM_USER + X)
RegisterWindowMessage: jamais utilisé, je connais pas
Tu cherches à faire quoi?

_____________________________________
Un éditeur de ressources gratuit pour Windows

acryline · Answer

Je voulais envoyer un message d'une fenêtre fille vers la fenêtre parent pour lui dire de se redessiner et ouvrir une autre fenêtre fille après la destruction de cette fenetre fille... une histoire de fenêtres quoi lol

Alors comme je n'arrivais pas à faire passer ni WM-PAINT ni WM_CREATE je voulais créer un nouveau message....j'ai vraiment tout essayé.

A la fin j'ai décidé de n'utiliser qu'une seule fenêtre comme tu l'as vu , d'où le problème qui m'a amenée ici...

vecchio56 · Answer

Tu envoie WM_USER et c'est bon, non?

_____________________________________
Un éditeur de ressources gratuit pour Windows

acryline · Answer

Nan c'était pas bon, mais je ne sais pas l'utilser et je devais avoir fait d'autres erreurs comme celle que tu  viens de corriger ... j'apprends !

racpp · Answer

Juste après les includes tu fais:
#define MONMESSAGE  WM_USER+1

Puis quand tu veux l'envoyer tu fais:
SendMessage(HWND,MONMESSAGE,0,0);

Pour le recevoir dans la WndProc  tu fais:
case MONMESSAGE:
    // traitement du message.....
    return 0;

racpp · Answer

Personnellement, je préfère utiliser WM_APP au lieu de WM_USER car il parait que certains controles se servent du dernier.

acryline · Answer

Merci 

Mais je n'ai pas vraiment saisi l'utilité de ce message WM_USER ou APP. Je sais que c'est utile pour passer d'un thread à un autre mais comme un thread pour moi c'est une sujet de forum ça ne m'avance pas !

Est ce qu'on peut l'utiliser pour passer d'une fenêtre fille à une fenêtre parent ? Est ce que tu as un exemple simple qui me ferait comprendre son utilité et me rendre accro au messages perso ?

Dernière petite question :  si on veut faire passer une valeur dans ce message est ce qu'on peut utiliser lParam et wParam ?

C'est génial de trouver des personnes qui répondent vite et bien ! J'abuse peut être ??

En tout cas merci encore et encore ...

racpp · Answer

Bien sûr tu peux envoyer ce message de n'importe quel endroit de ton code. wParam et lParam sont là pour passer des valeurs, des pointeurs ou tout ce que tu veux.
Je vais regarder certaines de mes sources où j'utilise ce genre de messages.

racpp · Answer

Voici un exemple:
http://www.cppfrance.com/code.aspx?ID=34188

Les messages sont définis dans "notify.h".
Les messages sont envoyés à notre fenêtre, à partir d'une classe de gestion des événements.

vecchio56 · Answer

Je ne saisis pas bien la différence entre WM_USER et WM_APP: WM_USER through 0x7FFF Integer messages for use by private window classes. WM_APP through 0xBFFF Messages available for use by applications. En gros si c'est une classe de fenêtre que tu as créée (avec RegisterClassEx), tu peux lui envoyer WM_USER sans souci. Par contre on ne doit pas l'envoyer à une DialogBox par exemple? _____________________________________ Un éditeur de ressources gratuit pour Windows

racpp · Answer

Dans une de mes applications, j'avais remarqué des comportements bizarres. J'ai longtemps cherché l'erreur dans le code sans y parvenir. C'est en soupçonant l'utilisation de WM_USER que j'ai réussi à résoudre le problème. J'ai commencé par changer les valeurs de mes messages et là j'ai découvert que tout devient normal. En cherchant à comprendre, j'ai appris que la plage de valeurs de WM_USER à 0x7FFF pourraient être utilisées par des contôles comme les EDIT, LISTBOX, COMBOBOX etc. Ce problème ne se pose pas avec l'utilisation des valeurs WM_APP (0x8000-0x8FFF). Les messages définies dans cette plage ne sont valides que dans l'application. Depuis que j'ai commencé à les utiliser, plus aucun problème.

racpp · Answer

En googlant, j'ai trouvé une page dans laquelle on précise que WM_USER est obsolète:
http://wwww.developerfusion.com/show/1713/2/

acryline · Answer

Ok c'est plus clair maintenant.
En résumé : 
- je déclare dans mon header les messages :
         #define WM_MONMESSAGE  WM_APP+1
         #define WM_UNAUTREPOURLAROUTE  WM_APP+2

Dans ma procédure de fenêtre j'envoie le message :
        SendMessage (hwnd, WM_MONMESSAGE, 5," ça marche");

Et dans la procédure de l'autre fenêtre je reçois le message  et je peux retrouver wParam et lParam.

case WM_MONMESSAGE :
  if (wParam ==5)
{
// code pour afficher le contenu de lParam 
}
return 0;

acryline · Answer

Voilà j'ai fait un petit code en C qui ne tient que dans un fichier pour illustré le passage de message perso. Je l'ai dodé dans Code::Blocks et ça marche, je ne garantis rien pour les autres IDE ... 

Voici mon code :

#include <windows.h>

// Déclarer le nouveau message
#define WM_MONMESSAGE  WM_APP+1

//Boutons
#define ID_BOUTON1      110
#define ID_BOUTON2      111

//Procédures
LRESULT CALLBACK  Fille(HWND , UINT , WPARAM, LPARAM );
LRESULT CALLBACK  Procedure(HWND , UINT , WPARAM, LPARAM );
void ecrireTexte (HDC,int, int, char*,char*, long, long ,long,long,long,BYTE,BYTE,
                  BYTE,BYTE,COLORREF);

//Variable globales
HINSTANCE hinst;
HWND hwnd;

int WINAPI WinMain(HINSTANCE hinstance, HINSTANCE hPrevInstance,LPSTR lpCmdLine, int nCmdShow)
{
    //Variables de la fonction principale

    MSG msg;
    WNDCLASS wc;

    hinst = hinstance;
    // Structure de la classe de la fenêtre principale
    wc.style = 0 ;
    wc.lpfnWndProc = Procedure;
    wc.cbClsExtra = 0;
    wc.cbWndExtra = 0;
    wc.hInstance = hinst;
    wc.hIcon = NULL;
    wc.hCursor = LoadCursor(NULL, IDC_ARROW);
    wc.hbrBackground =  (HBRUSH)(1+ COLOR_BTNFACE);
    wc.lpszMenuName =  NULL;
    wc.lpszClassName = "Class";

    //Enregistrer la classe de fenêtre
    if(!RegisterClass(&wc)) return FALSE;

    hwnd = CreateWindow("Class", "Petit test message : Fenêtre parent",WS_OVERLAPPEDWINDOW   ,
                              200,50, 360, 505,NULL, NULL, hinstance, NULL);
    if (!hwnd)  return FALSE;

    ShowWindow(hwnd,SW_SHOW);
    UpdateWindow( hwnd );

    //Boucle de message
    while (GetMessage(&msg, NULL, 0, 0))
    {
        TranslateMessage(&msg);
        DispatchMessage(&msg);
    }
    return msg.wParam;
}

LRESULT CALLBACK  Procedure(HWND hwnd, UINT uMsg, WPARAM wParam, LPARAM lParam)
{

    static HWND hwndFille=NULL, hBouton=NULL;
    static HDC hdc;
    WNDCLASS wc;
    switch (uMsg)
    {
        case WM_CREATE:

                //bouton de la fenêtre parent 
                hBouton = CreateWindow ("BUTTON", "Fenêtre Fille", WS_CHILD|WS_VISIBLE|BS_PUSHBUTTON ,
                           40,20,100,30,hwnd,(HMENU)ID_BOUTON1,hinst,NULL);

                 //Créer la fenêtre fille

                wc.style = 0 ;
                wc.lpfnWndProc = Fille;
                wc.cbClsExtra = 0;
                wc.cbWndExtra = 0;
                wc.hInstance = hinst;
                wc.hIcon = NULL;
                wc.hCursor = LoadCursor(NULL, IDC_ARROW);
                wc.hbrBackground =  (HBRUSH)(1+ COLOR_BTNFACE);
                wc.lpszMenuName =  NULL;
                wc.lpszClassName = "ClasseFille";

                //Enregistrer la classe de fenêtre
                if(!RegisterClass(&wc)) return FALSE;

                hwndFille = CreateWindow("ClasseFille",
                            " Petit test messages : Fenêtre Fille",WS_OVERLAPPEDWINDOW,
                             300,200, 300, 105,hwnd, NULL, hinst, NULL);
                return 0;

        case WM_COMMAND:
           if(LOWORD(wParam) ==(LPARAM)ID_BOUTON1)
           {
           ShowWindow (hwndFille,SW_SHOW);
           }
            return 0;

        case WM_MONMESSAGE:
            if (wParam==(WPARAM)5)
            {
             hdc=GetDC(hwnd);
             ecrireTexte (hdc,20, 200, (char*)lParam,"Comic Sans MS", 22, 12,2,NULL,900,FALSE, FALSE,
                  FALSE,FALSE,RGB(255,0,0));

            ReleaseDC(hwnd,hdc);
            }
            return 0;

        case WM_CLOSE:
             DestroyWindow(hwnd);
            return 0;

        case WM_DESTROY:
            PostQuitMessage(0);
            return 0;

        default:
            return DefWindowProc(hwnd, uMsg, wParam, lParam);
     }

}

LRESULT CALLBACK Fille(HWND hwndFille, UINT uMsg, WPARAM wParam, LPARAM lParam)
{
    HWND hBoutonFille=NULL;

    switch (uMsg)
    {
        case WM_CREATE:
            //boutonFille
        hBoutonFille = CreateWindow ("BUTTON", "Envoyer WM_MONMESSAGE ", WS_CHILD|WS_VISIBLE|BS_PUSHBUTTON ,
                        40,20,200,30,hwndFille,(HMENU)ID_BOUTON2,hinst,NULL);
            return 0;

        case WM_COMMAND:
        if(LOWORD(wParam) ==(LPARAM)ID_BOUTON2)
           {
           ShowWindow(hwndFille,SW_HIDE);
           SendMessage (hwnd,WM_MONMESSAGE,(WPARAM)5,"Le message est passé !!!");
           }

            return 0;

        case WM_PAINT:
          UpdateWindow(hwndFille);
            return 0;

        case WM_CLOSE:
            ShowWindow (hwndFille,SW_HIDE);
            return 0;

        case WM_DESTROY:
            ShowWindow (hwndFille,SW_HIDE);
            return 0;

        default:
            return DefWindowProc(hwndFille, uMsg, wParam, lParam);
     }

}

// Ecrire du texte
void ecrireTexte (HDC hdc,int x, int y, char phrase [],char police[], long hauteur, long largeur,
                  long espace, long orientation,long epaisseur,BYTE italique, BYTE souligne,
                  BYTE barre,BYTE famillePolice,COLORREF couleurTexte)
{
    HFONT NewFont, OldFont;
    LOGFONT lf;
        lf.lfHeight = hauteur;
        lf.lfWidth = largeur;
        lf.lfEscapement= espace;
        lf.lfOrientation=orientation;
        lf.lfWeight=epaisseur;
        lf.lfItalic =italique;
        lf.lfUnderline=souligne;
        lf.lfStrikeOut =barre;
        lf.lfPitchAndFamily;
        lstrcpy(lf.lfFaceName,police);

     SetBkMode(hdc, TRANSPARENT);

     NewFont = CreateFontIndirect(&lf);
     OldFont = (HFONT)SelectObject(hdc,NewFont);

     SetTextColor(hdc, couleurTexte);
     TextOut(hdc, x, y, phrase, lstrlen(phrase));

     SelectObject(hdc,OldFont);
     DeleteObject(NewFont);
}

vecchio56 · Answer

C'est pas écrit dans MSDN donc c'est pas obsolète (jusque là ce sont eux qui décident, pas Joseph M. Newcomer. Mais c'est bien précisé qu'on utilise WM_USER que pour les "private window classes", ce qui exclut donc button, listbox et autres contrôles

[auteurdetail.aspx?ID=773591 acryline]> Si ton code marche avec ton compilo, il doit marcher avec tous. Et ca dépend encore moins de l'IDE

_____________________________________
Un éditeur de ressources gratuit pour Windows

racpp · Answer

Bonjour,
vecchio56 >> Moi je me suis basé sur ma propre expérience. D'ailleurs, je ne fais totalement confiance ni à MSDN ni à qui que ce soit. Dès que j'ai un problème dans le code, je soupçonne tout. Les propos de M. Joseph sont également tirés d'expériences vécues. Pour valider ces propos, je viens de trouver la page MSDN qui parle justement de conflits possibles avec les messages de certains controles:
http://msdn.microsoft.com/library/default.asp?url=/library/en-us/winui/winui/windowsuserinterface/windowing/messagesandmessagequeues/messagesandmessagequeuesreference/messagesandmessagequeuesmessages/wm_app.asp

vecchio56 · Answer

On est donc d'accord. C'est aussi cette page sur laquelle je me basais. Elle dit bien qu'on peut utiliser WM_USER mais seulement pour les fenêtres de classe privée
Il n'y a donc aucun risque à utiliser WM_USER si on le fait dans l'usage pour lequel il est prévu

_____________________________________
Un éditeur de ressources gratuit pour Windows

Win Api : redessiner une fenêtre

45 réponses

Votre réponse

Discussions similaires