Question

我刚刚遇到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;
    }
}

有人可以澄清一下吗？

Answer 1

syncObject每次都在改变它（++将它转换为原始int，递增它，然后将其自动装箱回到Integer对象。整数对象是不可变的......一旦它们被创建，它们就会无法改变。

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

使用一个对象作为同步（称之为syncObj），并将其声明为最终对象：

private static final Object syncObject = new Object();

那么你的计数器应该是一个原始的（int）用于perofrmance，称之为'counter'或者什么。

在syncObject上同步，并递增计数器。

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

Answer 2

假设是因为Integer对象的不变性。

我已将同步块更改为

Integer old = syncObj;
syncObj ++;
System.out.println(syncObj == old);

我的控制台充满了false s

所以每次我增加Integer时，新对象都是createt。

因此我只读取旧对象并且不会被锁定。

Answer 3

这些操作通常使用Atomic完成。看看here。这些结构专为多线程计算而设计。正常实现不是线程安全的。

Integer对象的synchronized块

3 个答案: