SpringAOP是Spring中除了依赖注入以外最为核心的功能，其原理是利用CGlib和JDK动态代理等方式来实现运行期动态方法增强，从而降低系统耦合，提升代码的复用性。

不过，在享受AOP强大功能便利的同时，我们也会经常遇到一些看起来莫名其妙的bug。

今天，我们来聊一聊，

为什么说在AOP方法中，不要轻易使用this调用方法？

使用了this会出现什么样的情况？背后的原理是什么？又该如何解决？

废话不多说，直接实战上代码。

场景复现

假设我们有一个核心支付类，其中有pay()支付功能，同时会通过record()方法记录都有哪些用户访问过这个核心功能。

@Service
public class PayService {

    public void pay(){
        System.out.println("执行一些核心支付业务操作");
        //记录用户访问日志
        this.record();
    }

    public void record(){
        try {
            System.out.println("模拟将操作记录投递到日志系统，耗时100ms");
            TimeUnit.MICROSECONDS.sleep(100);
        } catch (InterruptedException e) {
            throw new RuntimeException(e);
        }
    }
}

随着业务的不断扩大，我们需要统计一下保存访问日志这个动作的耗时情况，看看是否会对核心支付功能有较大的影响。

所以，我们使用SpringAOP进行了切面处理。

@Aspect
@Component
public class LogAspect {

    @Around(value = "execution(* com.shishan.demo2023.service.PayService.record()) )")
    public Object record(ProceedingJoinPoint proceedingJoinPoint) throws Throwable {
        long begin = System.currentTimeMillis();
        Object proceed = proceedingJoinPoint.proceed();
        System.out.println("记录日志耗时:" + (System.currentTimeMillis() - begin));
        return proceed;
    }
}

切面类很简单，通过@Around方法对record方法进行切入切出，并记录该方法的执行时间。

看起来很完美是不是？

老代码不用改动，只需新增一个切面就可以实现新的需求。

我们新建一个controller，看看我们的切面类有没有生效。

@RestController
@RequestMapping(value = "/demo")
public class DemoController {

    @Resource
    private PayService payService;

    @RequestMapping(value = "/pay")
    public ResponseEntity<Object> pay(){
        this.payService.pay();
        return ResponseEntity.ok().build();
    }
}

启动服务，访问http://localhost:8080/demo/pay。