1

我有一个必须完成的繁重计算算法。为了使其尽可能高效，我根据参数的数量将其分为五种方法。这使我能够尽可能地节省开支。也就是说，我不希望我的用户知道这一点。

public AnalysisResult[] PerformAnalysis(double[] Inputs, double[] Outputs, int[][] Coefficients)
{
    AnalysisResult[] AR = new AnalysisResult[Coefficients.Length];

    for (int i = 0; i < Coefficients.Length; i++)
        AR[i] = GetResults(Inputs, Outputs, /* Based on length of Coefficients[i]*/ );

    return AR;
}

private AnalysisResult GetResults(double[] Inputs, double[] Outputs, int A)
{
    // Do stuff
}

private AnalysisResult GetResults(double[] Inputs, double[] Outputs, int A, int B)
{
    // Do stuff
}

private AnalysisResult GetResults(double[] Inputs, double[] Outputs, int A, int B, int C)
{
    // Do stuff
}

private AnalysisResult GetResults(double[] Inputs, double[] Outputs, int A, int B, int C, int D)
{
    // Do stuff
}

private AnalysisResult GetResults(double[] Inputs, double[] Outputs, int A, int B, int C, int D, int E)
{
    // Do stuff
}

如您所见，我只希望用户可以使用一种公共方法来调用算法。我想知道是否有一种简单的方法可以根据传入的数组的长度来确定要调用哪个私有方法PerformAnalysis？

有什么建议么？

到目前为止，我的方法是基于数组长度的 switch 语句，但我认为有更好的方法。

public AnalysisResult[] PerformAnalysis(double[] Inputs, double[] Outputs, int[][] Coefficients)
{
    AnalysisResult[] AR = new AnalysisResult[Coefficients.Length];

    for (int i = 0; i < Coefficients.Length; i++)
        switch (Coefficients[i].Length)
        {
            case 1:
                AR[i] = GetResults(Inputs, Outputs, Coefficients[i][0]);
                break;
            case 2:
                AR[i] = GetResults(Inputs, Outputs, Coefficients[i][0], Coefficients[i][1]);
                break;

            // Etc
        }                

    return AR;
}
4

2 回答 2

4

好吧，从一开始，您Coefficients[i]在循环中多次使用的事实对我来说就像是一种痛苦。我要做的第一个重构是：

// Names changed to fit .NET conventions
public AnalysisResult[] PerformAnalysis(int[][] coefficients)
{
    return Array.ConvertAll(coefficients, GetResult);
}

private AnalysisResult GetResult(int[] input)
{
    switch (input.Length)
    {
        case 1: return GetResult(input[0]);
        case 2: return GetResult(input[0], input[1]);
        case 3: return GetResult(input[0], input[1], input[2]);
        case 4: return GetResult(input[0], input[1], input[2], input[3]);
        case 5: return GetResult(input[0], input[1], input[2], input[3], input[4]);
        default:
            throw new ArgumentException("Invalid number of inputs: " + input.Length);
    }
}

那是假设您真的想为不同的情况保留单独的方法，这是问题中的假设。

于 2013-08-06T10:43:22.057 回答
1

如果您愿意，可以使用反射：

for (int i = 0; i < Coefficients.Length; i++) {
    // Get all private methods named GetResults with a number of parameters equal to Cofficients[i].Length
    var methodToCall = this.GetType().GetMethods(BindingFlags.Instance | BindingFlags.NonPublic).Where(mi => mi.Name == "GetResults" && mi.GetParameters().Count() == Coefficients[i].Length);
    // Invoke that method via reflection
    AR[i] = methodToCall.Invoke(this, Cofficients[i]);
}
于 2013-08-06T10:58:01.727 回答