随便百度一下, 双重检测,单例的java
都是这样编程
class Singleton{ private static Singleton singleton; private Singleton(){} public static Singleton getInstance(){ if(singleton == null){ synchronized(Singleton.class){ singleton = new Singleton(); //1 } } return singleton; } }
看起来确实没有问题,觉得这样就完美了。多线程
可是今天 我在看 《Java并发编程的艺术》并发
这本书的时候, 竟然说这种方式有个缺陷,是有问题的。线程
汗,用了那么久才知道是有缺陷的。只是可能这个缺陷很难发生。code
class Singleton{ private static Singleton singleton; private Singleton(){} public static Singleton getInstance(){ if(singleton == null){ synchronized(Singleton.class){ singleton = new Singleton(); //问题根源在这里,多线程并发的时候,可能返回是一个未初始化的对象。此时这个对象不是空对象,只是未初始化对象。 } } return singleton; } }
也就是A线程在执行 的时候, 初始化对象singleton的时候,为该对象分配了内存地址,未初始化完成对象。对象
此时线程B执行的时候 ,就 判断到 这个 对象 singleton不为空,直接返回了一个不是空的,未初始化的对象 出去了。排序
这是有问题的。这个过程涉及到java的指令重排序。内存
正确的写法是get
class Singleton{ private volatile static Singleton singleton; private Singleton(){} public static Singleton getInstance(){ if(singleton == null){ synchronized(Singleton.class){ if(singleton == null){ singleton = new Singleton(); } } } return singleton; } }
加上 volatile 便可