在程序开发中,if else是我们经常用到的条件判断语句。在程序逻辑中,免不了会有各种条件的判断,并根据结果执行对应的逻辑。if else的好处就是简单,可读性高。然而,随着判断条件变得复杂,判断条件越来越多,就不那么易读了。在一些老项目中,由于缺乏重构,容易出现if else过多太复杂,导致后来开发者不容易看懂因而不敢大改动,只敢在原来的逻辑上继续叠加if else,恶性循环。有句话叫做量变引起质变,时间久了最后成为一段能跑但看不懂改不动的烂代码。
在变成烂代码之前,趁还没到看不懂的地步,日常开发迭代中可以进行小步优化。下面将讲述几个比较常见的场景以及对应的优化方式。
if 条件判断过长
现象
重构书中的示例:
if (date.before(SUMMER_START) || date.after(SUMMER_END))
charge = quantity * this.winterRate + this.winterServiceCharge;
else
charge = quantity * this.summerRate;
该段逻辑中,if的判断条件是两个条件进行逻辑或进行组合,这种代码有个问题,可读性太差,无法理解业务含义。
解决方法
将很长的判断式封装成函数或者定义成宏,并以一个清晰表达意图的名字命名,名字能够体现出业务意义。
if (notSummer(date))
charge = winterCharge(quantity);
else
charge = summerCharge(quantity);
private boolean notSummer(Date date) {
return date.before(SUMMER_START) || date.after(SUMMER_END);
}
private winterCharge(int quantity) {
return quantity * this.winterRate + this.winterServiceCharge;
}
private summerCharge(int quantity) {
return quantity * this.summerRate;
}
if 一系列条件测试,返回相同的结果
现象
继续借用重构中的示例:
public String getStuLevel(int score) {
if (score == 100) {
return "A";
} else if (score >= 90) {
return "A";
} else if (score >= 80) {
return "B";
} else if (score >= 70) {
return "B";
} else if (score >= 60) {
return "C";
} else {
return "D";
}
}
score == 100和score >= 90都是返回A。
解决方法
将返回结果相同的分支进行合并处理。重构后代码如下所示:
public String getStuLevel(int score) {
if (score == 100 || score >= 90) {
return "A";
} else if (score >= 80 || score >= 70) {
return "B";
} else if (score >= 60) {
return "C";
} else {
return "D";
}
}
if else 嵌套过深
现象
还是上个例子
if (condition1) {
if (condition2) {
action2();
} else {
action1();
}
} else {
if (condition2) {
action2();
} else {
action3();
}
}
这个例子中,if 里面还嵌套了if else。这个例子只是嵌套了两层,就已经不容易读了。真实项目中这个问题严重的多,个人这是对代码 可读性差的重要元凶。
解决方法
提前return。把一些场景提前return掉。优化后的代码如下:
if (condition2) {
action2();
return;
}
if (condition1) {
action1();
return;
}
action3();
项目中也有很多案例可以参考,if (不为空) {处理业务逻辑} 的可以先判断if (为空), 后面处理业务逻辑的{}就可以去掉了。 还有一些特殊的场景,可以提前return掉,这个需要对业务逻辑比较熟悉,重构时要小心。
显式路由转发
现象
if else中,还有种场景,if else中只有单层,没有嵌套,功能上只是做路由转发作用,这种情况使用if else的方式逻辑上也是很简单。
解决方法
如果想要代码看起来更美观,可以做成隐式映射,隐式映射有两种方法:枚举和Map方式。
枚举示例
来看一段代码
String orderStatusDes;
if ("1".equals(orderStatus)) {
orderStatusDes = "订单未支付";
} else if ("2".equals(orderStatus)) {
orderStatusDes = "订单已支付";
} else if ("3".equals(orderStatus)) {
orderStatusDes = "订单已发货";
} else if ("4".equals(orderStatus)) {
orderStatusDes = "订单已签收";
} else if ("5".equals(orderStatus)) {
orderStatusDes = "订单已评价";
}
这段代码中,功能是根据订单状态找到对应的订单状态描述。采用枚举的方式优化如下:
public enum OrderStatusEnum {
UN_PAID("1","订单未支付"),
PAIDED("2","订单已支付"),
SENDED("3","订单已发货"),
SINGED("4","订单已签收"),
EVALUATED("5","订单已评价");
private String status;
private String statusDes;
static OrderStatusEnum of(String status) {
for (OrderStatusEnum statusEnum : OrderStatusEnum.values()) {
if (statusEnum.getStatus().equals(status)) {
return statusEnum;
}
}
return null;
}
}
String orderStatusDes = OrderStatusEnum.of(orderStatus).getStatusDes();
借用枚举的隐式映射特性,这段代码优化后一方面可续性上更加直观,订单状态和状态描述很直接就对应上;另一方面,代码也美观了好多,将映射的逻辑剥离出来放到枚举类中,外部调用一行代码就能解决。
Map映射示例
先来看一段代码
if (param.equals(value1)) {
doAction1(someParams);
} else if (param.equals(value2)) {
doAction2(someParams);
} else if (param.equals(value3)) {
doAction3(someParams);
}
// ...
这段代码中根据value1值类型,路由不同的处理逻辑,来看如何用Map来优化。
/1. 先定义一个 ActionService 接口
public interface ActionService {
void doAction(Context context);
}
//2. 然后定义 5 个实现类
public class ActionService1 implements ActionService{
public void doAction(Context context) {
//do something
}
}
//3. 加入表中
Map<String, ActionService> actionMap = new HashMap<>();
action.put("code1",new ActionService1());
action.put("code2",new ActionService2());
action.put("code3",new ActionService3());
action.put("code4",new ActionService4());
action.put("code5",new ActionService5());
//4. 调用
actionMap.get(action).doAction(someParams);
如果是spring项目,第3步可以从容器中自动找ActionService的实现类填充Map。
转载来源:https://juejin.cn/post/7329033935492202505