单独的方法或修改现有的功能

Question

我正在构建方法转换 turn Zeroe的字符串将转换为int：

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

    return ConvertInternal(binaryString); // Peforms convertion.
}

此方法运行提供的字符串验证：

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是valdate而不是抛出异常。

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;
}

这是有效的方式还是我缺少某些内容？

Answer 1

如果您的validate中的语句语句，我不喜欢嵌套的。如果您也给它一个OUT参数怎么办 -

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

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

     return true;
}

那么您有两种使用它的方法

public int Convert(string binaryString)
{
    if(!Validate(binaryString, out var error))
    {
        throw new InvalidDataException(error);
    }

    return ConvertInternal(binaryString);
}

-

public bool TryConvert(string binaryString, out int result)
{
    result = 0;

    if (!Validate(binaryString, out var error))
    {
        return false;
    }

    result = ConvertInternal(binaryString);
    return true;
}