Discussion :

[theclem35]

Salut,

J'ai un programme multifenetres avec l'api win32, et dans l'une d'elles je cree un thread lors d'un clic sur un bouton (fonction callback) avec le code suivant :

Code :

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

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
void GestBoutonsFHebergerPartie(HWND fenetre,UINT message, WPARAM wParam, LPARAM lParam)
{
    static InfosPartie MaPartie; // On met les variables du thread en memoire car on va sortir de cette fonction et donc les perdre
    static HANDLE hdlThreadServeur; // Le thread va donc recevoir n'importe quoi puisque c'était un pointeur!
    static DWORD CodeRetourThread;
 
    switch(LOWORD(wParam)) // Quand btn enfoncé, une partie de wParam vaut Id du bouton => recup avec LOWORD()
    {
    case ID_LANCER_SERVEUR:
 
               strcpy(MaPartie.NomPartie, NomPartie); // On remplie la structure qu'on enverra sur le reseau en reponse aux broadcasts
               strcpy(MaPartie.IPServeur, IPChoisie);
 
               hdlThreadServeur = CreateThread(NULL, 0, HebergementPartie, &MaPartie, 0, NULL); // On lance le thread pour une ecoute UDP en attente de demande d'infos, ou de connexion
               // faire une animation d'attente
 
               GetExitCodeThread(hdlThreadServeur, &CodeRetourThread); // Puis on recupere le code de retour du thread
 
               if(CodeRetourThread == -1) // ne fonctionne pas
               {
                                   MessageBox(NULL,"Erreur dans le thread","Erreur !",MB_OK | MB_ICONWARNING);
 
               }
         }
        break;
 
    }
}

Seulement, mon thread met quelques temps à s'executer, et le programme pendant ce temps suit son cours et sort de ma fonction callback de traitement des boutons.

Donc, je ne sais pas où placer mon appel GetExitCodeThread(hdlThreadServeur, &CodeRetourThread); pour savoir si le thread s'est bien terminé ou si on en est sorti de facon prématurée.

Pouvez vous m'aider ?

[homeostasie]

Salut,


GetExitCodeThread() retournera la valeur STILL_ACTIVE si le thread n'est pas encore terminé.

Afin d'attendre la fin d'exécution de ton thread, tu peux utiliser l'API WaitForSingleObject().

Ca pourrait être un truc du genre:

Code :

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

1
2
3
4
5
if(WaitForSingleObject(hdlThreadServeur, 0) == WAIT_OBJECT_0)
{
    // Thread terminé: récupération de la valeur de retour
    ...
}

;-)

[theclem35]

Oui mais où mettre cela dans mon programme ?

Car une fois que j'ai appuyé sur le bouton le thread se lance, ca c'est bien.

Mais moi je voudrais que mon programme principal continue à s'exécuter et pouvoir intéragir avec lui en cliquant sur des boutons par exemple (dont un bouton ANNULER par exemple)

Hors si par exemple je rajoute ce bout de code apres la création de mon thread :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
               hdlThreadServeur = CreateThread(NULL, 0, HebergementPartie, &MaPartie, 0, NULL); // On lance le thread pour une ecoute UDP en attente de demande d'infos, ou de connexion
               // faire une animation d'attente
 
 
               while(1)
               {
                     GetExitCodeThread(hdlThreadServeur, &CodeRetourThread); // Puis on recupere le code de retour du thread
 
                     if (CodeRetourThread == 1)
                     {
                          MessageBox(NULL,"Pas de probleme thread termine OK","OK",MB_OK | MB_ICONWARNING);
                           // On ferme toutes les fenetres et on demarrer la partie
                          break;
                     }
                     else if (CodeRetourThread == -1)
                     {
                          MessageBox(NULL,"Erreur dans le thread","Erreur !",MB_OK | MB_ICONWARNING);
                          break;
                     }
               }

Et bien le programme reste bloqué en attendant la fin de mon thread.
Il faudrait que mon thread envoie lui meme un message à ma fenetre (dans le meme style que les WM_CREATE, WM_COMMAND etc... avec son code de retour... Et la je seche ...

[theclem35]

C'est bon j'ai solutionné mon probleme grace aux timers.

Maintenant il me reste a recuperer le code de sortie du thread avec :
GetExitCodeThread(hdlThreadServeur, &CodeRetourThread);

La probleme c'est que j'execute cette fonction en dehors de celle ou j'ai créé le thread.

Ce que je voudrais donc savoir, c'est quelle est la fonction qui me permettrait de récuperer le handle du thread que j'ai créé ?
Il y a FindWindow() pour trouver le handle d'une fenetre, mais apparament FindThread n'existe pas !

Quelqu'un saurait comment faire ca ?

[ManusDei]

http://msdn.microsoft.com/en-us/library/ms684847.aspx


GetCurrentThread à partir du thread ou OpenThread si ça vient du programme principal.

Mais bon, CreateThread te renvoie le HANDLE du Thread créé, donc normalement tu l'as qui traine quelque part (dans hdlThreadServeur).

[theclem35]

Oui mais GetCurrentThread renvoie la valeur du thread depuis lequel il est executé, donc le thread principal de mon programme, et c'est pas ce que je veux (c'est le nouveau que je viens de creer donc pas le current)

Pour mon handle oui je l'ai deja mais il est dans une autre fonction, donc a part le passer en variable globale je vois pas comment je peux faire pour l'avoir. (Et j'aime pas trop les variables globales)

Je ne peux pas le passer en parametre a ma fonction qui appelle le code de retour, puisque c'est une fonction callback de traitement des messages de ma fenetre!

[sam1507]

Il faut séparer tout ce qui est gestion de l'interface graphique du reste.

Code de l'interface graphique :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
void GestBoutonsFHebergerPartie(HWND fenetre,UINT message, WPARAM wParam, LPARAM lParam)
{ 
    switch (LOWORD(wParam)) // Quand btn enfoncé, une partie de wParam vaut Id du bouton => recup avec LOWORD()
    {
        case ID_LANCER_SERVEUR :
            if (!DemarrerServeur())
            {
                /* Afficher un message d'erreur */
            }
            break;
 
        default :
            /* Afficher un message d'erreur même si on devrait jamais être dans ce cas là ! */
            break;
     }
}

Code de gestion du serveur :

Code :

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

1
2
3
4
5
6
7
8
9
BOOL DemarrerServeur()
{
    /* Ici, on peut améliorer la fonction en récupérant en paramètres les
informations en provenance de l'interface graphique (nom de partie, etc.)
et en les stockant dans une file (auquel cas le thread de gestion du serveur
sera réveillé sur l'évènement signalant la présence de données dans la file) */
 
    return SetEvent(g_hEvents[EVT_DEMARRER_SERVEUR]));    
}

Code de gestion du serveur (même module) :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
 
dWait = WaitForMultipleObjects(NB_EVENTS, g_hEvents, FALSE, 5000);
 
...
 
else if (dwWait == (WAIT_OBJECT_0 + EVT_DEMARRER_SERVEUR))
{
     /* Créer le thread (et d'abord lire les données dans la file si on a opté pour cette solution) */
     hThread = CreateThread(...);
}
 
else if (dwWait == (WAIT_OBJECT_0 + WAIT_TIMEOUT))
{
     if (WaitForSingleObject(hThread, 0) == WAIT_OBJECT_0)
     {
         GetExitCodeThread(hThread, &dwExitCode);
 
         ...
     }
}
 
...

Voilà (en gros) comment je le ferais. Peut-être qu'il y a des erreurs et que cette solution n'est pas idéale.

Attention, tous les codes de retour des fonctions n'ont pas été testés dans mon exemple. Loin de là !

[theclem35]

Ok merci pour ta proposition je vais tester tout ca!!!