1

所以这是来自 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;
}

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

4

1 回答 1

1

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

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

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

来源

于 2018-12-25T11:37:30.373 回答