方法：一度だけ呼び出すことができるスレッドセーフメソッドを記述しますか？

Question

私はスレッドセーフなメソッドを記述しようとしていますが、これは一度だけ（オブジェクトインスタンスごとに）呼び出すことができます。以前に呼び出された場合は、例外をスローする必要があります。

私は2つの解決策を考え出しました。彼らはどちらも正しいですか？そうでない場合、何が問題なのですか？ lockで

1. ：

   public void Foo() 
   { 
       lock (fooLock) 
       { 
        if (fooCalled) throw new InvalidOperationException(); 
        fooCalled = true; 
       } 
       … 
   } 
   private object fooLock = new object(); 
   private bool fooCalled; 
   

2. Interlocked.CompareExchange付：私は間違っていない場合は、このソリューションはロックフリーであるという利点を有する（

   public void Foo() 
   { 
       if (Interlocked.CompareExchange(ref fooCalled, 1, 0) == 1) 
        throw new InvalidOperationException(); 
       … 
   } 
   private int fooCalled; 
   

   私の場合は無関係と思われる）、それよりも少ないプライベートフィールド。

私はまた、どちらの解決策を優先すべきかについての正当な意見や、より良い方法があればさらに提案することができます。

Answer 1

あなたのInterlocked.CompareExchangeソリューションが最もよく見え、（あなたが言ったように）ロックフリーです。他のソリューションよりも複雑さが大幅に軽減されています。ロックはかなり重いですが、CompareExchangeは1つのCAS cpu命令までコンパイルできます。私はそれと一緒に行くと言う。

Answer 2

ダブルチェックをロックパターはあなたが後にしているものです。

これはあなたが後にしているものです。これらのような問題にさらされた場合、一般的に

class Foo 
{ 
    private object someLock = new object(); 
    private object someFlag = false; 


    void SomeMethod() 
    { 
    // to prevent locking on subsequent calls   
    if(someFlag) 
     throw new Exception(); 

    // to make sure only one thread can change the contents of someFlag    
    lock(someLock) 
    { 
     if(someFlag) 
     throw new Exception(); 

     someFlag = true;      
    } 

    //execute your code 
    } 
} 

試してみても続くようなpattersを知っています上記。
パターンをたどるとき、特にスレッディングに遭遇したときに、何かを逃してしまう可能性が低いので、これは認識しやすく、エラーを起こしにくいからです。
あなたの場合、最初のifは多くの意味を持ちませんが、実際のロジックを実行してフラグを設定したいことがよくあります。あなたが（おそらくかなり高価な）コードを実行している間は、2番目のスレッドがブロックされます。

第2のサンプルについて：
はい、正しいですが、それよりも複雑にしないでください。あなたは単純なロックを使用しないようにすべき非常に良い理由があります。このような状況では、何も達成せずにコードを複雑にします（Interlocked.CompareExchange()はあまり知られていません）この場合のメリット）。

Answer 3

Task task = new Task((Action)(() => { Console.WriteLine("Called!"); })); 
    public void Foo() 
    { 
     task.Start(); 
    } 

    public void Bar() 
    { 
     Foo(); 
     Foo();//this line will throws different exceptions depends on 
       //whether task in progress or task has already been completed 
    }