ToB企服应用市场:ToB评测及商务社交产业平台

标题: 反面单例 [打印本页]

作者: 慢吞云雾缓吐愁 时间: 2023-12-29 00:41
标题: 反面单例
反面单例

代码

import java.util.ArrayList;
import java.util.List;
/**
* @since : 2023/11/17
**/
public class StupidSingleton {
private static final StupidSingleton instance = new StupidSingleton();
private static List<String> list = null;
private static Object object = null;
private StupidSingleton() {
list = new ArrayList<>();
object = new Object();
}
public synchronized static StupidSingleton getInstance() {
if (object == null) {
object = new Object();
}
return instance;
}
public void doSomething() {
list.add("Stupid");
}
}

复制代码

问题点解析

doSomething抛出空指针异常

Java中字段、代码块、构造函数初始化顺序是：

按照在代码中的定义顺序依次加载static变量和static代码块。
按照在代码中的定义顺序依次加载实例变量和实例代码块。
构造函数。

代入这段代码中顺序为:

初始化instance = new StupidSingleton()
执行构造函数
初始化list = null
初始化object = null

虽然result和object都在构造函数中初始化了，但是随后又被赋值为null。
多线程下效率低

估计后来的修改者发现了object为空的问题，所以在getInstance中又做了初始化，这样一来，多线程下就有问题，在getInstance上加个synchronized就好了。
getInstance是static的，这个时候配合synchronized锁的就是StupidSingleton.class，多线程下效率必然低。
修改方案

把instance的初始化放在所有static变量定义的最后或者在静态变量定义时直接初始化，不在构造函数中初始化。
删除getInstance的synchronized。
删除getInstance中对object的控制判断和初始化。

免责声明：如果侵犯了您的权益，请联系站长，我们会及时删除侵权内容，谢谢合作！

欢迎光临 ToB企服应用市场:ToB评测及商务社交产业平台 (https://dis.qidao123.com/)