Zzro 2016-01-20 13:31 采纳率: 100%
浏览 1456
已采纳

请教一下,我这边的代码该如何优化一下

图片说明

  • 写回答

3条回答

  • 毕小宝 博客专家认证 2016-01-20 14:13
    关注

    首先,你的if内层分支过多,可以用switch替代,并且抽取成一个方法;其次,你的if(isContinue)可以与外层的if条件合并的。
    参考优化代码如下:

     boolean isTokenOk = tokenCheck.equals(token);
    if(isTokenOk&&isContinue){
        obj = orderStatusService.updateOrderStatus(orderStatus,actionId);
        printActionInfo(actionId);  
    }else {
        obj.setStatus(1);
        if(isContinue){
            obj.setMsg("当前状态与上级状态不匹配,无法修改");
        }else{
            obj.setMsg("校验不通过");
        }
    }
    

    抽取方法if分支方法如下:

     printActionInfo(int actionId){
        switch(actionId){
        case 20:
            //
            break;
        case 30:
            //
            break;
        case 40:
            //
            break;
        case 50:
            //
            break;
        }   
    }
    
    本回答被题主选为最佳回答 , 对您是否有帮助呢?
    评论
查看更多回答(2条)

报告相同问题?

悬赏问题

  • ¥30 截图中的mathematics程序转换成matlab
  • ¥15 动力学代码报错,维度不匹配
  • ¥15 Power query添加列问题
  • ¥50 Kubernetes&Fission&Eleasticsearch
  • ¥15 報錯:Person is not mapped,如何解決?
  • ¥15 c++头文件不能识别CDialog
  • ¥15 Excel发现不可读取的内容
  • ¥15 关于#stm32#的问题:CANOpen的PDO同步传输问题
  • ¥20 yolov5自定义Prune报错,如何解决?
  • ¥15 电磁场的matlab仿真