java - 同步语句上的 NullPointerException

Question

我尝试在下面的代码中同步一个对象:

public void myMethod() {
    synchronized (globalObj) {
        //Do something here
    }
}

代码在一个线程中执行。问题是，另一个线程可能会将“globalObj”设置为空。然后，当其他线程将“globalObj”设置为 null 时，“synchronized (globalObj)”将抛出 NullPointerxception。

在对象上进行同步以便不会抛出 NullPointerException 的最佳做法是什么？

谢谢。

Answer 1

您不应该在本身可能会更改的引用上进行同步。如果允许另一个线程替换 globalObj，这意味着您可能持有旧 globalObj 的锁，而另一个线程在完全不同的线程上工作 - 锁没有帮助

您应该做的是为此目的创建一个单独的对象:

static final Object lockObj = new Object();

public void myMethod() {
  synchronized (lockObj) {
    // do something with globalObj here
  }
}

由于 lockObj 永远不会改变，您将始终使用相同的锁 - 没问题。