Вопрос безопасности в этом случае?

Question

Должен ли я иметь какой-либо механизм безопасности потока в следующих простых сборщиках и сеттерах для Словаря?

При необходимости, пожалуйста, примеры как для замка, так и для ConcurrentDictionary.

public virtual void Add(IFoo foo) 
    { 
     dictionary.Add(foo.name, foo); 
    } 

    public virtual IFoo Get(string name) 
    { 
     if (!dictionary.TryGetValue(name, out IFoo foo)) return null; 
     return foo; 
    } 

    public virtual bool Has(string name) 
    { 
     return dictionary.TryGetValue(name, out IFoo foo); 
    } 

    public virtual IFoo Remove(string name) 
    { 
     if (dictionary.TryGetValue(name, out IFoo foo)) 
     { 
      dictionary.Remove(name); 
     } 
     return foo; 
    } 

    protected readonly IDictionary<string, IFoo> dictionary = new Dictionary<string, IFoo>(); 

Answer 1

В соответствии с запросом, используйте поточную версию с блокировкой. Метод «Добавить» может вызывать либо GetOrAdd, либо AddOrUpdate, в зависимости от того, хотите ли вы выиграть «первый выигрыш» или «последний выигрывает» в случае разногласий.

А вот версия с ConcurrentDictionary:

public virtual void Add(IFoo foo) 
{ 
    AddOrUpdate(foo); // If you want "last one wins" 
    //GetOrAdd(foo);  // If you want "first one wins" 
} 

private virtual IFoo GetOrAdd(IFoo foo) 
{ 
    return concurrentDictionary.GetOrAdd(foo.name, foo); 
} 
public virtual void AddOrUpdate(IFoo foo) 
{ 
    concurrentDictionary[foo.name] = foo; 
} 

public virtual IFoo Get(string name) 
{ 
    IFoo result; 
    concurrentDictionary.TryGetValue(name, out result); 
    return result; // will be null if TryGetValue returned false 
} 

public virtual bool Has(string name) 
{ 
    return concurrentDictionary.ContainsKey(name); 
    // But there's no guarantee it's still there when this method returns 
} 

public virtual IFoo Remove(string name) 
{ 
    IFoo result; 
    concurrentDictionary.TryRemove(name, out result); 
    return result; 
} 

private readonly ConcurrentDictionary<string, IFoo> concurrentDictionary = new ConcurrentDictionary<string, IFoo>(); 

Answer 2

Если ваш словарь будет доступен из разных потоков, то вы должны прочитать следующее о Словаре:

Словарь может поддерживать несколько читателей одновременно, пока коллекция не изменяется. Тем не менее, перечисление через коллекцию по существу не является потокобезопасной процедурой. В редком случае, когда перечисление связано с доступом к записи, сбор должен быть заблокирован во время всего перечисления. Чтобы обеспечить доступ к коллекции несколькими потоками для чтения и записи, вы должны реализовать свою собственную синхронизацию.

Я предлагаю использовать ConcurrentDictionary<TKey, TValue> как поточно-коллекции он обеспечивает методы, которые являются атомные и поточно-:

Все эти операции являются атомарными и потокобезопасны в отношении всех других операций по класс ConcurrentDictionary. Единственными исключениями являются методы, которые принимают делегат, то есть AddOrUpdate и GetOrAdd.

проверить эту ссылку: https://msdn.microsoft.com/en-us/library/dd287191(v=vs.110).aspx

вместо того, чтобы использовать свой собственный код блокировки, однако вы можете использовать lock заявление для осуществления синхронизации потоков в случае, если вы хотите использовать Dictionary, проверьте эту ссылку: https://msdn.microsoft.com/en-us/library/c5kehkcz.aspx