Accès à la question de fermeture modifiée

Question

J'ai eu un peu de recherche, mais je ne pouvais pas trouver la même situation que je suis confronté ici.

est ici l'extrait que je suis inquiet:

var threadsafeVacantStats = new Dictionary<Thread, StatsNode>(); 
    var threadSafeVacantLocker = new Object(); 
    var threadsafeZoneStats = new Dictionary<Thread, StatsNode>(); 
    var threadSafeZoneLocker = new Object(); 
    Parallel.ForEach(m_TeamAreasByZone[zone.ContainerID], team => 
    { 
     var tempVacantStatNode = CreateASyncStatsNodes(threadSafeVacantLocker, threadsafeVacantStats); 
     var tempZoneStatNode = CreateASyncStatsNodes(threadSafeZoneLocker, threadsafeZoneStats); 
     //...other stuff 
    } 

est ici la fonction qu'il appelle:

private static StatsNode CreateASyncStatsNodes(object threadSafeLocker, Dictionary<Thread, StatsNode> threadsafeTeamStats) 
{ 
    StatsNode tempStatsNode; 
    var currentThread = Thread.CurrentThread; 
    lock (threadSafeLocker) 
    { 
     if (!threadsafeTeamStats.ContainsKey(currentThread)) 
      threadsafeTeamStats[currentThread] = new StatsNode(0, 0); 

     tempStatsNode = threadsafeTeamStats[currentThread]; 
    } 
    return tempStatsNode; 
} 

Pour moi, cela semble bien, cependant ReSharper donne un avertissement pour le second appel à CreateASyncStatsNodes (le premier appel est bien). Après son avis, il transforme le bloc en:

var threadsafeVacantStats = new Dictionary<Thread, StatsNode>(); 
var threadSafeVacantLocker = new Object(); 
var threadsafeZoneStats = new Dictionary<Thread, StatsNode>(); 
var threadSafeZoneLocker = new Object(); 
object locker = threadSafeZoneLocker; 
Dictionary<Thread, StatsNode> stats = threadsafeZoneStats; 
Parallel.ForEach(m_TeamAreasByZone[zone.ContainerID], team => 
    { 
     var tempVacantStatNode = CreateASyncStatsNodes(threadSafeVacantLocker, threadsafeVacantStats); 
     var tempZoneStatNode = CreateASyncStatsNodes(locker, stats); 
     //... 
    } 

Cela ne me fait pas vraiment de sens, et ne semble pas vraiment fixer quoi que ce soit (si elle a même été cassé en premier lieu) . Resharper est-il en train de marquer par erreur un avertissement ou est-ce que je manque quelque chose?

Answer 1

Resharper ne sait pas que Parallel.ForEach execute passé lambda immédiatement. Il suppose que ce lambda pourrait être exécuté plus tard, quand la fermeture sera modifiée et cela pourrait causer quelques problèmes. Vous pouvez ignorer cet avertissement.

Answer 2

Quel est exactement l'avertissement que ReSharper vous donne? Gardez à l'esprit que l'analyse statique n'est pas parfaite, je dirais qu'il est très probable que ReSharper commette une erreur. Le code original me va bien.