Question

我正在学习AOP春天并尝试一些例子。关于@ AfterReturning，我所理解的是只有当目标成功返回并且与切入点匹配时才调用该方法。但是在我的情况下，如下所示，我有一个切入点，它定义了所有返回String的方法，但是它调用所有的void方法以及返回String的方法。

我的建议：

@AfterReturning(value= "execution(*  com.aop..CustomerServiceImpl.*(..))",
        returning= "string")
public void returnStringPointcut(JoinPoint joinPoint,String string){

    System.out.println("logAfter() is running!");
    System.out.println("String : " + string);
    System.out.println("hijacked : " + joinPoint.getSignature().getName());
    System.out.println("******");

}

请在下面找到我的Impl课程：

public void addCustomer() {
    // TODO Auto-generated method stub

}

public String getCustomer() {
    // TODO Auto-generated method stub
    return "string";
}

public boolean deleteCustomer() {
    // TODO Auto-generated method stub
    return false;
}
public void addCustomerAppended() {
    // TODO Auto-generated method stub

}

public void deleteCustomerVoid() {
    // TODO Auto-generated method stub
    //return false;
}

请在下面找到我的MainApp课程：

public class App {
    public static void main(String[] args)

    {
        ApplicationContext context = new ClassPathXmlApplicationContext(
                "Spring-Customer.xml");

        CustomerService cs =  context.getBean("customerBo", CustomerService.class);
        cs.addCustomer();
        cs.addCustomerAppended();
        cs.deleteCustomer();
        cs.deleteCustomerVoid();
        cs.getCustomer();


    }
}

我原以为只会调用getCustomer（），因为它是唯一一个返回字符串的人，但是当我运行应用程序时，我在控制台中得到以下输出：

logAfter() is running!
String : null
hijacked : addCustomer
******
logAfter() is running!
String : null
hijacked : addCustomerAppended
******
logAfter() is running!
String : null
hijacked : deleteCustomerVoid
******
logAfter() is running!
String : string
hijacked : getCustomer
******

请在下面找到我的pom.xml：

<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
    xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
    <modelVersion>4.0.0</modelVersion>

    <groupId>com.aop.spring</groupId>
    <artifactId>SpringAopOnly</artifactId>
    <version>0.0.1-SNAPSHOT</version>
    <packaging>jar</packaging>

    <name>SpringAopOnly</name>
    <url>http://maven.apache.org</url>
    <properties>
        <spring.version>3.0.5.RELEASE</spring.version>
        <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
    </properties>

    <dependencies>
        <dependency>
            <groupId>junit</groupId>
            <artifactId>junit</artifactId>
            <version>4.8.2</version>
            <scope>test</scope>
        </dependency>

        <!-- Spring 3 dependencies -->
        <dependency>
            <groupId>org.springframework</groupId>
            <artifactId>spring-core</artifactId>
            <version>${spring.version}</version>
        </dependency>

        <dependency>
            <groupId>org.springframework</groupId>
            <artifactId>spring-context</artifactId>
            <version>${spring.version}</version>
        </dependency>

        <!-- Spring AOP  -->
        <dependency>
            <groupId>org.springframework</groupId>
            <artifactId>spring-aop</artifactId>
            <version>${spring.version}</version>
        </dependency>

            <dependency>
            <groupId>org.aspectj</groupId>
            <artifactId>aspectjrt</artifactId>
            <version>1.6.11</version>
        </dependency>

        <dependency>
            <groupId>org.aspectj</groupId>
            <artifactId>aspectjweaver</artifactId>
            <version>1.6.11</version>
            </dependency>

    </dependencies>
</project>

请在下面找到我的配置文件：

<beans xmlns="http://www.springframework.org/schema/beans"
    xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" 
    xmlns:aop="http://www.springframework.org/schema/aop"
    xsi:schemaLocation="http://www.springframework.org/schema/beans
    http://www.springframework.org/schema/beans/spring-beans-3.0.xsd 
    http://www.springframework.org/schema/aop 
    http://www.springframework.org/schema/aop/spring-aop-3.0.xsd ">

    <!-- <aop:aspectj-autoproxy />  -->
    <aop:aspectj-autoproxy>
    <aop:include name ="logAspect" />
    </aop:aspectj-autoproxy>

    <bean id="customerBo" class="com.aop.impl.CustomerServiceImpl" />

    <!-- Aspect -->
    <bean id="logAspect" class="com.aop.aspect.CustomerAspect" />

</beans>

知道为什么要调用void方法吗？

我还尝试将afterReturning从String更改为boolean然后我得到预期的结果，只调用deleteCustomer，因为它返回一个布尔值。

非常感谢您的回复。

Answer 1

returning参数仅指定

用于绑定返回的通知签名中的参数名称价值

你的实际切入点

@AfterReturning(value= "execution(*  com.aop..CustomerServiceImpl.*(..))",
    returning= "string")

正在指定

execution(* com.aop..CustomerServiceImpl.*(..))

其中*匹配所有返回类型。

您应该将其更改为

execution(java.lang.String com.aop..CustomerServiceImpl.*(..))

如果您只想要声明为返回String的方法。

注释value表达式中的返回类型和方法参数类型都将在将要建议的方法中发挥作用。例如

@AfterReturning(value = "execution(String com.aop..CustomerServiceImpl.*(..))", returning = "random")
public void returnStringPointcut(JoinPoint joinPoint, Integer random) {

不匹配。

另外，您应该考虑升级Spring和aspectj版本。我认为很多这些问题要么已修复，要么整体库更稳定。

AOP Spring @AfterReturning没有按预期工作

1 个答案: