今天小A同学来了一家公司入职,发现公司的代码对于日志的处理非常不好,全部都是在代码中直接采用硬编码输出,没有统一进行处理。代码如下
小A同学,立刻就想到了,是不是可以通过AOP来对项目的日志进行统一操作呢?说起来就直接写起来代码
然后开发完成后,小a就进行了简单的测试,发现aop正常运行,达到效果
然后小A就将项目上线了,上线1min后,突然线上有很多方法报错,出现NPE异常
赶紧代码回滚,后期小A进行代码排查,发现竟然项目中有人将controller中的方法写成了private
最关键的是,没加aop之前,方法正常运行,一旦加了aop,UserService发现无法注入到spring容器中,从而导致NPE。
分析原因
Spring处理@autowire注解在bean实例化时候,初始化之前。而controller类生成代理在 bean初始化之后生成,此时虽然代理继承controller对象的属性,有@autowire注解,但是spring已经不处理@autowire注解了。
因此,原来的controller在spring中正常生成,autowire生效,代理在spring中是后来才加入到容器中,autowire不生效。
由于private的方法不会被代理类调用到原类,因此拿不到注入的属性。而public方法是可以的。
所以才会出现如下现象
结论
controller方法中,不要写private的RequestMapping方法,否则一不小心就入坑。