Java volatile 需要同步访问吗？

Question

在以下情况下，volatile 是多余的吗？删除volatile 在这里会有什么不同吗？或者，离开 volatile 但删除同步块和同步修饰符在这里会有什么不同吗？

private volatile boolean registered;

synchronized boolean registered() {
  return registered;
}

void setRegistered(boolean registered) {
  synchronized (this) {
    this.registered = registered;
  }
}

Answer 1

不，除非您有其他未显示的访问权限，否则registered 没有volatile 也可以。

我认为删除volatile 会稍微加快代码速度。但我认为删除volatile 的主要好处是它为维护者清理了代码。

在这种情况下，volatile 和我可以看到的同步方法之间没有区别，因此请使用其中一个。 （除了同步方法使您用来保护registered 字段的对象public，这可能好坏参半。私有volatile 保持同步锁私有。）

---

如果您有这样的课程，请尝试澄清最后一条语句：

public class Reg {
    private boolean registered;

    synchronized boolean registered() {
      return registered;
    }

    void setRegistered(boolean registered) {
      synchronized (this) {
        this.registered = registered;
      }
    }
}

然后你创建一个新的实例：

public class Test {
  public static Reg reg;
  public void static main( String... args ) {
     reg = new Reg();
     synchronized( reg ) {
            // do stuff
     }
  }
}

当Test 中的锁被持有时，任何其他试图访问reg.registered() 的人都会阻塞，直到该锁被释放。该锁对任何可以看到该对象的人都是公开的。这可能是好是坏。它允许其他类控制对您的方法的访问，这可能有利于原子性，但它也可能将一些线程控制烘焙到源代码库中，在那里它被分散到各处，而不是仅仅被一个类封装。好和坏。可能还有其他的例子来说明为什么这可能是好是坏，这只是一个。