自定义@MethodExecutionTime注解部分方法失效，私有方法不生效的解决方法

问题背景 #

自定义了用于统计方法执行时间的注解：

@Target(ElementType.METHOD) // 注解可应用于方法
@Retention(RetentionPolicy.RUNTIME) // 注解在运行时保留
public @interface MethodExecutionTime {
} 

对应的切面实现：

@Aspect
@Component
public class MethodExecutionTimeAspect {
    private static final Logger logger = LoggerFactory.getLogger(MethodExecutionTimeAspect.class);
    @Around("@annotation(com.teamsapi.custom_annotation.annotation.MethodExecutionTime)")
    public Object measureExecutionTime(ProceedingJoinPoint joinPoint) throws Throwable {
        long startTime = System.currentTimeMillis();
        Object result = joinPoint.proceed();
        long endTime = System.currentTimeMillis();
        long executionTime = endTime - startTime;
        logger.warn("Method '{}' took {} milliseconds to execute.", joinPoint.getSignature().toShortString(), executionTime);
        return result;
    }
}

项目依赖配置：

dependencies {
    implementation 'org.jsoup:jsoup:1.15.3'
    implementation 'org.springframework.boot:spring-boot-starter-data-mongodb'
    implementation 'org.springframework.boot:spring-boot-starter-web'
    testImplementation 'org.springframework.boot:spring-boot-starter-test'
    implementation 'org.springframework:spring-aop:5.3.9'
    implementation 'org.aspectj:aspectjweaver:1.9.7'
}

plugins {
    id 'java'
    id 'org.springframework.boot' version '3.1.0'
    id 'io.spring.dependency-management' version '1.1.0'
}

但Service类中标记了注解的私有方法不生效，部分其他方法正常工作：

@MethodExecutionTime
private String responseDataReceived(String url) {
   return "code";
}

问题分析 #

1. 部分方法生效、部分不生效的原因 #

Spring AOP默认基于动态代理实现，只有通过代理对象发起的方法调用才会被切面拦截：

• 生效的方法：通常是被外部组件（如Controller）调用的公共方法，调用路径经过代理对象，切面逻辑被触发。
• 不生效的场景：
  • 类内部直接调用（如Service的公共方法调用自身私有方法），此时是通过this引用调用，未经过代理对象，切面无法拦截。
  • 私有方法本身受Java访问权限限制，动态代理无法生成对应的代理方法（子类无法继承父类私有方法），因此无论调用方式如何都无法被拦截。

2. 私有方法不生效的核心原因 #

• 动态代理（JDK/CGLIB）的局限性：JDK代理仅支持接口方法，CGLIB可代理类的非私有方法，但私有方法无法被子类覆盖，代理类无法拦截其调用。
• 私有方法只能被类内部调用，而内部调用不会经过代理对象，因此标注的注解永远不会触发切面逻辑。

解决办法 #

方案1：调整方法权限+通过代理对象调用（Spring AOP范围内解决） #

如果业务允许，将私有方法改为public或protected，并确保调用时走代理对象：

1. 修改方法权限：

@MethodExecutionTime
protected String responseDataReceived(String url) {
   return "code";
}

2. 在类内部注入自身的代理对象，通过代理对象调用方法：

@Service
public class YourService {
    @Autowired
    private YourService selfProxy; // 注入自身代理对象

    public void somePublicMethod() {
        // 通过代理对象调用，触发切面
        selfProxy.responseDataReceived("https://example.com");
    }
}

方案2：切换为AspectJ织入（支持私有方法拦截） #

若必须保留私有方法，需使用AspectJ的原生织入能力（不受动态代理限制）：

1. 更新依赖配置：

dependencies {
    // 原有依赖不变
    implementation 'org.aspectj:aspectjrt:1.9.7'
    compileOnly 'org.aspectj:aspectjweaver:1.9.7'
}

2. 开启AspectJ加载时织入（LTW）：
   • 在application.properties中添加配置：

     spring.aop.proxy-target-class=true
     spring.aspectj.weaving.enabled=true
     

   • 或在Spring Boot主类上添加注解：

     @SpringBootApplication
     @EnableLoadTimeWeaving
     public class Application {
         public static void main(String[] args) {
             SpringApplication.run(Application.class, args);
         }
     }
     

3. 原有切面逻辑无需修改，AspectJ会自动拦截所有标注了@MethodExecutionTime的方法，包括私有方法。

正确实现的注意事项 #

• 使用Spring AOP动态代理时：
  • 仅对公共方法的代理调用生效，避免类内部直接调用。
  • 不要在私有方法上使用AOP注解，动态代理无法拦截。
• 使用AspectJ时：
  • 确保加载时织入配置正确，依赖完整。
  • AspectJ可拦截所有方法（私有、静态、构造方法等），适用场景更广泛。

内容的提问来源于stack exchange，提问作者Ansh Gupta