1、今天遇到一个问题,删除一条记录报错。报错信息:
org.springframework.dao.InvalidDataAccessApiUsageException: No EntityManager with actual transaction available for current thread - cannot reliably process 'remove' call
了解Spring的都知道只是事务的问题。查看代码发现删除方法以及加了@Transactional标签,理论上不会出现这个错误。debug代码发现此删除代码还是有特别之处:
1、InputParamService.java类删除方法:
public void deleteAppProduct(String key) {
parameterFacility.removeParameter(ApplicationProduct.class, key);
}
2、ParameterFacility删除
public <T> boolean removeParameter(Class<T> paramClass, String key) {
return this.removeParameter(paramClass, key, (Date)null);
}
3、LocalCachedParameterFacility
@Transactional
public <T> boolean removeParameter(Class<T> paramClass, String key, Date effectiveDate) {
AnnotationTransactionAspect var10000 = AnnotationTransactionAspect.aspectOf();
Object[] var14 = new Object[]{this, paramClass, key, effectiveDate};
return Conversions.booleanValue(var10000.ajc$around$org_springframework_transaction_aspectj_AbstractTransactionAspect$1$2a73e96c(this, new LocalCachedParameterFacility$AjcClosure5(var14), ajc$tjp_2));
}
ParameterFacility是抽象类,LocalCachedParameterFacility继承了ParameterFacility
ParameterFacility的抽象方法 public abstract <T> boolean removeParameter(Class<T> var1, String var2, Date var3);
实际上调用LocalCachedParameterFacility中的removeParameter方法。从表面上看已经加了@Transactional不应出现事务异常信息。经过测试发现如果直接使用LocalCachedParameterFacility.removeParameter方法能成功删除。经过查资料发现是AOP失效了。
2、原因:
Spring AOP是通过代理模式实现的,使用的是Java动态代理和CGLIB代理来创建aop代理。没有接口的类使用cglib代理。关于 spring aop 的java动态代理原理,请看这片博客:利用java 的动态代理模拟spring的AOP
熟悉AOP的可以注意看m.invoke(target, args);部分(我们讨论的问题其实就是invoke其他类的方法)。
我们知道当方法被动态代理后,其实会生成一个动态代理的对象,代理对象去执行invoke方法,在调用被代理对象的方法时执行了一些其他动作。所以当在被代理对象的方法中调用被代理对像的其他方法时,其实是没有用代理调用,是用了被代理对象本身调用的。
当我们调用ParameterFacility.removeParameter方法时,spring的动态代理已经帮我们生成一个动态代理类,暂且叫$ParameterFacility。所以调用removeParameter方法时,其实是$ParameterFacility代理对象的removeParameter方法。this指向的是ParameterFacility,而不是$ParameterFacility这个代理对象,没有走代理,所以失效了。
3、解决方案:
通过分析原因我门知道注解失效是因为执行方法时没有走代理,所以在同一个类的方法中调用其他注解方法,应该使用代理对象调用。
spring 解决方案
1、//通过AopContext.currentProxy()获取当前代理对象。
AopContext.currentProxy();
2、修改XML 新增如下语句;先开启cglib代理,开启 exposeProxy = true,暴露代理对象
<aop:aspectj-autoproxy proxy-target-class="true" expose-proxy="true"/>
- 1
public class TicketService{
//买火车票
@Transactional
public void buyTrainTicket(Ticket ticket){
System.out.println("买到了火车票");
try {
//通过代理对象去调用sendMessage()方法
(TicketService)AopContext.currentProxy().sendMessage();
} catch (Exception e) {
logger.warn("发送消息异常");
}
}
@Transactional
public void sendMessage(){
System.out.println("消息存入数据库");
System.out.println("执行发送消息动作");
}
}
当然最好的解决方案就是避免在对象内部调用
参考:点击打开链接