はこのコードのスレッドセーフですか？

Question

私は、このコードがスレッドセーフであるかどうか、なぜそうでないのかを知りたいと思います。

static IMyInterface _myinterface; 
     public static IMyInterface someStuff 
     { 
      get 
      { 
       if (_myinterface== null) 
       { 
        _myinterface= MyServiceLocator.GetCurrent().GetInstance<IMyInterface>(); 
       } 

       return _myinterface; 
      } 
     } 

ありがとうございます！

Answer 1

いいえあなたは、ロックを使用する必要があります。

private static readonly object m_lock = new object(); 
private static IMyInterface _myinterface; 

public static IMyInterface someStuff 
{ 
    get 
    { 
     lock(m_lock) 
     { 
      if (_myinterface == null) 
      { 
      //create instance 
      } 

      return _myinterface; 
     }    
    } 
} 

Answer 2

私にとってはそうではありません。 ロックを確実に追加する方法があるからです。 if文に2つのスレッドがある可能性があります。

http://msdn.microsoft.com/en-us/library/c5kehkcz%28v=vs.80%29.aspx

Answer 3

いいえ、これ以上のことをしたいと思います。 _myinterface==nullのダブルチェックに注目してください。これは、最初のチェックの後で別のスレッドがすでにロックを作成しているためです。だからあなたはあなたのロックに達するとすぐにダブルチェックする必要があります。

static IMyInterface _myinterface; 
private static object lockObj = new object(); 
public static IMyInterface someStuff 
{ 
    get 
    { 
     if (_myinterface== null) 
     { 
     lock(lockObj) 
     { 
      if (_myinterface== null) 
      {    
       _myinterface= MyServiceLocator.GetCurrent().GetInstance<IMyInterface>(); 
      } 
     } 
     } 
     return _myinterface; 
} 

Answer 4

いいえ、そうではありません。

なぜですか？

someStuffに2つのコールが入っています。if(_myInterface null)を取得しましたが、中断されました。_myInterface = MYServiceLoc...です。 2番目のスレッドはすべて終了し、スリープ状態に入る前に完了します。第2の目覚めは、_myInterfaceに割り当てます。

MyServiceLocator ...でGetInstance（）を呼び出してしまう、両方_myInterfaceに割り当てる、との両方が_myInterfaceに異なる値を返すことができます両方。

Answer 5

シングルトンに同時にアクセスしようとする複数の初期化スレッドが存在する可能性があるためではありません。

これは仕事ができる：

static object _lock=new object(); 
    static IMyInterface _myinterface; 
    public static IMyInterface someStuff 
    { 
     get 
     { 
      lock (_lock) { 
       if (_myinterface== null) 
       { 
        _myinterface= MyServiceLocator.GetCurrent().GetInstance<IMyInterface>(); 
       } 
      } 
      return _myinterface; 
     } 
    } 

Answer 6

はありません、2つの異なるスレッドが異なるインスタンスをインスタンス化することが起こると_ myInterfaceの前回保存した値をowerwritesことができます。

ちょうどそのスレッド＃1はifブロック

if (_myinterface== null)     
{ 
// ... here 

を入力しても_ ​​myinterfaceを読み、今回のスレッド＃1インスタンス化されたオブジェクトと_myinterfaceの更新を参照して、あまりにもこのブロックに入る＃2スレッド想像してみてください。 同時に、スレッド＃2はすでに_myinterfaceの値をキャッシュしています（NULLです）。スレッド＃1に格納されている新しい1つのオブジェクトとowerwriting値を作成しています。

ロックによってラップゲッター（）ので、一つだけのスレッドが他の人のよう（ダブルロックに関する他の回答を参照してください）で

Answer 7

を入力することができ、それはない、と述べています。ただし、プレーンロックやダブルチェックパターンからスレッドセーフなappletを作成するためのオプションがありますが、CLRのすべての（可能な）実装で動作することは保証されていません。

はまた、あなたが高い競合が予想される場合を除き、二重にチェックロックが本当に必要とされていないとほとんど違いをしないことに注意してください。間違ってしまうのは比較的簡単です（最初の誤った実装を参考にしてください）。

詳細については、Jon Skeet's articleを参照してください。