《重构:改善既有代码的设计》 简化条件表达式 之 2 合并条件表达式 consolidate conditional expression

版权声明:请注明转发出处 https://blog.csdn.net/mafucun1988/article/details/89741629

《重构:改善既有代码的设计》中提到过很多重构方法,关于简化条件表达式的方法有8种。本文介绍:
合并条件表达式 consolidate conditional expression

  • 名称:合并条件表达式 consolidate conditional expression
  • 概要:一系列条件测试,都得到相同结果。将这些测试合并为一个条件表达式,并将这个条件表达式提炼成为一个独立函数
  • 动机: 一串条件检查各不相同,最终行为却一致。如果发现这种情况,就应该使用“逻辑或”和“逻辑与”将它们合并为一个条件表达式。
  • 做法:
    • 确定这些条件语句都没有副作用
    • 使用适当的逻辑操作符,将一系列相关条件表达式合并为一个
    • 编译,测试
    • 对合并后的条件表达式实施extract method
  • 代码演示

修改之前的代码:

double SimplifyConditional::disabilityAmount()
{
    if (m_seniority < 2)
        return 0;
    if (m_monthsDisabled >12)
        return  0;
    if (m_isPartTime)
        return 0;
    //compute the disability amount
    return 1;
}
double SimplifyConditional::calcCommission()
{
    if (onVacation())
    {
        if (lengthOfService() > 10)
            return 1;
    }
    return  0.5;
}

修改之后的代码:

double SimplifyConditional::disabilityAmount()
{
    if (isNotEligibleForDisability())
        return 0;
    //compute the disability amount
    return 1;
}

bool SimplifyConditional::isNotEligibleForDisability()
{
    return  (m_seniority < 2 || m_monthsDisabled >12 || m_isPartTime);
}
double SimplifyConditional::calcCommission()
{
    return (onVacation() && lengthOfService() > 10) ?  1 : 0.5 ;
}

猜你喜欢

转载自blog.csdn.net/mafucun1988/article/details/89741629