方法调用同一个类中的其他方法

Question

在类设计中，如果一个方法调用同一个类中的另一个方法（例如，同一个类中3个方法调用1个方法），这是一个坏习惯吗？

谢谢

Answer 1

完全不是，如果你正确设计了你的类，这是可以预料到的。您应该有一个小的公共接口，如果实现很复杂，您可能最终会在同一个类中调用许多私有方法。

事实上，不这样做可能是一种反模式。如果您不断调用另一个类的方法，则称为“功能嫉妒”，您可能应该组合类。如果您将所有实现都放入一个庞大的函数中，那么这是无法维护的，并且可能包含大量重复代码。

将事物分解为其他可重用的方法是一件好事。您的 3 个方法调用同一个方法的情况是完美的代码重用。如果该函数的代码在每个调用函数中重复，那将是非常糟糕的。

Answer 2

不。事实上，这样做是一种很好的做法。或者您是否认为执行调用的三个方法都应该复制被调用的代码？

Answer 3

方法只是函数，函数是对代码重用和设计改进有用的抽象。如果一个类的三个方法在运行时需要做同样的事情，他们应该复制粘贴代码还是调用一个单独的方法？

将代码的一些有意义的部分分离到一个由其他方法调用的方法中只是代码重用的另一个示例，除非您真的做得太过分，否则这很好。

Answer 4

不会。看来您已经找到了维护DRY 原则的有效用途。

Answer 5

一点也不。事实上恰恰相反。方法是用来完成特定任务的，您应该准确地使用它们。

Answer 6

是的，如果这些方法是可重写的，即如果类和/或方法不是最终的。原因是尝试覆盖行为或提供服务层的子类无法可靠地做到这一点，因为该层可以多次进入。
示例（Scala 伪代码），我们假设 HashSet.addAll 调用自己的 HashSet.add：

class MemCountingSet[T] extends HashSet[T] {
  private def sizeOf(t: T) = ...

  private var memCount = 0

  def add(t: T) = {
    memCount += sizeOf(t)
    super.add(t)
  }
  def addAll(coll: Collection[T]) = {
    memCount += coll.foreach(sizeOf)
    super.addAll(coll)
  }
}

当使用 addAll 时，我们现在最终会得到双重结果：计数。