Verrouillage pour modifier le membre du type de valeur statique. Est-ce nécessaire?

Question

J'ai une classe CacheHelper pour faciliter l'interaction avec le cache. Je veux utiliser un champ static int pour spécifier le délai d'attente de mon cache. Le champ est initialement défini sur une valeur par défaut const, mais je souhaite fournir à l'application un moyen de modifier la valeur de délai d'attente par défaut.

Avez-vous besoin de verrouiller lors de la modification d'un type de valeur statique? Le verrouillage du setter est-il nécessaire? Y a-t-il d'autres problèmes que vous pouvez voir ici? Désolé, je suis encore assez stupide quand il s'agit de multithreading.

Merci.

public static class CacheHelper 
{ 
    private static object _SyncRoot; 
    private static int _TimeoutInMinutes = CacheDefaults.TimeoutInMinutes; 

    public static int TimeoutInMinutes 
    { 
     get 
     { 
      return _TimeoutInMinutes; 
     } 
     set 
     { 
      lock (_SyncRoot) 
      { 
       if (_TimeoutInMinutes != value) 
       { 
        _TimeoutInMinutes = value; 
       } 
      } 
     } 
    } 

    public static void Insert(string key, Object data) 
    { 
     if (HttpContext.Current != null && data != null) 
     { 
      HttpContext.Current.Cache.Insert(key, data, null, Cache.NoAbsoluteExpiration, TimeSpan.FromMinutes(CacheHelper.TimeoutInMinutes)); 
     } 
    } 
} 

Answer 1

Vous pouvez utiliser une variable volatile à la place ... mais vous avez besoin quelque chose, sinon il est possible qu'une valeur écrite par un fil ne serait jamais vu par un autre.

Notez que pour les « grands » types tels que double ou long vous vraiment devez utiliser une serrure ou la classe Interlocked, que les modifications à ces valeurs peuvent ne pas être atomique.

Answer 2

Vous n'avez pas besoin de verrouiller ici si le client de CacheHelper ne somthing comme

CacheHelper.TimeoutInMinutes = input.Value; 

Comme il ne repose pas sur la valeur précédente.

Si votre client fait quelque chose comme

CacheHelper.TimeoutInMinutes += input.Value; 

Ensuite, vous aurez besoin de faire quelques verrouillage

.