為什么說在SpringAOP中，不要使用This調用方法？

作者：程序員拾山 2022-12-06 08:26:16

開發前端

今天，我們來聊一聊，為什么說在AOP方法中，不要輕易使用This調用方法？

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();
    }
}