隨便百度一下， 雙重檢測，單例的

都是這樣

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