最佳实践：创建方法、不同的重载

Question

假设，例如，我们有一个方法，为了论证，我们将其称为 MethodOne；

public void MethodOne()
    {
      //do stuff. 
    }

现在假设我们要创建一个可选参数，并且我们可能决定创建另一个具有相同名称的方法，例如采用不同的重载；

public void MethodOne()
    {
       //do stuff.
    }

public void MethodOne(bool checkVar)
    {
       if(checkVar)
        {
           //do stuff
        }
       else
        {
          //do other stuff
        }
    }

所以现在我们有了一个方法，它有两种不同的重载组合（？）。在实践中，这是否比使用一种方法更好，例如只检查可选重载是否为空或包含信息；

 public void MethodOne(int? testVar)
     {
        if(testVar != null)
         { 
           //do stuff
         }
     }

这对于只有一个重载似乎微不足道，但想象一下我有 5 个要传递的变量，我会创建 5 个方法，不同重载的同名方法，还是只有一个方法并检查传递的变量？

Answer 1

有一些解决方法。例如，您可以使用一个枚举器和一个 Object 数组作为包含实际参数值的第二个参数，这样您就可以通过切换枚举器来知道如何处理数据……或者您可以只声明 5 个 Object 参数，然后检查它们的输入一个开关，相应地将它们装箱并继续。但这两种选择都是非常糟糕的做法。

我建议你坚持使用不同的重载：

public void MethodOne(Boolean value)
{
    // Process the value...
}

public void MethodOne(Int32 value)
{
    // Process the value...
}

public void MethodOne(Int32 value, String text)
{
    // Process the value and the text...
}

// And so on...

或参数声明中的默认数据：

public void MethodOne(Int32 integer = 1, String text = "hello", ...)
{
    // Process everything inside the method...
}

或参数化方法（如果每个对象类型都有共同的处理）：

public void MethodOne(params Object[] parameters)
{
    for (int i = 0; i < parameters.Length; ++i)
        // Check type of parameter and process the value...
}

或者，如果您的设计允许，方法会冒泡（这是我最喜欢的一种，因为第一种只会产生大量代码冗余，而第二种可能有时会让您或与您一起工作的其他开发人员感到困惑）：

public void MethodOne(Int32 value)
{
    MethodOne(value, "hello");
}

public void MethodOne(Int32 value, String text)
{
    // Process everything inside the method...
}

Answer 2

你可以做的是使用optional arguments:

 public void MethodOne(int testVar = 0)
 {
     if(testVar != 0)
     { 
       //do stuff
     }
 }

Answer 3

你通常会在这样的重载上冒泡：

public void MethodOne()
{
    MethodOne(1)
}
public void MethodOne(int testVar)
{
    MethodOne(testVar, "test")
}
public void MethodOne(int testVar, string testString)
{
    MethodOne(testVar, testString, null)
}
public void MethodOne(int testVar, string testString, object testObject)
{
    // Do your actual code here
}

这相当于

public void MethodOne(int testVar = 1, string testString = "test", object testObject = null)
{
    // Do your actual code here
}

但通常你应该避免重载默认参数。此外，通过像我的示例中那样“冒泡”，您可以避免冗余代码或冗余“默认参数”

Answer 4

随着参数数量的增加，我不想查看方法的嵌套 if/else 逻辑来确定正在使用哪些参数。它很难阅读、理解、维护并且可能导致错误。使用重载并保持方法简洁、精简和可维护。