这个单例实现是否正确？

Question

由于双锁检查在优化编译器上不起作用，我将它从我的单例类中删除，而不是进行早期初始化。

下面是我的新单例类：

class MySingleton {
    private static volatile MySingleton myInstance = new MySingleton();
    public static getInstance() {
        return myInstance;
    }
}

除了 getInstance() 之外，类中还有设置成员字段值的 setter 方法。

希望当多个线程将使用同一个对象更新各个成员字段时，此实现不会导致任何数据不一致。

欢迎任何建议或意见。

Answer 1

此代码是线程安全的。 实际上，您可以删除volatile并且它仍然是线程安全的。

myInstance变量在第一次getInstance调用触发静态初始化（如果尚未发生）时初始化。 初始化和第一次调用有一个happens-before 。

不过，你也说了：

希望当多个线程将使用同一个对象更新各个成员字段时，此实现不会导致任何数据不一致。

您还没有显示执行此操作的任何代码，因此我们无法告诉您该代码是否是线程安全的。 如果MySingleton实例是可变的，那么与状态相关的方法必须适当地同步以防止内存危险和竞争条件。 您向我们展示的代码没有解决这个问题。

---

由于默认的MySingleton构造函数是public ，这不是一个合适的单例类。 可以创建多个实例。

---

此实现与许多其他实现的不同之处在于显着的MySingleton实例是急切地而不是懒惰地创建的。 这大大简化了问题。

请注意，单例的延迟初始化通常是不必要的。 如果您不是特别需要延迟初始化，那么更简单的 Eager 替代方案就可以了。

Answer 2

您的单例是线程安全的，但除了调用getInstance()之外，还有其他事情可能会导致MySingleton类被初始化，这将构造实例。

通常的解决方法是将单例放在一个私有嵌套类中，如下所示：

class MySingleton {
    private MySingleton() {
        // there can only be one
    }
    private static class Holder {
        static final MySingleton instance = new MySingleton();
    }
    public static MySingleton getInstance() {
        return Holder.instance;
    }
}

现在只能在Holder类初始化时构造实例，并且只在第一次调用getInstance()