Discussion :

[yridwenn]

Bonjour,

J'ai un peu de mal à comprendre quel est le sens de l'objet que l'on doit passer à l'instrcution lock ?

Dans mon cas, j'ai une classe qui réalise des appels à des procédures stockées. Toutes ces fonctions sont en static.

Dans mon thread, j'aimerais que l'appel à une procédure stockée particulière soit protégée des autres threads.

Ma première idée était de mettre le lock dans l'appel de la procédure stockée, juste autour de l'exécution pour limiter le nombre de lignes de code vérouillées.
Dans beaucoup d'exemple que j'ai trouvé, l'instruction "lock" s'appliquait à l'objet "this". Or, dans une fonction statique je ne peux pas utiliser "this"!

Voici ma fonction statique avec ce que j'ai voulu faire :

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
 
class IAlimDB
    {
 
        /// <summary>
        /// Exécute la procédure stockée "lockCalcCube"
        /// </summary>
        /// <param name="IdTraitement">Nom du cube à locker</param>
        /// <param name="IdConso">Nom du thread réalisant le lock</param>
        /// <returns></returns>
        public static int ExecLockCalcCube(String CalcCube, String ThreadName)
        {
...
 
                    lock (this)
                    {
                        Connect.Open();
                        nb = (int)Cmd.ExecuteNonQuery();
                        res = Convert.ToInt32(Cmd.Parameter.["ReturnValue"].Value);
                    }
...
        }
 
 
    }

Puis-je mettre le "lock" sur un autre objet que "this", par exemple "cmd" ?
Ou alors, dois-je mettre le lock sur la classe appelant la fonction statique :

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
 
 
    class EssbaseLoad
    {
 
        public void LoadConso()
        {
                int res = 0;
 
                lock (this)
                {
                    res = IAlimDB.ExecLockCalcCube(Calc_Cube, _IdTraitement + "_" + _IdConso)
                }
 
        }
 
     }

Je n'arrive pas à tranposer les exemples que j'ai trouvé à mon cas parce que je n'arrive pas à comprendre comment on choisit l'objet sur lequel appliquer le lock.

Merci de votre aide!
Yridwenn.

[SaumonAgile]

Tu peux utiliser une unique variable static dédiée dans ta classe. Dans toutes tes méthodes qui en ont besoin tu lock cette variable.
C'est pratique aussi dans le cas où tu voudrais protéger plusieurs ressources, ça évite les deadlock.

[Sphax]

Salut,

Utiliser un lock sur le "this" est une mauvaise pratique. Car le "this" se réfère à l'objet en cours et qu'il est dont accessible depuis l'extérieur de ton objet, avec risque de deadlock éventuel. Il faut locker sur un objet privé que personne d'autre ne pourra locker.

Puis-je mettre le "lock" sur un autre objet que "this", par exemple "cmd" ?
Ou alors, dois-je mettre le lock sur la classe appelant la fonction statique :

Tu peux effectuer un lock sur n'importe quel objet, mais il faut que ça ait du sens: un lock sur un objet local n'en a aucun, puisque cet objet local sera recréé lors du prochain appel de ta fonction et que le lock pourra donc être repris. Pour que plusieurs threads ne puissent accéder au même code en même temps il faut locker une variable commune à ces threads. Pour synchroniser une méthode statique, rien de tel qu'une variable statique .

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
class IAlimDB 
{   
   private static object _Loackable = new object();
   public static int TaFonction()
   {
      lock(_Loackable)
      {
             //le code
      }
   }
}

Un trés bon article sur la synchronisation en .net ici.

[yridwenn]

Merci beaucoup pour les explications et pour l'article!

C'est beaucoup plus clair quand on sait comment ça marche...

Yridwenn.