java - 这样的同步不会阻止并发

Question

这个问题已经有答案了:

Threading and synchronized methods (3 个回答)

已关闭 5 年前。

我在java多线程方面非常新手，所以如果有人给我简要解释以下内容，我将非常感激:

这是我的代码:

public class Lesson6 {
    private static volatile Long value = 0L;

    public static void main(String[] args) throws InterruptedException {
        Thread inc = new Thread(new Runnable() {
            @Override
            public void run() {
                for (int i = 0; i < 100_000; i++) {
                    synchronized (this){
                        ++value;
                    }
                }
            }
        });
        inc.start();

        Thread dec = new Thread(new Runnable() {
            @Override
            public void run() {
                for (int i = 0; i < 100_000; i++) {
                    synchronized (this){
                        --value;
                    }
                }
            }
        });
        dec.start();

        inc.join();
        dec.join();

        System.out.println(value);
    }
}

在我看来，输出应该为零，但它永远不会为零。调试器显示，随着 run() 方法的执行，THIS 总是不时地变化。为什么会出现这种情况？

谢谢。

Answer 1

Debugger shows that THIS is always different from time to time as run() methods go

每个new Runnable都是一个不同的对象，因此this在每种情况下都是不同的。

如果您使用公共(public)对象，程序应该可以运行。

顺便说一句，我建议使用原始的 long 作为值，而不是对 Long 的引用

public class Lesson6 {
    private static volatile long value = 0L;

    public static void main(String[] args) throws InterruptedException {
        final Object locked = new Object();
        Thread inc = new Thread(new Runnable() {
            @Override
            public void run() {
                for (int i = 0; i < 100_000; i++) {
                    synchronized (locked){
                        ++value;
                    }
                }
            }
        });
        inc.start();

        Thread dec = new Thread(new Runnable() {
            @Override
            public void run() {
                for (int i = 0; i < 100_000; i++) {
                    synchronized (locked){
                        --value;
                    }
                }
            }
        });
        dec.start();

        inc.join();
        dec.join();

        System.out.println(value);
    }
}

打印

0