Blocca il dizionario all'interno della stessa discussione

Question

Ho una funzione che restituisce una voce su un dizionario, basata sulla chiave (nome) e, se non esiste, ne restituisce una appena creata.

La domanda che ho è con il "doppio blocco": SomeFunction blocca il _dictionary, per verificare l'esistenza della chiave, quindi chiama una funzione che blocca anche lo stesso dizionario, sembra funzionare ma non sono sicuro se c'è un potenziale problema con questo approccio.

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

Questo è garantito per funzionare - i blocchi sono ricorsivi in. NET. Che si tratti di una buona idea o no è una questione diversa ... che ne dite di questo, invece:

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

Nessun problema qui, il blocco è rientrante dallo stesso thread. Ad esempio, non tutti gli oggetti di sincronizzazione hanno affinità di thread, semaforo. Ma Mutex e Monitor (lock) vanno bene.

Answer 3

Nuovo da .net 4.0, consultare ConcurrentDictionary - ConcurrentDictionary è un insieme di coppie chiave/valore accessibile da più thread contemporaneamente. Maggiori informazioni allo https://msdn.microsoft.com/en-us/library/dd287191(v=vs.110).aspx.