在循环中处理事务

问题阐述及解决过程

现在需要写一个定时任务，其处理过程要在for循环进行，并且循环体中要调用两个方法，大致代码是这样的：

public void regionRecoveryProtectionPeriod() {
        for (？？？？) {
            try {
                method1();
                method2();
            } catch (Exception e){
                log.error("",e.getMessage());
            }
        }
}

用try将循环体包裹起来防止循环中断，但是这段存在一个问题——循环体中没有用事务管理，这将会造成很严重的后果。

 

很容易想到的方法就是将循环体单独写到一个方法里，将这个新的方法上加事务，代码如下：

public void regionRecoveryProtectionPeriod() {
        for (？？？？) {
            try {
                this.releaseOrigin();
            } catch (Exception e){
                log.error("",e.getMessage());
            }
        }

@Transactional(rollbackFor = Exception.class)
 void releaseOrigin() {
        method1();
        method2();
    }

这段代码看似没有问题，但是经过试验，发现事务并没有起作用，和原来的结果没有任何区别。

 

原因是自调用不走代理对象，所以用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方法，不知道是否会对性能有啥影响。如果有更好的方法的话可以交流一下