2つのスレッドを使用して2つの数値を加算する

Question

2つのスレッドを使用して2つの数値を追加したいが、どこが間違っているのか分からなかった。
私の出力は間違っています。私は問題が同期されていますが、解決できないことを知っています。

import java.io.*; 

class GFG { 
public static void main (String[] args) throws InterruptedException { 
    final Addition a=new Addition(); 

    Thread t1 = new Thread(new Runnable() 
    { 
     public void run(){ 
      try { 
       a.add(1,10); 
      } catch(Exception e) { 
      } 
     } 
    }); 

    Thread t2 = new Thread(new Runnable() 
    { 
     public void run(){ 
      try { 
       a.add(1,4); 
      } catch(Exception e) { 
      } 
     } 
    }); 

    t1.start(); 
    t2.start(); 

    t1.join(); 
    t2.join(); 
} 

public static class Addition{ 
    int a, b; 
    int sum=0; 
    public void add(int a, int b) throws InterruptedException{ 
     synchronized(this){ 
      for (int i=a;i<=b;i++){ 
       sum=sum+i; 
       Thread.sleep(1000); 
      } 
      System.out.println("Sum="+sum); 
     } 
    } 
    } 
} 

出力：

Sum=55 
Sum=65 

と出力を見て、私はそれを言うことができます...スレッドT1のタスクを完了した後、スレッドT2はT1にその結果を追加して...？ ？

Answer 1

実際の問題を推測するの景品として、このいずれかを持つことができますので、私はこれに答えていること嫌い - コードは明らかに誤解されて横にいること目的は - sumはAdditionのクラスメンバーです。両方のスレッドで同じAdditionのインスタンスを使用しているので、合計状態はスレッド間で共有されます。つまり、2番目のスレッドは、0と集計を開始しませんが、前の追加結果は55となります。これは観測された出力につながる。

Additionを次のように変更すると、プログラムは正常に動作します。

public static class Addition { 
    int a, b; 

    public void add(int a, int b) throws InterruptedException { 
    int sum = 0; 
    synchronized (this) { 
     for (int i = a; i <= b; i++) { 
     sum = sum + i; 
     Thread.sleep(1000); 
     } 
     System.out.println("Sum=" + sum); 
    } 
    } 
} 

Answer 2

これは意図的に誤解を招くように設計されました（おそらく、パズルがオーバーフローしているのでしょうか？）。インデントが中括弧に一致しません。

「加算」は一度ゼロに初期化され、一度もゼロに再初期化されないことがわかります。

Additionオブジェクトを独立して機能させたい場合は、synchronized内でsomeをゼロに設定します。

私はちょっとそれが明らかなものの構築パズル/宿題のタイプですが、あなたは私が

Answer 3

あなたはAddition: "a"の同じインスタンスを使用していて、隠されたが、"sum"さ"a" and "b"Addition'sグローバル変数はありませんが。

Answer 4

• 2つのスレッドは、クラスAdditionの同じインスタンスを使用します。
• クラスAdditionの一意のインスタンスには、スレッドt1とt2の間で共有される変数sumがあります。
• それぞれのスレッドが実行されているので、値はsumに変更され、結果はマージされます。

私はあなたのプログラムを実行し、あなたが見ることができるように、ログを追加するためにいくつかの変更を加えた：

public static class Addition{ 
    int a, b; 
    private int sum=0; 

    public void add(int a, int b, int id) throws InterruptedException{ 
     System.out.println("INIT ADD t:[" + id + "]"); 

     synchronized(this){ 
      for (int i=a;i<=b;i++){ 
       System.out.println("Sum Before t:[" + id + "] i:[" + i + "]=" + sum); 
       sum=sum + i; 
       System.out.println("Sum After t:[" + id + "] i:[" + i + "]=" + sum); 
       Thread.sleep(1000); 
      } 
     } 
     System.out.println("Sum t:[" + id + "]=" + sum); 
    } 

    } 

ログは、私は、変数の和が各トレッドずつ影響を受けている、あなたに言ったことを確認。ここでは、ログ：あなたがforブロックを同期化したよう

INIT ADD t:[1] 
Sum Before t:[1] i:[1]=0 
INIT ADD t:[2] 
Sum After t:[1] i:[1]=1 
Sum Before t:[1] i:[2]=1 
Sum After t:[1] i:[2]=3 
Sum Before t:[1] i:[3]=3 
Sum After t:[1] i:[3]=6 
Sum Before t:[1] i:[4]=6 
Sum After t:[1] i:[4]=10 
Sum Before t:[1] i:[5]=10 
Sum After t:[1] i:[5]=15 
Sum Before t:[1] i:[6]=15 
Sum After t:[1] i:[6]=21 
Sum Before t:[1] i:[7]=21 
Sum After t:[1] i:[7]=28 
Sum Before t:[1] i:[8]=28 
Sum After t:[1] i:[8]=36 
Sum Before t:[1] i:[9]=36 
Sum After t:[1] i:[9]=45 
Sum Before t:[1] i:[10]=45 
Sum After t:[1] i:[10]=55 
Sum t:[1]=55 
Sum Before t:[2] i:[1]=55 
Sum After t:[2] i:[1]=56 
Sum Before t:[2] i:[2]=56 
Sum After t:[2] i:[2]=58 
Sum Before t:[2] i:[3]=58 
Sum After t:[2] i:[3]=61 
Sum Before t:[2] i:[4]=61 
Sum After t:[2] i:[4]=65 
Sum t:[2]=65 

は、第二のスレッド（t2）はsumに結果を追加し、保存するために開始しますが、forの始まりでt1（55）の結果をsumています。

このログによれば、取得したい動作は何ですか？