請不要再使用低級別的AOP API

 

在iteye上，諮詢我Spring問題中最多的一個就是：AOP方面的問題，我之前也寫過類似的帖子解答那些疑問：

 

• 注入FactoryBean失敗分析+解決方案
• spring的二次代理原因及如何排查
• struts2+spring集成bug——使用AOP時可能遇到的問題分析
• 關於spring的aop攔截的問題 protected方法代理問題
• Spring事務不起作用 問題彙總
• Shiro+Struts2+Spring3 加上@RequiresPermissions 後@Autowired失效
• 如何爲spring代理類設置屬性值
• 在spring中獲取代理對象代理的目標對象工具類
• Spring事務處理時自我調用的解決方案及一些實現方式的風險

 

大家有興趣可以參考我的《java開發常見問題分析》分類。裏邊收集了許多在開發過程中我遇到的或別人遇到的問題。

 

其中最主要的問題一方面是我們使用問題，另一方面是其他原因（比如bug、設計缺陷等）。

 

Spring已經太龐大了，大的連我這個熟手有時候都載了。

 

所以我想做一件事情：

 

很希望大家遇到問題時能反饋給我，我做個收集，方便後來人。點擊這前往《 那些年我們遇到的各種坑》

 

當然spring還是足夠好的，只討論問題，不討論其他。

===========================================================

在此我再給大家舉一個使用低級別AOP API遇到的坑。

 

出問題的配置

    <bean class="org.springframework.aop.aspectj.autoproxy.AspectJAwareAdvisorAutoProxyCreator">
        <property name="proxyTargetClass" value="true"/>
    </bean>
    <tx:annotation-driven transaction-manager="transactionManager"/> 

此配置的目的是想進行cglib類代理。但是實際上當進行直接注入類，而不是接口時會找不到Bean錯誤。

 

但是如果是這樣配置： 

    <aop:aspectj-autoproxy proxy-target-class="true"/>
    <tx:annotation-driven transaction-manager="transactionManager"/>

此配置可以很好的工作，並注入類（不是接口）。 

 

分析

1、<aop:aspectj-autoproxy proxy-target-class="true"> 該命名空間會交給org.springframework.aop.config.AopNamespaceHandler處理： 

registerBeanDefinitionParser("aspectj-autoproxy", new AspectJAutoProxyBeanDefinitionParser());

在AspectJAutoProxyBeanDefinitionParser中，會執行parse方法解析配置：

	public BeanDefinition parse(Element element, ParserContext parserContext) {
		AopNamespaceUtils.registerAspectJAnnotationAutoProxyCreatorIfNecessary(parserContext, element);
		extendBeanDefinition(element, parserContext);
		return null;
	}

其中AopNamespaceUtils.registerAspectJAnnotationAutoProxyCreatorIfNecessary(parserContext, element);目的是註冊AnnotationAwareAspectJAutoProxyCreator： 

return registerOrEscalateApcAsRequired(AnnotationAwareAspectJAutoProxyCreator.class, registry, source);

但是注意了： 

		if (registry.containsBeanDefinition(AUTO_PROXY_CREATOR_BEAN_NAME)) {
			BeanDefinition apcDefinition = registry.getBeanDefinition(AUTO_PROXY_CREATOR_BEAN_NAME);
			if (!cls.getName().equals(apcDefinition.getBeanClassName())) {
				int currentPriority = findPriorityForClass(apcDefinition.getBeanClassName());
				int requiredPriority = findPriorityForClass(cls);
				if (currentPriority < requiredPriority) {
					apcDefinition.setBeanClassName(cls.getName());
				}
			}
			return null;
		}

大家可以看到一句話： 

• if (registry.containsBeanDefinition(AUTO_PROXY_CREATOR_BEAN_NAME)) 
• AUTO_PROXY_CREATOR_BEAN_NAME=“org.springframework.aop.config.internalAutoProxyCreator”，
• 即首先判斷當前容器中是否包含名字爲AUTO_PROXY_CREATOR_BEAN_NAME的Bean， 如果包含：然後判斷優先級，誰優先級高誰獲勝，即最後那個獲勝的是實際的AutoProxyCreator

到此我們可以看到跟"<bean class="org.springframework.aop.aspectj.autoproxy.AspectJAwareAdvisorAutoProxyCreator">"配置沒什麼區別，除了沒有名字外。

 

2、接下來看一下<tx:annotation-driven>：

該命名空間交給org.springframework.transaction.config.TxNamespaceHandler處理： 

registerBeanDefinitionParser("annotation-driven", new AnnotationDrivenBeanDefinitionParser());

其中<annotation-driven> 會交給AnnotationDrivenBeanDefinitionParser進行解析： 

	public BeanDefinition parse(Element element, ParserContext parserContext) {
		String mode = element.getAttribute("mode");
		if ("aspectj".equals(mode)) {
			// mode="aspectj"
			registerTransactionAspect(element, parserContext);
		}
		else {
			// mode="proxy"
			AopAutoProxyConfigurer.configureAutoProxyCreator(element, parserContext);
		}
		return null;
	}

默認mode="proxy"，所以走AopAutoProxyConfigurer.configureAutoProxyCreator，其代碼中第一句話是： 

AopNamespaceUtils.registerAutoProxyCreatorIfNecessary(parserContext, element);

public static void registerAutoProxyCreatorIfNecessary(
			ParserContext parserContext, Element sourceElement) {

		BeanDefinition beanDefinition = AopConfigUtils.registerAutoProxyCreatorIfNecessary(
				parserContext.getRegistry(), parserContext.extractSource(sourceElement));
		useClassProxyingIfNecessary(parserContext.getRegistry(), sourceElement);
		registerComponentIfNecessary(beanDefinition, parserContext);
	}

AopConfigUtils.registerAutoProxyCreatorIfNecessary是： 

registerOrEscalateApcAsRequired(InfrastructureAdvisorAutoProxyCreator.class, registry, source);

private static BeanDefinition registerOrEscalateApcAsRequired(Class cls, BeanDefinitionRegistry registry, Object source) {
		Assert.notNull(registry, "BeanDefinitionRegistry must not be null");
		if (registry.containsBeanDefinition(AUTO_PROXY_CREATOR_BEAN_NAME)) {
			BeanDefinition apcDefinition = registry.getBeanDefinition(AUTO_PROXY_CREATOR_BEAN_NAME);
			if (!cls.getName().equals(apcDefinition.getBeanClassName())) {
				int currentPriority = findPriorityForClass(apcDefinition.getBeanClassName());
				int requiredPriority = findPriorityForClass(cls);
				if (currentPriority < requiredPriority) {
					apcDefinition.setBeanClassName(cls.getName());
				}
			}
			return null;
		}
//省略

此處我們又看到了registry.containsBeanDefinition(AUTO_PROXY_CREATOR_BEAN_NAME)，如果是：

• 配置1，那麼實際是兩個AutoProxyCreator；
• 配置2，那麼實際是共用一個AutoProxyCreator；

而且如果配置1時，因爲我們沒有指定<tx:annotation-driven transaction-manager="transactionManager" proxy-target-class="true"/> 所以是JDK動態代理，因此不管怎麼樣，都無法注入類的。

 

問題找到了，原因是註冊了兩個AutoProxyCreator，造成了二次代理引發的問題，這個和之前的《spring的二次代理原因及如何排查》一樣。

 

如果解決

• 給配置1起名字爲”org.springframework.aop.config.internalAutoProxyCreator“；
• 或者使用配置2 

建議

1、如果沒有必要，請不要使用低級別API，如上述-->自己去創建AutoProxyCreator

2、首先選擇使用如：

<aop:config>

<org.springframework.aop.config.internalAutoProxyCreator>

 

如上配置已經非常好了，根本沒必要使用低級別API。

 

如<tx:annotation-driven>使用的AutoProxyCreator都是和上邊是一樣的。這樣還能防止二次代理。

 

聲明式/@AspectJ風格的AOP都非常好了，完全沒必要使用低級別API，請不要再使用低級別API了。

 

 

如果用過shiro的朋友都應該知道如下配置：

<!-- Enable Shiro Annotations for Spring-configured beans.  Only run after -->
<!-- the lifecycleBeanProcessor has run: -->
<bean class="org.springframework.aop.framework.autoproxy.DefaultAdvisorAutoProxyCreator" depends-on="lifecycleBeanPostProcessor"/>
<bean class="org.apache.shiro.spring.security.interceptor.AuthorizationAttributeSourceAdvisor">
    <property name="securityManager" ref="securityManager"/>
</bean>

 

其實我們可以這樣：

    <aop:config proxy-target-class="true"/>
    <bean class="org.apache.shiro.spring.security.interceptor.AuthorizationAttributeSourceAdvisor">
        <property name="securityManager" ref="securityManager"/>
    </bean>

或者使用<aop:aspectj-autoproxy>也行，這樣也不會存在二次代理的問題。  

 

可以參考我的配置spring-mvc-shiro.xml。