Java中写了个AOP,差点被开除

时间:2022-03-08 05:29:04

为了说明这个问题,作者就采用伪代码+故事的方式描述了。

今天小A同学来了一家公司入职,发现公司的代码对于日志的处理非常不好,全部都是在代码中直接采用硬编码输出,没有统一进行处理。代码如下

Java中写了个AOP,差点被开除

小A同学,立刻就想到了,是不是可以通过AOP来对项目的日志进行统一操作呢?说起来就直接写起来代码

Java中写了个AOP,差点被开除

然后开发完成后,小a就进行了简单的测试,发现aop正常运行,达到效果

Java中写了个AOP,差点被开除

然后小A就将项目上线了,上线1min后,突然线上有很多方法报错,出现NPE异常

Java中写了个AOP,差点被开除

赶紧代码回滚,后期小A进行代码排查,发现竟然项目中有人将controller中的方法写成了private

Java中写了个AOP,差点被开除

最关键的是,没加aop之前,方法正常运行,一旦加了aop,UserService发现无法注入到spring容器中,从而导致NPE。

分析原因

Spring处理@autowire注解在bean实例化时候,初始化之前。而controller类生成代理在 bean初始化之后生成,此时虽然代理继承controller对象的属性,有@autowire注解,但是spring已经不处理@autowire注解了。

因此,原来的controller在spring中正常生成,autowire生效,代理在spring中是后来才加入到容器中,autowire不生效。

由于private的方法不会被代理类调用到原类,因此拿不到注入的属性。而public方法是可以的。

所以才会出现如下现象

Java中写了个AOP,差点被开除

Java中写了个AOP,差点被开除

结论

controller方法中,不要写private的RequestMapping方法,否则一不小心就入坑。