Discussion :

[on2101]

Bonjour !

J'ai une petite question concernant les Critical Section.

J'ai un objet de la classe A. J'éxecute un code du type suivant dans ma boite de dialogue :

Code :

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

1
2
3
4
5
6
7
8
9
A monObjet;
CRITICAL_SECTION cs;
InitializeCriticalSection(&cs);
 
EnterCriticalSection(&cs);
 
      monObjet.UneFonction();
 
LeaveCriticalSection(&cs);

Est ce que le corps de UneFonction fait ou non partie du code protégé par la Critical Section ? Le fait que la fonction soit d'une autre classe gene t il ?
Dois je remettre des Enter / Leave dans le corps de la fonction ???

Merci pour votre aide !!!

[JulienDuSud]

Oui, c'est protégé.

[Mac LAK]

Non, il ne l'est pas : rien ne t'empêches d'appeler, ailleurs dans le code, "monObjet.UneFonction()" de façon non protégée. Dans l'exemple que tu as donné, il n'y a que cet appel particulier à UneFonction qui est sous la CS, mais pas forcément les appels concurrents.

Si tu veux le garantir, il faut, au choix :

1. Déplacer les appels à EnterCriticalSection et LeaveCriticalSection dans le corps de "UneFonction", ce qui la protègera quel que soit l'endroit ou la manière de l'appeler. Ta fonction devient en fait thread-safe.
2. Vérifier et/ou garantir que tous les appels à "UneFonction" seront bien TOUS encadrés par les appels à EnterCriticalSection et LeaveCriticalSection.
   Ta fonction n'est pas thread-safe, mais tu protèges à un niveau plus élevé tous les appels potentiellement concurrents, donc le problème de réentrance ne se produira pas.

Si ta CS doit protéger d'autres éléments, il est préférable d'utiliser la deuxième solution. Si elle ne doit protéger QUE la méthode "UneFonction", alors la première est préférable.
Tu peux même faire les deux : une fois la CS acquise par un thread, tu peux appeler autant de fois que tu veux EnterCriticalSection, cela ne provoquera pas de deadlock. Il faut juste veiller à équilibrer le nombre d'appels à LeaveCriticalSection.

[on2101]

Bonjour,
Un grand merci un peu tardif pour cette réponde claire et limpide qui m'a beaucoup aidé.