單例模式,這個(gè)我曾經(jīng)以為掌握的很不錯(cuò)了的DesignPattern,最近因?yàn)槲以诳匆坏烂嬖囶}的時(shí)候再次來到了我的面前舍扰,然后我又想到了我很久以前寫了的一個(gè)泛型單例,然后希坚,然后我在多線程的情況下跑了我的這個(gè)泛型單例边苹,然后......然后跑出來了好幾個(gè)對(duì)象,然后我就懵逼了裁僧!我這是在寫代碼么个束?我是在寫bug吧!
廢話不多說聊疲,如果你還不懂啥叫做單例模式茬底,請(qǐng)看陳皓大哥的深入淺出單實(shí)例SINGLETON設(shè)計(jì)模式,如果你懂了售睹,那么跟我走一走桩警,看看下面我親手寫的bug。我的代碼是參考了這位大哥的代碼來實(shí)現(xiàn)的JAVA語言泛型編程實(shí)現(xiàn)單例模式昌妹,當(dāng)時(shí)沒有對(duì)并發(fā)的理解捶枢,直接改動(dòng)了之后我又沒有在多線程的情況下測(cè)試,導(dǎo)致了隱藏的bug飞崖。
package com.example.curry.intellijsystem.tools;
import java.lang.reflect.Constructor;
import java.lang.reflect.InvocationTargetException;
import java.util.concurrent.ConcurrentHashMap;
/**
* Created by Curry on 17/6/26.
*/
public class Singleton<T> {
/**
* 私有構(gòu)造函數(shù)烂叔,在這里顯得表雞肋,沒有任何作用
* 但是處于對(duì)單例模式的尊重固歪,要給全它定義中的蒜鸡,私有構(gòu)造函數(shù)、私有靜態(tài)自身實(shí)例牢裳,共有靜態(tài)的獲取單例的方法
*/
private Singleton(){}
/**
* 私有逢防,靜態(tài),并發(fā)哈希表蒲讯,用于存儲(chǔ)所有的實(shí)例的唯一單例,而且其本身也是靜態(tài)唯一的
*/
private static ConcurrentHashMap<Class, Object> map = new ConcurrentHashMap<>();
/**
* 公開忘朝,靜態(tài),泛型的獲取對(duì)應(yīng)類型的單例
*/
public static <T> T getDefault(Class<T> type) {
Object object = map.get(type);
if (object == null) {
synchronized (map) {
if(object == null){//兩次判斷判帮,以此應(yīng)付多線程的沖擊
try {
/**
* 這里利用反射局嘁,將私有的構(gòu)造方法改為共有的溉箕,用于創(chuàng)建實(shí)例,否則無法創(chuàng)建實(shí)例
*/
Constructor constructor = type.getDeclaredConstructor();
constructor.setAccessible(true);
object = constructor.newInstance();
map.put(type, object);
} catch (NoSuchMethodException e) {
e.printStackTrace();
} catch (IllegalAccessException e) {
e.printStackTrace();
} catch (InstantiationException e) {
e.printStackTrace();
} catch (InvocationTargetException e) {
e.printStackTrace();
}
}
}
}
return (T) object;
}
/**
* 根據(jù)類型移除存儲(chǔ)在哈希表中的單例
*/
public static <T> void removeSingleton(Class<T> type) {
map.remove(type);
}
}
你運(yùn)行第一次的時(shí)候覺得跑出來的是單例悦昵,但是你把程序運(yùn)行10次試試肴茄,保證你懷疑人生。
或許你看了看但指,覺得沒有問題啊寡痰,這不是雙重檢查版本的單例模式么?那位大哥的還是單重檢查你可能覺得不夠安全枚赡,但是這里雙重檢查了氓癌,難道還能有鬼谓谦?巧了贫橙,還真有鬼,那么問題在哪里反粥?
問題就在于Object ob = map.get(type);
卢肃,然后竟然還狗血的拿著這個(gè)ob去做雙重檢查?腦子有病是不是才顿?每次線程來到這里都要new一個(gè)Object來接收map里面的type對(duì)應(yīng)的對(duì)象莫湘,但是你想過沒,當(dāng)程序開始執(zhí)行郑气,幾個(gè)線程一起來幅垮,同時(shí)走到了Object ob = map.get(type);
,然后每個(gè)線程就都持有了一個(gè)ob尾组,問題就在這里忙芒,每個(gè)線程都有ob,都是null讳侨,然后你實(shí)例化之后扔進(jìn)了map呵萨,然后釋放了鎖,下一個(gè)線程他自己的ob是不是還是空的翱缈纭潮峦?明顯是啊,線程1和線程2相互獨(dú)立勇婴,你的ob實(shí)例化與否關(guān)我啥事忱嘹,我只知道我自己的ob還是空的,我還得往下走耕渴,然后毫不可避免的就是拘悦,你搞出了很多個(gè)對(duì)象,根本都不是單例了萨螺。
那該咋辦呢窄做?單例寫到這里了愧驱,不能毀在并發(fā)手里吧,可以這么改進(jìn)椭盏,不要new那個(gè)Object對(duì)象了组砚,按下面這么干
public static <T> T getDefault(Class<T> type) {
if (map.get(type) == null) {
synchronized (map) {
if (map.get(type) == null) {
try {
/**
* 這里利用反射,將私有的構(gòu)造方法改為共有的掏颊,用于創(chuàng)建實(shí)例糟红,否則無法創(chuàng)建實(shí)例
*/
Constructor constructor = type.getDeclaredConstructor();
constructor.setAccessible(true);
map.put(type, constructor.newInstance());
} catch (NoSuchMethodException e) {
e.printStackTrace();
} catch (IllegalAccessException e) {
e.printStackTrace();
} catch (InstantiationException e) {
e.printStackTrace();
} catch (InvocationTargetException e) {
e.printStackTrace();
}
}
}
}
return (T)map.get(type);
}
這樣每次判斷的就是map里面的對(duì)象了,至于二次判斷還是為了多線程乌叶,因?yàn)楸热缯f有兩個(gè)線程都在第一個(gè)判斷那里發(fā)現(xiàn)了條件成立盆偿,然后
有個(gè)線程先運(yùn)行,將目標(biāo)對(duì)象實(shí)例化了准浴,釋放鎖之后第二個(gè)線程再運(yùn)行事扭,要是不二次判斷的話,就會(huì)再搞出一個(gè)實(shí)例來乐横,這就不是單例了求橄。
以前對(duì)網(wǎng)上的那些東西有很大的信任程度,很多人的博客葡公,現(xiàn)在看來都不能全信罐农,每個(gè)東西都要有自己的理解。