Spring MVC Controller层事物注解不生效

@RestController
@RequestMapping("/chart")
@Transactional
public class ChartController {

    @RequestMapping(value = "/addPie", method = RequestMethod.POST)
    public ResponseEntity addPie(@RequestBody ReqPieDto pieDto) {
         try {
              weChatService.insertCharData(wxChart);
              wxPieService.insertWxPie(pieData);
              wxConditionService.insertWxCondition(conditions);
         } catch (Exception e) {
            rsp=new ResponseEntity("fail", HttpStatus.GONE);
            logger.error("pie chart config fail:",e);
         }
         return rsp;
    }
}

    这个代码存在很明显的问题，首先对Spring的事物机制没有理解。默认spring事务只在发生未被捕获的 runtimeexcetpion时才回滚，spring aop异常捕获原理：被拦截的方法需显式抛出异常，并不能经任何处理，这样aop代理才能捕获到方法的异常，才能进行回滚，默认情况下aop只捕获runtimeexception的异常，但可以通过配置来捕获特定的异常并回滚。换句话说在service的方法中不使用try catch 或者在catch中最后加上throw new runtimeexcetpion（），这样程序异常时才能被aop捕获进而回滚。

  解决方案： 

  方案1.例如service层处理事务，那么service中的方法中不做异常捕获，或者在catch语句中最后增加throw new RuntimeException()语句，以便让aop捕获异常再去回滚，并且在service上层（webservice客户端，view层action）要继续捕获这个异常并处理

  方案2.在service层方法的catch语句中增加：TransactionAspectSupport.currentTransactionStatus().setRollbackOnly();语句，手动回滚，这样上层就无需去处理异常。

  那就修改代码，Controller层修改后代码如下：

@RestController
@RequestMapping("/chart")
@Transactional
public class ChartController {

    @RequestMapping(value = "/addPie", method = RequestMethod.POST)
    public ResponseEntity addPie(@RequestBody ReqPieDto pieDto) {
         try {
              weChatService.insertCharData(wxChart);
              wxPieService.insertWxPie(pieData);
              wxConditionService.insertWxCondition(conditions);
         } catch (Exception e) {
              rsp=new ResponseEntity("fail", HttpStatus.GONE);
              logger.error("pie chart config fail:",e);
              throw new SystemException("添加饼图配置失败");
         }
         return rsp;
    }
}

    Service层代码也要抛出异常：

    public void insertWxCondition(WxConditions conditions){
        try {
            mapper.insertSelective(conditions);
        } catch (Exception e) {
            logger.error("insert into report config conditions data fail");
            throw new SystemException("insert into report config conditions data fail", e);
        }
    }

    这时候，事物是回滚了，但是页面显示的返回结果却是这样的：



    到这里，事物问题虽然解决了，但是页面的返回信息太不友好了。这是因为Controller方法抛出异常后，程序就中断了，中断后，直接把异常抛给前台页面了。如此看来，在Contrller层进行这种事物处理的时候，既要保证事物的执行，又不要抛出异常、返回自定义消息给前台页面，这二者不可兼得。那就只有一个办法了，把3个service封装到另外一个service层进行事物控制，然后抛出异常，代码如下：

public void insertPieCharData(ReqPieDto reqPieDto) {
        try {
            this.insertCharData(wxChart);
            wxPieService.insertWxPie(pieData);
            wxConditionService.insertWxCondition(conditions);
            }
            
        } catch (Exception e) {
            
            logger.error("pie chart config fail:",e);
            throw new SystemException("添加饼图配置失败");
        }
        
    }

     然后Controller层去掉trasaction注解，否则异常信息还是会被抛到前台页面，在catch exception中处理异常，代码如下：

    @RequestMapping(value = "/addPie", method = RequestMethod.POST)
    public ResponseEntity addPie(@RequestBody ReqPieDto reqPieDto) {
        
        logger.info("receive request pie config dto:{}",JsonUtil.toFullJson(reqPieDto));
        ResponseEntity rsp=new ResponseEntity("SUCCESS", HttpStatus.OK);
        
        try {
            wxChartService.insertPieCharData(reqPieDto);
        } catch (Exception e) {
            rsp=new ResponseEntity("系统异常", HttpStatus.BAD_REQUEST);
        }
       
        logger.info("返回消息:{}",JsonUtil.toFullJson(rsp));
        return rsp;
    }

     问题搞定。