版权声明:请注明转发出处 https://blog.csdn.net/mafucun1988/article/details/89763913
《重构:改善既有代码的设计》中提到过很多重构方法,关于简化条件表达式的方法有8种。本文介绍:
以卫语句取代嵌套条件表达式 replace nested conditional with guard clauses
- 名称:以卫语句取代嵌套条件表达式 replace nested conditional with guard clauses
- 概要:函数中的条件逻辑使人难以看清正常的执行路径,使用卫语句表现所有特殊情况。
- 名词解释: 如果两条分支都是正常行为,就使用if..else...的条件表达式。如果某个条件极其罕见,就应该单独检查该条件,并在该条件为真时立刻从函数中返回,这样的单独检查常常被成为“卫语句” guard clauses.
- 动机:卫语句告诉读者,各个分支的重要性不同。某些分支很罕见,如果真发生了,请做一些必要的整理工作,然后退出。
- 做法:
- 对于每个检查,放进一个卫语句。卫语句要不就从函数中返回,要不就抛出一个异常。
- 每次将条件检查替换成卫语句后,编译并测试。如果所有的卫语句都导致相同的结果,请使用consolidate conditional expressions.
- 代码演示
- 假设一个薪酬管理系统,其中以特殊规则处理死亡员工,驻外员工,退休员工的薪资。这些情况不常有,但的确偶尔会出现。
修改之前的代码:
double SimplifyConditional::getPayAmount()
{
double result;
if (m_isDead)
result = deadAmount();
else
{
if (m_isSeparated)
result = separatedAmount();
else
{
if (m_isRetired)
result = retiredAmount();
else
result = normalPayAmount();
}
}
return result;
}
1)在这段代码中,非正常情况的检查掩盖了正常情况的检查,所以逐一使用卫语句来取代这些检查,以提高程序清晰度。
2)替换后,result没有太大的意义,所以直接删除
修改之后的代码:
double SimplifyConditional::getPayAmount()
{
if (m_isDead)
return deadAmount();
if (m_isSeparated)
return separatedAmount();
if (m_isRetired)
return retiredAmount();
return normalPayAmount();
}