其他分享
首页 > 其他分享> > 在循环中处理事务

在循环中处理事务

作者:互联网

问题阐述及解决过程

现在需要写一个定时任务,其处理过程要在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