我有一个函数返回一个字典上的一个条目,基于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;
}
解决方法
这是保证工作 – 锁在.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);
}
