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 这是哪个作者做的宝宝起名网站
  • ¥60 版本过低apk如何修改可以兼容新的安卓系统
  • ¥25 由IPR导致的DRIVER_POWER_STATE_FAILURE蓝屏
  • ¥50 有数据,怎么建立模型求影响全要素生产率的因素
  • ¥50 有数据,怎么用matlab求全要素生产率
  • ¥15 TI的insta-spin例程
  • ¥15 完成下列问题完成下列问题
  • ¥15 C#算法问题, 不知道怎么处理这个数据的转换
  • ¥15 YoloV5 第三方库的版本对照问题
  • ¥15 请完成下列相关问题!