c# - 这是重载方法的坏习惯吗？

Question

我有一个这样的方法，它的消费者正在调用它:

static public void DisplayOrderComments(param1, param2, param3, param4)

现在我为它添加了一个重载:

static public void DisplayOrderComments(param1, param2, param3, param4, param5)
{
    DisplayOrderComments(param1, param2, param3, param4);
    param5.Foo();
}

这是一种不好的做法吗？有更好的方法吗？

Answer 1

这绝对没问题 - 它保留代码 DRY并避免不必要的重复。

这不仅不是坏习惯，而且是好的习惯。

---

如果您使用的是 C# 4.0 及更高版本 (VS 2010+)，则可以使用 optional argument为您的 param5 而不是重载，正如 Mikey Mouse 在 this answer 中提到的那样.