使用AtomicBoolean代替同步块

Question

假设我有一个带有2个实例变量的类和以下方法(简化为这个问题)：

private final Object lock = new Object();
private boolean running;

public MyClass() {
    synchronized(lock) {
        running = false;
    }
}
public void methodA() {
    synchronized(lock) {
        running = true;
    }
}
public void methodB() {
    synchronized(lock) {
        if (!running) {
            return;
        }
    }
}

我看了这段代码，在阅读了关于AtomicBoolean的文章之后，我认为这里可能适合一段代码，特别是在查看了MyClass构造函数和methodA之后。不过，我对methodB并不太确定。

假设这些方法可以被多个线程调用，下面的方法是否是线程安全的？：

private AtomicBoolean running;

public MyClass() {
    running = new AtomicBoolean(false);
}
public void methodA() {
    running.set(true);
}
public void methodB() {
    if (!running.get()) {
        return;
    }
}

会保证running.get()通过running.set(true)或running.set(false)从另一个线程看到更新吗？

Answer 1

在您的示例中，一个简单的volatile boolean就足够了，因为您似乎只执行原子操作。如果您需要诸如AtomicBoolean之类的方法，那么compareAndSet是很有用的。

因此，在回答您的问题时，是的，当使用volatile boolean或AtomicBoolean时，其他线程将看到对变量的更新。

Answer 2

一般来说，对于methodB来说，这些代码块并不相等，因为读取volatile变量不会创建同步顺序。

假设您的类中有一些其他字段int x = 42，在methodB中更新：

public void methodB() {
    if (!running.get()) {
        return;
    }
    if (x < 50) x++; // just example
}

然后有几个线程调用methodB

• 使用同步关键字时，更新对所有线程都是安全和可见的。
• 使用Atomic布尔/易失性时，可见性被破坏

如果在变量更新中没有这样的情况，并且任务只是保证methodA - methodB序列之间的可见性，那么就可以了-- AtomicBoolean就足够了。

Answer 3

是。来自AtomicBoolean的Javadoc

可以原子地更新的{@code boolean}值。

这意味着对AtomicBoolean的任何更新都是不可分割的。因此，我认为AtomicBoolean的这种使用是线程安全的。

您仍然应该考虑将AtomicBoolean的声明变为最终的：

private final AtomicBoolean running;