THREADS : PARTAGE DE VARIABLE GLOBALE, FIN DE THREADS

Commentaire de ymca2003 le 19/01/2004 13:26:11

Salut, quelques petites remarques :

la ligne :
while(!terminer);

qui sert à attentre que le "super thread " soit terminé est à proscrire.
- tout d'abord, cette instruction consomme quasiment 100% du CPU pour rien.
- ensuite, cela ne marche pas en mode release a cause des optimisations.

en mode Debug, la variable est évaluée à chaque while mais en mode release, elle est évaluée une fois au début et puis c'est tout. On obtient une boucle sans fin (suffit de regarder le code assembleur généré) :

004010E8 : mov  al,BYTE PTR[terminer]
004010ED : test al,al      <---- test une seule fois au début
004010EF : jne  004010F3
004010F1 : jmp  004010F1   <---- boucle sans fin
004010F3 : ...

pour éviter la boucle sans fin, il suffit de déclarer la variable "terminer" en tant que "volatile" :
volatile bool terminer;
cette technique marchera en mode Release mais consomera toujours le CPU. code généré :
004010E8 : mov  al,BYTE PTR[terminer]
004010ED : test al,al
004010EF : jne  004010F3
004010F1 : mov  al,BYTE PTR[terminer]
004010F6 : test al,al       <---- test à chaque boucle
004010F8 : jmp  004010F1
004010EF : ...

La meilleur technique pour attendre q'un thread se termine tout seule est:
WaitForSingleObject(hThread,INFINITE) qui n'utilise pas le CPU.
Cependant, si le thread que l'on attent utilise "SendMessage" pour envoyer un message à une fenêtre du thread qui attend, il y a un risque de DeadLock  (le message ne sera pas traité car le thread censé le géré est en attente et le thread qui a envoyé le message attendra que le message soit traité). D'où tout le monde s'attend.


Ta variable CRITICAL_SECTION ne sert à rien (tu ne l'utilise pas). une utilisation possible serait à chaque fois que du manipule la variable texte:
EnterCriticalsection(&Sync);
strcat(texte,"....");
LeaveCriticalSection(&Sync);

Cela empêchera 2 threads différents de manipuler "texte" au même moment.

Cependant, on utilisant cette technique, TerminateThread est à proscrire car il risque d'interrompre le thread pendant qu'il est dans une section critique qui ne sera jamais relâchée.

Pour éviter d'utiliser TerminateThread, il suffit de poster un message aux boîtes de dialogues pour les fermer:
PostMessage(hDlg, WM_COMMAND,IDC_QUITTER,0);

puis d'attendre que les threads se terminent avec WaitForMultipleObject.


Conclusion : la synchronisation des threads n'est pas aisée, il faut savoir ce qu'il font à tout moment pour éviter des DeadLock.

pour en savoir plus sur les threads : JR4.zip
http://perso.wanadoo.fr/persans-brunews/brunews/download.htm