在循环中处理事务
作者:互联网
问题阐述及解决过程
现在需要写一个定时任务,其处理过程要在for循环进行,并且循环体中要调用两个方法,大致代码是这样的:
1 public void regionRecoveryProtectionPeriod() { 2 for (????) { 3 try { 4 method1(); 5 method2(); 6 } catch (Exception e){ 7 log.error("",e.getMessage()); 8 } 9 } 10 }
用try将循环体包裹起来防止循环中断,但是这段存在一个问题——循环体中没有用事务管理,这将会造成很严重的后果。
很容易想到的方法就是将循环体单独写到一个方法里,将这个新的方法上加事务,代码如下:
1 public void regionRecoveryProtectionPeriod() { 2 for (????) { 3 try { 4 this.releaseOrigin(); 5 } catch (Exception e){ 6 log.error("",e.getMessage()); 7 } 8 } 9 10 @Transactional(rollbackFor = Exception.class) 11 void releaseOrigin() { 12 method1(); 13 method2(); 14 }
这段代码看似没有问题,但是经过试验,发现事务并没有起作用,和原来的结果没有任何区别。
原因是自调用不走代理对象,所以用this.调用内层方法时注解是不生效的,因此要通过代理对象的方式调用内层方法:Object proxy = AopContext.currentProxy();
代码如下:
public void regionRecoveryProtectionPeriod() { Service proxy = (Service)AopContext.currentProxy(); for (????) { try { proxy.releaseOrigin(); } catch (Exception e){ log.error("",e.getMessage()); } } @Transactional(propagation = Propagation.NESTED,rollbackFor = Exception.class) void releaseOrigin() { method1(); method2(); }
这样做的话,内部事务就能正常生效了,由于外层的方法通常也是要加事务的,所以内层的事务要加上propagation = Propagation.NESTED让内层事务不会影响到外层事务。
总结
关于这种问题的解决方案有人说可以将内层事务的方法写在别的service里,或者自己注入自己。代理对象的方法里用了ThreadLocal里的get方法,不知道是否会对性能有啥影响。如果有更好的方法的话可以交流一下
标签:Exception,循环体,void,事务,内层,循环,处理事务,方法 来源: https://www.cnblogs.com/qsbnj/p/14389703.html