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条)

报告相同问题?

悬赏问题

  • ¥15 多电路系统共用电源的串扰问题
  • ¥15 slam rangenet++配置
  • ¥15 有没有研究水声通信方面的帮我改俩matlab代码
  • ¥15 对于相关问题的求解与代码
  • ¥15 ubuntu子系统密码忘记
  • ¥15 信号傅里叶变换在matlab上遇到的小问题请求帮助
  • ¥15 保护模式-系统加载-段寄存器
  • ¥15 电脑桌面设定一个区域禁止鼠标操作
  • ¥15 求NPF226060磁芯的详细资料
  • ¥15 使用R语言marginaleffects包进行边际效应图绘制