反面单例

反面单例

代码

1. import java.util.ArrayList;
2. import java.util.List;
4. /**
5. * @since : 2023/11/17
6. **/
7. public class StupidSingleton {
8.     private static final StupidSingleton instance = new StupidSingleton();
10.     private static List<String> list = null;
12.     private static Object object = null;
14.     private StupidSingleton() {
15.         list = new ArrayList<>();
16.         object = new Object();
17.     }
19.     public synchronized static StupidSingleton getInstance() {
20.         if (object == null) {
21.             object = new Object();
22.         }
23.         return instance;
24.     }
26.     public void doSomething() {
27.         list.add("Stupid");
28.     }
29. }

复制代码

问题点解析

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的控制判断和初始化。
  

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