C# статический конструктор безопасности инициализации нити при заполнении ConcurrentDictionary

Question

Я звоню var person = PersonDB.pDict["395096"];

Может кто-нибудь объяснить мне, почему этот код блоки:

static class PersonDB 
{ 
    internal static readonly ConcurrentDictionary<string, Person> pDict; 

    static PersonDB() 
    { 
     pDict = new ConcurrentDictionary<string, Person>(); 
     var headers = File.ReadLines(FindPath.DataSetPerson).First().Split(';'); 


     File.ReadLines(FindPath.DataSetPerson).AsParallel().Skip(1).Select(s => s.Split(';')).ForAll(fa => 
      pDict.TryAdd(fa[0], new Person() { all = Enumerable.Range(0, fa.Length).ToDictionary(t => headers[t], d => fa[d]) }) 
     ); 
    } 
} 

sealed class Person 
{ 
    public Dictionary<string, string> all; 
} 

В то время как эта часть не блок:

static class PersonDB 
{ 
    internal static readonly ConcurrentDictionary<string, Person> pDict; 

    static PersonDB() 
    { 
     pDict = new ConcurrentDictionary<string, Person>(); 
     var headers = File.ReadLines(FindPath.DataSetPerson).First().Split(';'); 


     //File.ReadLines(FindPath.DataSetPerson).AsParallel().Skip(1).Select(s => s.Split(';')).ForAll(fa => 
     // pDict.TryAdd(fa[0], new Person() { all = Enumerable.Range(0, fa.Length).ToDictionary(t => headers[t], d => fa[d]) }) 
     //); 

     Parallel.ForEach(File.ReadLines(FindPath.DataSetPerson).Skip(1).Select(s => s.Split(';')), line => 
     { 
      pDict.TryAdd(line[0], new Person() { all = Enumerable.Range(0, line.Length).ToDictionary(t => headers[t], d => line[d]) }); 
     }); 

    } 
} 

sealed class Person 
{ 
    public Dictionary<string, string> all; 
} 

Честно говоря, я даже не уверен, что последний теперь потокобезопасен, но, по крайней мере, он работает без проблем. Я хотел бы знать, как сделать PersonDB потокобезопасным классом таким образом, чтобы не было условий гонки или тупиков. PDict необходимо создать один раз при использовании pDict. Я думал, что статический конструктор был хорошим решением для этого, но остановка выполнения в запросе PLINQ делает меня очень неуверенным ...

Answer 1

Это тупик статического конструктора. Доступ к параллельным потокам PersonDB, который блокируется до PersonDB, статически инициализируется. Переместите код инициализации в другую функцию. Заставьте его вернуть словарь вместо изменения pDict на место.

Я стараюсь избегать статических конструкторов, которые делают то, что может потерпеть неудачу. Ваш код, конечно, может выйти из строя, потому что это IO. Если это класс, он постоянно закрыт. Lazy может быть лучше.