请不用再使用低级别的AOP API

请不要再使用低级别的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 value="true"/>    </bean>    <tx:annotation-driven transaction-manager="transactionManager"/>?

此配置的目的是想进行cglib类代理。但是实际上当进行直接注入类，而不是接口时会找不到Bean错误。

?

但是如果是这样配置：?

    <aop:aspectj-autoproxy proxy-target-name="code">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 name="code">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-name="code"><!-- Enable Shiro Annotations for Spring-configured beans. Only run after --><!-- the lifecycleBeanProcessor has run: --><bean depends-on="lifecycleBeanPostProcessor"/><bean ref="securityManager"/></bean>

?

其实我们可以这样：

    <aop:config proxy-target-ref="securityManager"/>    </bean>

或者使用<aop:aspectj-autoproxy>也行，这样也不会存在二次代理的问题。??

?

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

?

?