当前位置：首页 > article >正文

重构长方法之分解条件表达式

article 2024/10/24 6:57:24

分解条件表达式 是一种重构长方法中常用的技术，它适用于复杂的条件逻辑。在方法中，条件分支（if-else 或 switch）有时会变得条件非常多，非常复杂，难以理解和维护。通过分解条件逻辑，可以让代码更具可读性、可维护性和易于扩展。

一、问题背景

复杂的条件表达式（如嵌套的 if-else、冗长的逻辑判断）往往使得代码难以理解，尤其当这些条件逻辑混杂在业务逻辑中时，代码的核心业务变得模糊。分解这些复杂的条件可以帮助提取出各个独立的逻辑，让代码更清晰，并且将条件逻辑与业务逻辑分离。

二、重构思路

分解条件表达式的核心思想是将复杂的条件判断抽取到独立的方法中。抽取出来的方法的命名应该清晰地表达出条件的含义，可以提高代码的可读性。让每个条件的判断逻辑集中在一个单独的方法中，促使调用方的业务逻辑更简洁。

三、重构过程

示例问题：
假设我们有一个复杂的条件判断，决定用户支付金额的逻辑：

public double GetPayAmount(Employee employee)
{
    double result;
    if (employee.IsDead)
    {
        result = 0;
    }
    else if (employee.IsSeparated)
    {
        result = employee.SeverancePay;
    }
    else if (employee.IsRetired)
    {
        result = employee.Pension;
    }
    else
    {
        result = employee.Salary;
    }
    return result;
}

这个 GetPayAmount 方法中有一系列的条件逻辑，判断员工的状态来确定支付的金额。随着条件的增多，这样的逻辑容易变得更加复杂。

四、重构：将条件逻辑分解为独立方法

我们可以将每个条件判断提取到独立的布尔方法中，以提高代码的可读性：

public double GetPayAmount(Employee employee)
{
    if (IsDead(employee))
    {
        return DeadAmount();
    }
    if (IsSeparated(employee))
    {
        return SeparatedAmount(employee);
    }
    if (IsRetired(employee))
    {
        return RetiredAmount(employee);
    }
    return RegularPayAmount(employee);
}

private bool IsDead(Employee employee)
{
    return employee.IsDead;
}

private bool IsSeparated(Employee employee)
{
    return employee.IsSeparated;
}

private bool IsRetired(Employee employee)
{
    return employee.IsRetired;
}

private double DeadAmount()
{
    return 0;
}

private double SeparatedAmount(Employee employee)
{
    return employee.SeverancePay;
}

private double RetiredAmount(Employee employee)
{
    return employee.Pension;
}

private double RegularPayAmount(Employee employee)
{
    return employee.Salary;
}

五、优点

提高可读性：条件逻辑被抽取到独立的方法中，每个方法的名称清晰地描述了条件的含义。这样，主方法的业务逻辑更为简洁，读者可以快速了解代码的意图。
增强可维护性：当条件逻辑发生变化时，只需修改相应的布尔方法或处理逻辑，而不必担心对整个业务逻辑产生影响。
便于扩展：如果以后有新的条件加入（例如新增一个员工状态），可以轻松添加新的条件判断，而不需要修改核心的业务逻辑。
减少重复：当多个地方出现类似的复杂条件时，提取出来的方法可以复用，减少重复代码。

六、适用场景

当一个方法中包含复杂的条件判断（if-else 或 switch 语句）时。
当条件逻辑过于复杂且难以理解时，分解可以让代码更清晰。
业务逻辑容易变化，或者需要根据不同条件执行不同操作时，分解条件可以提高代码的灵活性。

Tip：

命名的重要性：为布尔方法命名时，要确保命名能准确反映条件的意图。如果命名不清晰，反而可能会增加代码理解的难度。
适度分解：分解条件逻辑可以提高代码可读性，但如果过度分解，导致每个条件的判断都变得非常细碎，可能会带来反效果。因此要找到一个合适的平衡点。
逻辑一致性：确保提取后的条件方法与业务逻辑保持一致，不要在提取过程中引入不必要的逻辑变化。