euhhh ... :-) desolé, c'est compliqué a expliquer...Résolu

Question

salut

c'est assez compliqu&#233; a expliquer donc je met une source, ca parle mieux !



(API win32)



tout le tralala de la creation de la fenetre etc...

            TranslateMessage(&msg);

            DispatchMessage(&msg);

// un des deux appelle le module d'en dessous (si j'ai bien compris...)





LRESULT CALLBACK WinProc(HWND hwnd, UINT msg, WPARAM wParam, LPARAM lParam)

{

   int i;

      switch (msg)

      {

            case WM_TIMER: // ce n'est pas le message qui est important

                  switch (i)

                      {

                                case 1:

               
         // instructions

                                i++;

                               break;

                                case 2:

               
         // instructions

                                i++;


                               break;

                                // etc...

                        }

       }

       return 0;

}



voila, mon probleme c'est que ... en fait je n'arrive pas a garder la bonne valeur de i... 

a chaque fois que le programme arrive a WinProc, i se reinitialise
(normal ! enfin il me semble, vu que c'est une variable d'un module)

il retourne 0, et lorsqu'il revient, i a une valeur differente de celle d'avant...

je sais pas si vous comprenez... (j'explique tres mal je le sais, d&#233;sol&#233;...)



je voudrais pouvoir garder la valeur de i apres avoir retourn&#233; 0 (tout simplement !)



merci et bravo a ceux qui m'ont compris !

meech · Accepted Answer

Salut,



En effet, pas tout compris... 



En fait, il me semble que la fonction WinProc est charg&#233;e de traiter les &#233;v&#232;nements (quand on clique quelque part, qu'on d&#233;place la fen&#234;tre, qu'on appuie sur une touche, etc.).



Si tu souhaites conserver la valeur de i, il faudrait mieux que cette
variable soit d&#233;clar&#233;e de mani&#232;re globale (en dehors de la fonction) et
non localement (dans la fonction).



En fait, essaie de placer ta d&#233;claration int i
dans un fichier d'en-t&#234;tes (ou au-dessus de toutes tes fonctions), en
pr&#233;cisant &#233;ventuellement le mot-clef static... L'incr&#233;ment de ton i devrait alors se r&#233;aliser correctement lors de chaque appel d'un &#233;v&#232;nement Win32.



#include <windows.h>



// d&#233;claration variabale globale i.

int i = 0;



// fonction WinMain...



// fonction WinProc... 

LRESULT CALLBACK WinProc(...)

{

// (!) ne pas d&#233;clarer de nouveau la variable i 

switch(msg)

  {

     case WM_TIMER:

       switch(i)

       {

          case 2:

            i++;

          case 3:

            i++;

          // (...)

       }

  }

}





J'esp&#232;re avoir compris le but et t'avoir un peu aid&#233;.



NB. Profites-en pour initialiser une bonne fois pour toutes cette variable &#224; z&#233;ro (par exemple, au d&#233;marrage de l'application).

cs_Joky · Answer

Tu lances ton Timer au moins ??? lol

Balance tout le code si c'est petit parce que c'est vrai que t'explique mal lol :p

void Aurevoir( void ); //Bonne journ&#233;e

cs_Joky · Answer

Ah j'viens de voir que le message n'est pas important :)

D&#233;sol&#233; j'lis toujours tout en 2 fois

void Aurevoir( void ); //Bonne journ&#233;e

BruNews · Answer

ben non, i est dans la wndproc donc initialis&#233; "nouveau" &#224; chaque appel de cette wndproc.
Mets i en variable globale, il conservera sa valeur.

ciao...
BruNews, MVP VC++

cs_Joky · Answer

et si on fait

static int i;



et une initialisation dans WM_CREATE, ou WM_INITDIALOG ?

void Aurevoir( void ); //Bonne journ&#233;e

BruNews · Answer

vi vi c'est bon Joky ainsi mais ce n'est qu'une variante de syntaxe pour &#233;viter qu'on y retouche ailleurs dans le code (hors de la wndproc) sinon le compilo g&#233;n&#232;rera le m&#234;me code que si on met i en var globale.

ciao...
BruNews, MVP VC++

BertaNd · Answer

"// d&#233;claration variabale globale i.

 int i = 0;"



ok merci ! 

c'est tout simple !!!

c'est juste parce que je pensais que les variables globales ne pouvait pas etre chang&#233;es...



j'ai confondu avec les constantes...



enfin bon ! il est quand meme 1h du matin, je suis plus tres frais !

meech · Answer

BertaNd,



Ca serait sympa d'accepter ma r&#233;ponse 



Peace.

BruNews · Answer

ok meech, je te l'ai valid&#233;e.

ciao...
BruNews, MVP VC++

xterminhate · Answer

static me semble la meilleure solution, pour garantir une bonne encapsulation des données.

magic_Nono · Answer

BN, Joky , XTH,...> 

parfaitement d'accord avec [auteurdetail.aspx?ID=213313 xterminhate]


static me semble bcp plus propre, je suis de l'&#233;cole &#224; proscrire au max les var globales



mm si le code g&#233;n&#233;r&#233; est le mm,

les protection pr&#233;compil me semble plus fortes et plus protectrices



y a d&#233;j&#224; suffisement d'err possible ds un code ....

___________________________________________________________
Magicalement
Nono

racpp · Answer

Salut,

Moi aussi je pr&#233;f&#232;re la solution du static. On met juste static int i =0; au d&#233;but de la WindProc.

Euhhh ... :-) desolé, c'est compliqué a expliquer...

12 réponses

Votre réponse