我刚刚遇到Java中的synchronized块,并编写了一个小程序来测试它是如何工作的.

我创建10个线程,让每个线程增加一次Integer对象1000次.

因此,通过同步,我会假设在所有线程完成工作后结果为10000,结果小于10000而没有同步.

然而,同步并不像我预期的那样.

我想它与物体的不变性有关.

我的节目：

public class SyncTest extends Thread{

    private static Integer syncObj = new Integer(0);
    private static SyncTest[] threads = new SyncTest[10];

    private boolean done = false;

    public void run(){
        for(int i = 0; i < 1000; i++){
            synchronized(syncObj){
                syncObj ++;
            }
        }
        done = true;
    }

    public static void main(String[] args) {

        for(int i=0; i < threads.length; i++){
            threads[i] = new SyncTest();
            threads[i].start();
        }

        while(!allDone()); //wait until all threads finished

        System.out.println(syncObj);
    }

    private static boolean allDone(){
        boolean done = true;
        for(int i = 0; i < threads.length; i++){
            done &= threads[i].done; 
        }

        return done;
    }
}

有人可以澄清一下吗？

解决方法:

syncObject每次都在改变它(将它转换为原始int,递增它,然后将它自动放回到Integer对象.整数对象是不可变的……一旦它们被创建,它们就无法改变.

底层的是你没有在所有线程中使用相同的syncPObj,不同的线程在不同的时间使用不同的syncObject来同步.

使用一个对象作为同步(称之为syncObj),并将其声明为最终对象：

private static final Object syncObject = new Object(); 

然后你的计数器应该是一个原始的(int)为perofrmance,称之为’counter’或其他东西.

在syncObject上同步,并递增计数器.

编辑：根据@jsn,完成标志也会被破坏,因为你的代码在isAllDone()方法上有一个“紧密循环”,这是不好的做法.你应该使用thread [i] .join()来等待(阻塞)每个线程的完成,然后从中检查状态.使用ExecutorService是“正确的方法”.