填充ConcurrentDictionary时C#静态构造函数初始化线程安全

时间:2015-08-05 10:17:08

标签: c# multithreading thread-safety plinq static-constructor

我正在呼叫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查询的执行停止使我非常不确定......

1 个答案:

答案 0 :(得分:3)

这是一个静态构造函数死锁。并行线程访问PersonDB,阻塞直到PersonDB被静态初始化。将初始化代码移动到其他功能。让它返回字典而不是修改pDict

我试图避免做可能失败的事情的静态构造函数。您的代码肯定会失败,因为它是IO。如果确实如此,则该类被永久性地冲洗。 Lazy可以更好。