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

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

今天小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方法，否则一不小心就入坑。