3条回答
关注 首先,你的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; } }
本回答被题主选为最佳回答 , 对您是否有帮助呢?解决评论 打赏 举报无用 1
悬赏问题
- ¥30 这是哪个作者做的宝宝起名网站
- ¥60 版本过低apk如何修改可以兼容新的安卓系统
- ¥25 由IPR导致的DRIVER_POWER_STATE_FAILURE蓝屏
- ¥50 有数据,怎么建立模型求影响全要素生产率的因素
- ¥50 有数据,怎么用matlab求全要素生产率
- ¥15 TI的insta-spin例程
- ¥15 完成下列问题完成下列问题
- ¥15 C#算法问题, 不知道怎么处理这个数据的转换
- ¥15 YoloV5 第三方库的版本对照问题
- ¥15 请完成下列相关问题!