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

《重构:改善既有代码的设计》中提到过很多重构方法,关于简化条件表达式的方法有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();
}
评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包

打赏作者

giantmfc123

你的鼓励将是我创作的最大动力

¥1 ¥2 ¥4 ¥6 ¥10 ¥20
扫码支付:¥1
获取中
扫码支付

您的余额不足,请更换扫码支付或充值

打赏作者

实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值