记录一次单例模式乱用带来的危害。

代码魔法师Sunny2024-08-09 3:08

项目场景：

我们在接受到短信网关下发的回执之后，需要将回执内容也下发给我们的下游服务。为了防止下游响应超时，我们需要将超时的信息存放到Redis中然后进行补发操作。

问题描述

在使用Redis进行数据存储的时候，报NPE问题。

原因分析：

在Spring托管的Bean中手动通过单例模式的方式去构造了实体对象,导致Bean对象不受Spring托管，无法通过Autowried的注入方式进行注入。

代码示例如下：

java 复制代码

@Component
public class ApplicationInitor {
      @Autowired
      private UserService userService;

      public static ApplicationInitor getApplicationInitor(){
        return Inner.instance;
      }

      private static class Inner {
        private static final ApplicationInitor instance = new ApplicationInitor();
      }

    public void register(String name) {
        try {
            TestDemo testDemo = new TestDemo(userService);
            testDemo.registerUser(name);
        }catch (Exception e) {
            e.printStackTrace();
        }
    }

    class TestDemo{

          UserService userService;

          TestDemo(UserService userService){
              this.userService = userService;
          }

          public void registerUser(String name) {
              userService.setUserName(name);
              System.out.println("注册成功");
          }
      }
}

通过DEBUG的方式我们直观的看到并没有注入进来：

原因就是我们提到的手动通过单例模式获取示例对象

这里记不起来当初是为什么要通过这种方式去获取Bean，可能想调用静态的变量,但是中间又参杂着其他的逻辑不能通过类名的方式直接获取😭。

解决方案：

去掉通过单例模式获取实例的方法，因为Spring Bean本身就是单例的，无非就是通过Autowried进行注入下就行。