c# - 在循环中调用具有相同输出和不同输入的多个方法

Question

我有多种具有不同输入参数和相同输出的方法。

我用数字输入类型调用 Method1，然后检查其结果，如果结果有效，则调用下一个方法(这次输入字符串类型)等等。

在这个例子中我有 3 个方法，但是如果我有 10 个方法或 20 个方法具有不同的输入和相同的输出我必须编写冗余代码，我该如何防止这些冗余代码？

这是方法示例:

public ValidationResult Method1(int number, string family)
{
    var validationResult = new validationResult();
    if(number > 10 || family="akbari")
    {
        validationResult.Errors.Add(new ValidationFailure("", "Invalid Number"));
    }
    return validationResult;
}

public ValidationResult Method1(string name)
{
    var validationResult = new validationResult();
    if(name.Length > 20)
    {
        validationResult.Errors.Add(new ValidationFailure("", "Invalid name"));
    }
    return validationResult;
}

public ValidationResult Method1(double average, string family)
{
    var validationResult = new validationResult();
    if(average < 14)
    {
        validationResult.Errors.Add(new ValidationFailure("", "Invalid average"));
    }
    return validationResult;
}

我这样称呼这个方法:

var validationResult = Method1(20, "test");
if (!validationResult.IsValid)
{
    return validationResult.FirstError();
}
validationResult = Method2("Samsung");
if (!validationResult.IsValid)
{
    return validationResult.FirstError();
}   
validationResult = Method3(15.5);

if (!validationResult.IsValid)
{
    return validationResult.FirstError();
}

Answer 1

(另请检查下面的替代解决方案!)

var valMethods = new List<Func<ValidationResult>>
{
    ()=>Method1(number,family),
    ()=>Method2(name),
    // ...
};

foeach(var valMethod in valMethods)
{
    var valResult = valMethod();
    if (!valResult.IsValid)
    {
        return valResult.Errors.First();
    }
}

然而，这对委托(delegate)有一些性能损失，并且由于您需要参数，因此很难放置在外部，例如在返回委托(delegate)的方法中。

作为替代方案，您可以创建一个值包作为参数(可能是具有适当接口(interface)的调用类)，这对所有方法都是相同的，并且每个方法都选择它需要的东西(这反过来使验证方法不太清楚)。

另一种选择是只返回 bool 值并将验证细节保存在单独的对象中，可能实现验证的扩展方法(也检查 Myleo 的谓词答案):

internal static class ValidationMethods
{
    public static bool CheckIsValid1(this IList<ValidationResult> valResults, int number, string family)
    {
        var validationResult = new validationResult();
        if(number > 10 || family="akbari")
        {
            validationResult.Errors.Add(new ValidationFailure("", "Invalid Number"));
        }
        valResults.Add(validationResult);
        return validationResult.IsValid;
    }

    public static bool CheckIsValid2(this IList<ValidationResult> valResults, string name)
    {
        // next check ...
    }
}

然后，在验证代码中:

var valResults = new List<ValidationResult>();
if (!valResults.CheckIsValid1(number, family)
    || !valResults.CheckIsValid2(name)
    || // more checks... will stop if one fails
    )
{
    return valResults.Last().Errors.First();
}

这样一来，您就不必与委托(delegate)人打交道了。当第一次失败时检查停止(|| 或条件检查在第一次成功时停止，这里是一个否定的 IsValid)。并且您的 valResults 仍然包含验证数据。

从您的代码来看，简单地使用异常也可能是可行的。如果错误频繁发生并且异常会破坏性能，那么这种无异常验证可能是必要的，但如果错误是异常的，那么请随意使用异常!