Question

我有一个函数，它根据Key（名称）返回字典上的条目，如果它不存在，则返回一个新创建的条目。

我的问题是“双锁”：SomeFunction锁定_dictionary，检查键的存在，然后调用一个也锁定同一个字典的函数，它似乎工作但我不确定是否这种方法存在潜在的问题。

public Machine SomeFunction(string name) 
{
    lock (_dictionary)
    {
        if (!_dictionary.ContainsKey(name))
                    return CreateMachine(name);
        return _dictionary[name];
    }
}


private Machine CreateMachine(string name)
{
    MachineSetup ms = new Machine(name);
    lock(_dictionary)
    {
        _ictionary.Add(name, ms);
    }
    return vm;
}

Answer 1

确保工作 - 在.NET中递锁是递归的。这是否真的是一个好主意是一个不同的问题......相反如何：

public Machine SomeFunction(string name) 
{ 
    lock (_dictionary)
    {
        Machine result;
        if (!_dictionary.TryGetValue(name, out result))
        {
            result = CreateMachine(name);
            _dictionary[name] = result;
        }
        return result;
    } 
}

// This is now *just* responsible for creating the machine,
// not for maintaining the dictionary. The dictionary manipulation
// is confined to the above method.
private Machine CreateMachine(string name)
{
    return new Machine(name);
}

Answer 2

这里没问题，锁是由同一个线程重入的。并非所有同步对象都具有线程亲和性，例如Semaphore。但是Mutex和Monitor（锁定）很好。

Answer 3

自.net 4.0以来的新版本，请查看ConcurrentDictionary - ConcurrentDictionary是一个线程安全的键/值对集合，可以由多个线程同时访问。有关详情，请访问https://msdn.microsoft.com/en-us/library/dd287191(v=vs.110).aspx。

在同一个线程内锁定字典

3 个答案: