C#最佳实践：重构相似重载方法消除重复switch分支代码

你的代码重复问题本质是公共解析逻辑没有抽离、重载之间没有做逻辑复用，用以下方案可以完全消除重复的switch分支，后续维护只需要改一处代码：

• 第一步：把两个方法里完全一致的「根据dataType解析response为标准格式字符串」逻辑抽成独立的私有方法，所有重载共用这部分逻辑。后续新增数据类型、修改格式化规则只需要改这一个方法，不会出现漏改重载的问题。
• 第二步：保留入参为List<string>的重载作为核心实现，内部直接调用抽离好的解析方法拿到标准格式值，再调用对应checkEquality重载即可，不需要写重复分支。
• 第三步：入参为单个string的重载不需要单独实现逻辑，直接把单个值包装为单元素列表，调用列表版本的重载返回结果即可。

重构后的完整代码如下：

// 公共解析逻辑：所有重载共用，仅维护这一处格式化规则
private string ParseToStandardFormat(string response, string dataType)
{
    return dataType switch
    {
        "Date" => DateTimeOffset.Parse(response).ToString("yyyy-MM-dd"),
        "Time" => DateTimeOffset.Parse(response).ToString("HH:mm:00.000"),
        // 原逻辑遇到不支持的类型会静默返回false，这里可以按需选择抛异常/返回原字符串
        _ => throw new ArgumentOutOfRangeException(nameof(dataType), $"不支持的数据类型: {dataType}")
    };
}

// 列表入参作为核心实现
private bool FindMatchingValue(List<string> values, string response, string dataType)
{
    var standardFormattedValue = ParseToStandardFormat(response, dataType);
    return checkEquality(values, standardFormattedValue);
}

// 单值入参直接复用核心逻辑，无重复代码
private bool FindMatchingValue(string value, string response, string dataType)
{
    return FindMatchingValue(new List<string> { value }, response, dataType);
}

补充说明 #

这种实现方式的改造成本极低，可读性完全不受影响，单值场景下创建单元素列表的性能开销在绝大多数业务场景下可以忽略。如果你后续需要支持更多入参类型，也可以把核心逻辑进一步抽象为泛型方法，但针对你当前的两个重载场景，上面的方案已经是维护成本最低的实现。

内容的提问来源于stack exchange，提问作者Mike J