在循环中处理事务

问题阐述及解决过程

现在需要写一个定时任务，其处理过程要在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方法，不知道是否会对性能有啥影响。如果有更好的方法的话可以交流一下

 

原文转载：http://www.shaoqun.com/a/540593.html

跨境电商：https://www.ikjzd.com/

photobucket：https://www.ikjzd.com/w/132

美菜网：https://www.ikjzd.com/w/1874

问题阐述及解决过程现在需要写一个定时任务，其处理过程要在for循环进行，并且循环体中要调用两个方法，大致代码是这样的：1publicvoidregionRecoveryProtectionPeriod(){2for(？？？？){3try{4method1();5method2();6}catch(Exceptione){7log.error("",e.getMessage())
kk馆：kk馆
lastpass：lastpass
跨境新手卖家第一步 速卖通商品上架完整流程：跨境新手卖家第一步 速卖通商品上架完整流程
口述：见我喂奶 小叔子摸胸耍流氓：口述：见我喂奶 小叔子摸胸耍流氓
你了解真正的亚马逊无货源模式吗？它是怎么做的？：你了解真正的亚马逊无货源模式吗？它是怎么做的？