标签：Exception 循环体 void 事务 内层 循环 处理事务 方法

问题阐述及解决过程

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

本站声明： 1. iCode9 技术分享网（下文简称本站）提供的所有内容，仅供技术学习、探讨和分享；
2. 关于本站的所有留言、评论、转载及引用，纯属内容发起人的个人观点，与本站观点和立场无关；
3. 关于本站的所有言论和文字，纯属内容发起人的个人观点，与本站观点和立场无关；
4. 本站文章均是网友提供，不完全保证技术分享内容的完整性、准确性、时效性、风险性和版权归属；如您发现该文章侵犯了您的权益，可联系我们第一时间进行删除；
5. 本站为非盈利性的个人网站，所有内容不会用来进行牟利，也不会利用任何形式的广告来间接获益，纯粹是为了广大技术爱好者提供技术内容和技术思想的分享性交流网站。