同期後の競合状態

Question

複数のスレッドを実行しようとしています。私は明らかに競合状態を取得し、次のようにそれを解決することができています：

final Data data = new Data(); 
for (int i = 0; i < numberOfThreads; i++) { 
    final Thread thread = new Thread(new Runnable() { 
     @Override 
     public void run() { 
      //using this sync block to stop the race condition 
      synchronized (data){ 
       final int value = data.getValue(); 
       data.setValue(value + 1); 
      } 
     } 
    }); 
    thread.start(); 
} 

をしかし、私はこのブロックに同期して、代わりにデータのクラスでそれを処理する必要はありません。そこで、私は上記の同期ブロックを削除し、Dataクラスのgetメソッドとsetメソッドを次のように同期させましたが、それでも競合状態が発生します。私はそれらを同期させたのになぜ問題なのですか？個々のメソッドに​​を追加

public class Data { 

    private int value; 

    public synchronized int getValue(){ 
     return this.value; 
    } 

    public synchronized void setValue(int num){ 
     this.value = num; 
    } 
} 

Answer 1

スレッドが非常に明確にGETとセットの間で動けなくなる可能性があり、この

final Data data = new Data(); 
for (int i = 0; i < numberOfThreads; i++) { 
    final Thread thread = new Thread(new Runnable() { 
     @Override 
     public void run() { 
      synchronized (data){ 
       final int value = data.getValue(); 
      } 
      synchronized (data){ 
       data.setValue(value + 1); 
      } 
     } 
    }); 
    thread.start(); 
} 

のようなものをやってに似ています。これを解決するには、タスクを実行するDataクラスに新しい同期メソッドを追加するか、コード内で行ったように​​ブロックに両方の行をラップする必要があります。

Answer 2

あなたはしていないので。 2つのスレッドが同時にメソッドを実行することはできませんが、1つのスレッドが実行できるgetValue()はgetValue()を終了し、setValue()に入る前にもう1つのスレッドが回転し、getValue()が完全に有効で、競合状態です。

Btw。ちょうどDataがクラス全体になる場合は、AtomicIntegerは同じですが、正しく行われます。そこにあなた。 g。あなたの場合の要点である同期ブロックの1つの読み取りと書き込み操作を行うincrementAndGet()メソッドがあります。

Answer 3

まず、Dataクラスvalueはvolatileである必要があります。

あなたが言及している問題については、コードを変更することはできません。 発生する可能性が次にそれを変更する方法であるので：Thread 1は（valueを読み込みDataで

1）valueが0

2である）Thread 0が

3）0を読み込む（valueを読み込みます）読み取り0）

4）Thread 1増分valueとその手順5であり、ここで）

5（1を書き込み）Data.valueにThread 0ずつvalueとData.valueに新しい値を書き込む（1を書き込み）

問題に新しい値を書き込む）1ました書かれて以来Thread 0Thread 1が既にThread 0を読んでから増分していることを知らない。

Answer 4

メソッドレベルでsynchronizedを追加しました。そのため、1つのスレッドがgetValue（）を呼び出していて、1つのスレッドがsetValue（）を呼び出している可能性があります。同期されたものをすべて削除して、プライベート "value"メンバーのタイプをAutomicIntegerに変更することができます。次に、そのクラス内でスレッドセーフなメソッドを使用します。