反面单例
反面单例代码
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的控制判断和初始化。
免责声明：如果侵犯了您的权益，请联系站长，我们会及时删除侵权内容，谢谢合作！
页:
[1]