C#二进制串转int：参数化复用校验逻辑方案是否合理？

我正在编写Convert方法，用于将由0和1组成的二进制string转换为int类型，初始实现代码如下：

public int Convert(string binaryString)
{
    ValidateOrThrow(binaryString);

    return ConvertInternal(binaryString); // 执行转换逻辑
}

该方法会先对传入的字符串执行校验，原有校验方法实现如下：

private void ValidateOrThrow(string binaryString)
{
    if (binaryString is null || binaryString.Length == 0)
        throw new InvalidDataException("Can't convert null or empty string");

    if (binaryString.Any(value => (byte)value is not 48 or 49))
        throw new InvalidDataException("Can't convert non-binary string");
}

现在需要新增TryConvert方法，校验失败时返回bool类型的校验结果，而非抛出异常，预期实现代码如下：

public bool TryConvert(string binaryString, out int result)
{
    if (!Validate(binaryString))
        return false;

    result = ConvertInternal(binaryString);
    return true;
}

如果额外新增一个返回bool类型的Validate方法，会违反DRY原则；因此考虑实现一个带参数的统一校验方法，在两种场景下传入不同参数调用，实现代码如下：

private bool Validate(string binaryString, bool throwOnInvalidData)
{
    if (binaryString is null || binaryString.Length == 0)
    {
        if (throwOnInvalidData)
            throw new InvalidDataException("Can't convert null or empty string");
        else
            return false;
    }

    if (binaryString.Any(value => (byte)value is not 48 or 49))
    {
        if (throwOnInvalidData)
            throw new InvalidDataException("Can't convert non-binary string");
        else
            return false;
    }

    return true;
}

请问这种参数化复用校验逻辑的实现方式是否合理，是否存在我未考虑到的设计问题？

这种写法可以正常运行，但算不上好的设计，存在几个明显的隐患：

• 职责混杂：同一个方法同时承担「校验失败抛异常」和「校验失败返回布尔值」两套错误处理逻辑，后续新增校验规则时，必须在每个判断分支里同步写两套处理代码，很容易出现两边逻辑不一致的问题——比如新增字符串长度溢出的校验时，只加了抛异常的分支，忘了加返回false的分支，就会埋下隐蔽bug。
• 调用语义差：在调用点写Validate(str, true)或者Validate(str, false)时，光看代码根本不知道布尔参数控制的是什么逻辑，后续维护的人必须点进方法内部才能看懂参数作用，可读性很差。
• 扩展性弱：如果后续需要针对不同的校验失败场景返回细分错误码、不同类型的异常，这种双分支写法会让方法内部的冗余代码快速膨胀，越来越难维护。

更合理的复用方案是把纯校验逻辑和错误处理逻辑拆分，核心校验逻辑只保留一份，两个入口方法分别适配不同的错误处理需求：

// 私有枚举，标记校验结果类型
private enum ValidationStatus
{
    Success,
    NullOrEmpty,
    InvalidCharacter
}

// 核心校验逻辑，只做校验、返回结果，不处理错误逻辑
private ValidationStatus ValidateCore(string binaryString)
{
    if (binaryString is null || binaryString.Length == 0)
        return ValidationStatus.NullOrEmpty;
    if (binaryString.Any(value => (byte)value is not 48 or 49))
        return ValidationStatus.InvalidCharacter;
    // 后续新增校验规则只需要在这里加分支即可
    return ValidationStatus.Success;
}

// 抛异常的校验入口
private void ValidateOrThrow(string binaryString)
{
    var status = ValidateCore(binaryString);
    switch (status)
    {
        case ValidationStatus.NullOrEmpty:
            throw new InvalidDataException("Can't convert null or empty string");
        case ValidationStatus.InvalidCharacter:
            throw new InvalidDataException("Can't convert non-binary string");
    }
}

// 返回布尔值的校验入口
private bool TryValidate(string binaryString)
{
    return ValidateCore(binaryString) == ValidationStatus.Success;
}

这种实现完全符合DRY原则，核心校验逻辑只有一份，同时把不同的错误处理逻辑拆分到了对应的入口方法中，不管是新增校验规则还是修改错误提示，都不需要在多个分支里重复改代码，可维护性比布尔参数的方案高很多。

如果不需要区分具体的错误类型，也可以用更轻量的实现：让ValidateOrThrow直接调用TryValidate，校验失败时直接抛异常即可，代码量更小：

private bool TryValidate(string binaryString)
{
    if (binaryString is null || binaryString.Length == 0)
        return false;
    if (binaryString.Any(value => (byte)value is not 48 or 49))
        return false;
    return true;
}

private void ValidateOrThrow(string binaryString)
{
    if (!TryValidate(binaryString))
        throw new InvalidDataException("Invalid binary string input");
}

内容的提问来源于stack exchange，提问作者Paul Pestov