java - 我应该在我的示例中同步方法吗？

Question

我不确定在我的示例中是否应该同步方法 methodOne() 。我认为不会，但我不是 100% 确定。您能给我一些建议吗？

public class SynchroIssue {

class Test {
    private double a = 0;

    void methodOne() { 
        a++;
    }

    void go() {
        new Thread(new Runnable() {

            @Override
            public void run() {
                for (int i = 0; i < Integer.MAX_VALUE; i++) {
                    methodOne();
                    System.out.println(Thread.currentThread().getName() + ", a = " + a);
                }
            }
        }).start();
    }
}

public static void main(String... args) {
    SynchroIssue mainObj = new SynchroIssue();

    SynchroIssue.Test object1 = mainObj.new Test();
    SynchroIssue.Test object2 = mainObj.new Test();

    object1.go();
    object2.go();
}
}

Answer 1

假设您实际上要同时使用 SynchroIssue 类的实例(目前您还没有这样做)，那么答案是肯定的。

增量运算符不是原子的。它实际上是3条指令:

1. 获取当前值。

2. 加 1。

3. 存储新值。

如果未同步，并发线程可能会重叠这些步骤，从而导致奇怪的行为。

如果您确实只对整数感兴趣，另一个选择是使用 AtomicInteger ，它具有原子递增的方法。