Blokowanie słownika w obrębie tego samego wątku

Question

Mam funkcję, która zwraca pozycję w słowniku, na podstawie klucza (nazwa), a jeśli nie istnieje, zwraca nowo utworzony.

Mam pytanie z "podwójnym zamkiem": SomeFunction blokuje _dictionary, aby sprawdzić istnienie klucza, a następnie wywołuje funkcję, która również blokuje ten sam słownik, wydaje się działać, ale nie jestem pewien jeśli istnieje potencjalny problem z tym podejściem.

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

To gwarantowane działanie - blokady są rekurencyjne w .NET. Czy to naprawdę dobry pomysł, czy nie to inna sprawa ... jak to zamiast o:

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

No Problem, blokada zostanie ponownie uczestnik tego samego wątku. Nie wszystkie obiekty synchronizacji mają powinowactwo do wątków, na przykład Semafor. Ale Mutex i Monitor (blokada) są w porządku.

Answer 3

Nowy od .net 4.0, sprawdź ConcurrentDictionary - ConcurrentDictionary to bezpieczny dla wątku zbiór par klucz/wartość, do których można jednocześnie uzyskać dostęp z wieloma wątkami. Więcej informacji pod adresem https://msdn.microsoft.com/en-us/library/dd287191(v=vs.110).aspx.