《重构：改善既有代码的设计》 简化条件表达式 之 5 以卫语句取代嵌套条件表达式 replace nested conditional with guard clauses

版权声明：请注明转发出处 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();
}