正确的单例写法：双重检测同步锁实现

随便百度一下， 双重检测，单例的

都是这样

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并发编程的艺术》

这本书的时候， 居然说这种方式有个缺陷，是有问题的。

汗，用了那么久才知道是有缺陷的。只是可能这个缺陷很难发生。

 

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的指令重排序。

 

正确的写法是

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 即可

 

 

 

 

 

 

 

转载于:https://my.oschina.net/ouminzy/blog/3048120