C＃スレッドと同期

Question

私は同期（ロック）のために使用するプライベート静的フィールドを持っています。今私は同時に実行したくない2つの機能を持っています。だから私はこのでした：私はオブジェクトにロックすると、単一の関数の並列実行を防ぐことができますことを知っている

public class Synchronization 
{ 
    private static object _lock = new object(); 

    public void MethodA() 
    { 
     lock (_lock) 
     { 
      Console.WriteLine("I shouldn't execute with MethodB"); 
     } 
    } 

    public void MethodB() 
    { 
     lock (_lock) 
     { 
      Console.WriteLine("I shouldn't execute with MethodA"); 
     } 
    } 
} 

を、同じ仕事、私は同時に実行するさまざまな方法で同じロックオブジェクトを使用する場合は？単純に、他のスレッドが既にロックされているオブジェクトを別の関数でロックすることはできますか？

Answer 1

一度に1つのスレッドのみがロックを取得できるため、この状態は1つのロックインスタンス上のすべてのスレッドに対して排他的です。したがって、あなたの例では、あなたのロックが静的であるので、クラスSynchronizationのすべてのインスタンスに対して、指定された時間に1つのメソッド本体しか実行されないかもしれません。クラスのインスタンスごとにロックを設定する場合は、ロックオブジェクトを静的としてマークしないでください。

同期に関するあなたの仮定は正しいです。

ロックオブジェクトreadonlyを完全に水密なソリューションにする必要があることに注意してください。コードは、現状ではロックオブジェクトを再割り当てられ、そのロックのセマンティクスを破るされるために、それは例えば、可能でしょう：

public class Synchronization 
{ 
    private static object _lock = new object(); 

    public void MethodA() 
    { 
     lock (_lock) 
     { 
      Console.WriteLine("I shouldn't execute with MethodB"); 
     } 
    } 

    public void MethodB() 
    { 
     //This shouldn't be allowed! 
     _lock = new object(); 

     lock (_lock) 
     { 
      Console.WriteLine("I shouldn't execute with MethodA"); 
     } 
    } 
} 

ロックオブジェクトはreadonly、すなわちとしてマークする必要があります。

private static readonly object _lock = new object(); 

Answer 2

MSDNでLock Statementを正しく読んでいると思います。

Answer 3

正しく実行しています。同じ時間に入力されない2つの重要なセクションを作成しました。

したがって、MethodAとMethodBは同時にアクティブではありません。また、MethodA（とMethodB）が同時にアクティブになるのは1つだけです。

これは、作成するすべてのオブジェクトにわたって有効です。私が意味すること：任意のオブジェクトから任意のMethodAまたはMethodBに1つのスレッドだけがあります。ロックを1つのオブジェクト内でのみ実行する場合は、_lockオブジェクトを静的にすることはできません。

Answer 4

ロックは、lockステートメントが発生するメソッドではなく、ロックの対象となるオブジェクトに基づいて付与されます。あなたの場合、複数のスレッドがさまざまなメソッドを入力するかもしれませんが、一度に1つのスレッドだけがlockステートメント内のコードを実行することができます。

Answer 5

まず、_lockは静的であってはなりません。または、オブジェクトの複数のインスタンスを互いにロックする必要がありますか？次に、クラス内に同期メソッドを1つだけ持つ必要があります。さらに、クラス内の同期メソッド間の依存関係を避ける必要があります。そうしないと、メソッドの呼び出し元が間違って予期しない動作をするリスクがあります。

考えてみては、例えば、次のコード：

class Synchronized 
{ 
    object lockObj = new object(); 
    int counter = 100; 

    public void Decrement() 
    { 
     lock (this.lockObj) 
     { 
      this.counter--; 
     } 
    } 

    public int IsZero() 
    { 
     lock (this.lockObj) 
     { 
      return this.counter == 0; 
     } 
    } 
} 

は今1は、共有同期インスタンスに何をしますか？この

while (!synchronized.IsZero()) 
{ 
    synchronized.Decrement(); 
} 

今すぐ1つの通話デクリメントを通すよう

は、それを使用して、カウンタが0になると、それはデクリメント方法、ないIsZero方法でロックを待機していたので、すぐに、2つの通話デクリメントを通します。カウンタは-1になり、ループは無限になります。

ロック機構が誤ってプログラムされているのではなく、呼び出し元がそれをうまく使用していないとは限りません。 Synchronizedクラスで1つの同期メソッドしか公開していない場合は、プログラマが安全だと盲目的に信じることを欺くことはありません。

それはこのようなものでなければなりません：なぜ、単純に自分でそれを試してみていない

class Synchronized 
{ 
    object lockObj = new object(); 
    int counter = 100; 

    public bool IfNotZeroDecrement() 
    { 
     lock (this.lockObj) 
     { 
      if (this.counter > 0) 
       this.counter--; 

      return this.counter > 0; 
     } 
    }  
} 

/// Usage: 
while (synchronized.IfZeroDecrement()) 
{ 
}