public static NewReportBox getInstance() {
if (sInstance == null) {
synchronized (NewReportBox.class) {
if (sInstance == null) {
sInstance = new NewReportBox(BrothersApplication.sApplication);
}
}
}
return sInstance;
}
“双重检测锁”模式,看似“聪明”的方式,其实有着巨大的漏洞。简单的说,在1.5之前的JVM中,代码会进行“重整”,单例引用sInstance 有时尽管不为null,但是此时所引用的那个“单例对象”,并没有被完全初始化。也就是new NewReportBox()函数未正式完成其工作之前,JVM可以根据Java规范,重整代码,使得sInstance 先获得这个“单例对象”的引用,这样一来,第二个线程直接判定单例已完成实例化,故接下来的客户代码会直接使用单例对象的数据,但是有些数据并没有被正确的初始化,因为new NewReportBox()尚未正式完成。
目前处理成下面的方式
public static synchronized NewReportBox getInstance() {
if (sInstance == null) {
sInstance = new NewReportBox(BrothersApplication.sApplication);
}
return sInstance;
}
虽然每次获取都有个同步锁的消耗,但至少保证不会出多线程问题,如果有更好的方案,继续补充。