getアクセサにコレクションをロックすることはできますか？

Question

私は、個々のアイテムがイベントを発生させてその結果をイベントargsで返すので、読み込みや更新はほとんど行われない軽く使われた辞書を持っています。実際、スレッドは常に同じスレッドで更新されます。安全のために単純なロックを追加することを考えていました。私はちょうど私がアクセサを取得するにロックを配置することができますかと思っていた。これは機能しますか？

 Dictionary<string,Indicator> indicators = new Dictionary<string,Indicator>(); 
     Dictionary<string, Indicator> Indicators 
     { 
      get 
      { 
       lock (indicators) 
       { 
        return indicators; 
       } 
      } 
     } 

     public void AddIndicator(Indicator i) 
     { 
      lock (indicators) 
      { 
       indicators.Add(i.Name, i); 
      } 
     } 

Answer 1

これは特に役に立たないものではありません。特に

、あなたが持っている場合：

x = foo.Indicators["blah"] 

を、それはスレッドセーフではありませんので、その後インデクサが...ロックを保持してのないスレッドを実行されます。上記のコードを次のように考えてください：

Dictionary<string, Indicator> indicators = foo.Indicators; 
// By now, your property getter has completed, and the lock has been released... 
x = indicators["blah"]; 

インデクサー経由でアクセスする以外に何かする必要はありますか？ない場合は、あなただけの方法でプロパティを置換する場合があります

public Indicator GetIndicator(string name) 
{ 
    lock (indicators) 
    { 
     return indicators[name]; 
    } 
} 

（あなたがなど、代わりにTryGetValueを使用することをお勧めします - それはあなたが達成しようとしているかに依存します。）個人的に

コレクションの参照をロックするのではなく、私的に所有されていてロックされていないロックオブジェクトへの参照を使用することをお勧めしますが、これは別の問題です。

他の場所で述べたように、あなたは.NET 4を使用している場合ConcurrentDictionaryはあなたの友人であるが、もちろん、それはその前に利用できません:(

Jonの入力以外に

Answer 2

短い答え：はい。
Jonが言及しているように、インデックスを使用するときに意図したとおりにロックされないのはなぜですか。

Answer 3

、私はないと言うだろうMSDNから、とにかくコレクションindicators自体をロック：

ご注意くださいVisual BasicでC＃または SyncLock（Me）の中で（この）、インスタンス上の例のロックの をロックする際に、アプリケーション内の他の コード場合は、。外部の の外部にオブジェクトがロックされていると、デッドロックが発生する可能性があります（ ）。

専用のobjectインスタンスを使用してロックすることをお勧めします。これがさらに詳しい情報と理由でカバーされている場所は他にもあります。なぜなら、ここでさえ、あなたが時間があるときに情報を検索する必要があるからです。

Answer 4

また、スレッドセーフティを処理するConcurrentDictionaryを使用することもできます。