动机
有时你会发现,一组条件表达式的所有分支都执行了相同的某段代码。如果是这样,你就应该将这段代码搬移到条件表达式外面。这样,代码才能更清楚地表明那些东西随条件的变化而变化、哪些东西保持不变。
做法
- 鉴别出“执行方式不随条件变化而变化”的代码。
- 如果这些共通代码位于条件表达式起始处,就将它移到条件表达式之前。
- 如果这些共通代码位于条件表达式尾端,就将它移到条件表达式之后。
- 如果这些共通代码位于条件表达式中段,就需要观察共通代码之前或之后的代码是否改变了什么东西。如果的确有所改变,应该首先将共通代码向前或向后移动,移动条件表达式的起始处或尾端,再以前面所说的办法来处理。
- 如果共通代码不止一条语句,应该首先使用Extract Method将共通代码提炼到一个独立函数中, 再以前面所说的办法来处理。
范例
if (isSpecialDeal()) {
total = price * 0.95;
send();
}
else {
total = price * 0.98;
send();
}
重构后
if (isSpecialDeal())
total = price * 0.95;
else
total = price * 0.98;
send();