この場合、スレッドの安全性が懸念されますか？

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

要求通り、ここではロックを使用するスレッドセーフバージョンです。 Addメソッドは、競合が発生した場合に「最初の1つの勝利」または「最後の1つの勝利」を行うかどうかによって、GetOrAddまたはAddOrUpdateのいずれかを呼び出すことができます。

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

あなたの辞書は、あなたが辞書については、以下をお読みください別のスレッドからアクセスされる場合は、次の

A辞書限り、コレクションが変更されないように、同時に複数のリーダーをサポートすることができます。それでも、コレクションを列挙することは、本質的にスレッドセーフな手順ではありません。まれに列挙型が書き込みアクセスと競合する場合は、列挙全体でコレクションをロックする必要があります。読み取りと書き込みのために複数のスレッドがコレクションにアクセスできるようにするには、独自の同期を実装する必要があります。

すべてのこれらの操作はアトミックであり、上の他のすべての操作に関して、スレッドセーフです：

私はそれは、原子およびスレッドセーフなメソッドを提供してスレッドセーフなコレクションとしてConcurrentDictionary<TKey, TValue>を使用することをお勧めConcurrentDictionaryクラス唯一の例外は、代理人を受け入れるメソッド、つまりAddOrUpdateとGetOrAddです。

チェックこのリンク：https://msdn.microsoft.com/en-us/library/c5kehkcz.aspx

： https://msdn.microsoft.com/en-us/library/dd287191(v=vs.110).aspx

の代わりに、独自のロックコードを使用するが、しかし、あなたは、このリンクをチェックし、Dictionaryを使用したい場合は、スレッドの同期を実現するためにlock声明 を使用することができます