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

随便百度一下, 双重检测,单例的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 便可

相关文章
相关标签/搜索