メソッドスコープのローカル変数に対して操作を保護する必要がありますか

Question

ある作業の完了後にコールバックと呼ばれる次のメソッドがあるとします。

public class MyClass 
    { 
     private static object _synblock = new object(); 

    public void MyCallBackMethod() 
    { 
     Dictionary<int, int> myDict = new Dictionary<int, int>(); 

     lock(_synblock) 
     { 
      myDict.Add(1, 1); 
     } 

    } 
} 

このメソッドは、複数のスレッドで呼び出すことができます。このシナリオでは、ローカル変数myDictに対して、このメソッドのスコープで定義されたすべての操作を同期させる必要がありますか？それとも完全に不要なのでしょうか？

Answer 1

一般的なルールは、マルチスレッド環境での同時アクセスからインスタンスレベルのメンバー（およびデータ）を保護することではありません。理由は非常に簡単です。マルチスレッドは別の問題です。

私のアドバイスは、アクセスの同期方法のみを知っているマルチスレッドラッパーを作ることです。そのためには、クラスの機能を公開する基本クラスまたはインタフェースが必要です。

public interface ISomeInterface 
{ 
    void MyCallBackMethod(); 
} 

public class MyClass : ISomeInterface 
{ 
    private int Data { get; set; } 
    public void MyCallBackMethod() 
    { 
     Dictionary<int, int> myDict = new Dictionary<int, int>(); 
     myDict.Add(1, 1); 
     // access this.Data - this is the part that would 
     // make problems in case of multithreaded access 
    } 
} 

public class ThreadSafe : ISomeInterface 
{ 
    private ISomeInterface Contained { get; } 
    private object SyncRoot { get; } = new object(); 

    public ThreadSafe(ISomeInterface contained) 
    { 
     this.Contained = contained; 
    } 

    public void MyCallBackMethod() 
    { 
     lock (this.SyncRoot) 
     { 
      this.Contained.MyCallBackMethod(); 
     } 
    } 
} 

Answer 2

これはまったく必要ありません。 MyCallbackMethodが呼び出されるたびに、新しいmyDictオブジェクトがインスタンス化されます。同じインスタンスが複数のスレッドによって同時に使用された場合は、Dictionary<,>へのアクセスを保護する必要があります。

Answer 3

ローカル変数はスレッド間で共有されないため、完全に不要です。