最近一直在看很多第三方库代码，看到这段代码让我很困惑

Question

这里是 EventBus getDefault() 静态方法的一段代码，它返回 EventBus 类的静态实例。

/** Convenience singleton for apps using a process-wide EventBus instance. */
public static EventBus getDefault() {
    EventBus instance = defaultInstance;
    if (instance == null) {
        synchronized (EventBus.class) {
            instance = EventBus.defaultInstance;
            if (instance == null) {
                instance = EventBus.defaultInstance = new EventBus();
            }
        }
    }
    return instance;
}

我看到代码首先检查实例是否为空，然后在同步块中再次进行相同的检查。为什么会这样。

如果我这样写会怎样。

/** Convenience singleton for apps using a process-wide EventBus instance. */
public static EventBus getDefault() {
    EventBus instance = defaultInstance;
    if (instance == null) {
        synchronized (EventBus.class) {
            instance = EventBus.defaultInstance = new EventBus();
        }
    }
    return instance;
}

我的版本有问题吗？我在这里缺少什么？

Answer 1

在您的代码中，当实例为空时，两个线程可以同时进入 if 语句。然后，一个线程进入同步块初始化实例，而另一个被阻塞。当第一个线程退出同步块时，等待线程进入并创建另一个 Singleton 对象。请注意，当第二个线程进入同步块时，它不会检查实例是否为非空。

所以我们遵循双重检查初始化，它包括：

1. 检查变量是否已初始化（未获得锁）。如果已初始化，则立即返回。
2. 获得锁。
3. 再次检查变量是否已经初始化：如果另一个线程先获取了锁，它可能已经完成了初始化。如果是，则返回初始化的变量。
4. 否则，初始化并返回变量。

Source