Java同期の問題

Question

私はjavaを使い慣れていないので、multiThreadingを使用して100までの数字を追加するサンプルプログラムとの同期を試していました。そして、以下は私が思いついたコードです。私はコードをテストするときに時々4590の正しい値を与えるが、時には正しい値を与えない。誰もが、私はそれが不変であるInteger、だからあなたがcounterで同期することはできません間違っ

class Counter{ 
    Integer counter = 0; 

    public void increment(int i){ 
     synchronized (counter){ 
      counter += i; 
     } 
    } 
} 

class ObjectTest implements Runnable{ 

    int i; 
    Counter blah; 

    public ObjectTest(Counter counter,int i){ 
     blah =counter; 
     this.i = i; 
    } 
    @Override 
    public void run() { 
     blah.increment(i); 
    } 
} 

public class SyncTest { 

    public static void main(String args[]) throws InterruptedException { 
     ThreadPoolExecutor executor = new ThreadPoolExecutor(4,10,60, TimeUnit.SECONDS,new SynchronousQueue<Runnable>(), new ThreadPoolExecutor.CallerRunsPolicy()); 
     Counter counter = new Counter(); 
     for (int index = 0; index < 100; index++) { 
      ObjectTest objectTest = new ObjectTest(counter,index); 
      executor.execute(objectTest); 
     } 
     executor.shutdown(); 
     while (!executor.isTerminated()){ 
      Thread.sleep(1000L); 
     } 

     System.out.println(counter.counter); 
    } 
} 

Answer 1

をやっていると指摘することができます。したがって、counter += iは、新しいIntegerオブジェクトを作成します。このオブジェクトは、同期されたものと同じではありません。

int counterにすることができ、lockに同期して別のObject lock = new Object();を持つことができます。