假设我有一个被设计为实例化的类。我在类中有几个私有的“helper”方法,它们不需要访问任何类成员,只对它们的参数进行操作,并返回一个结果。

public class Example {
   private Something member;

   public double compute() {
       double total = 0;
       total += computeOne(member);
       total += computeMore(member);
       return total;         
   }

   private double computeOne(Something arg) { ... }
   private double computeMore(Something arg) {... } 
} 

是否有特别的理由指定computeOne和computeMore作为静态方法-或者有特别的理由不这样做?

当然,让它们保持非静态状态是最简单的,尽管它们当然可以是静态的,而不会引起任何问题。


当前回答

我将它们声明为静态,从而将它们标记为无状态。

对于不能导出的小操作,Java没有更好的机制,所以我认为私有静态是可以接受的。

其他回答

答案是……视情况而定。

如果member是特定于您正在处理的对象的实例变量,那么为什么要将它作为参数传递呢?

例如:

public class Example {
   private Something member;

   public double compute() {
       double total = 0;
       total += computeOne();
       total += computeMore();
       return total;         
   }

   private double computeOne() { /* Process member here */ }
   private double computeMore() { /* Process member here */ } 
}

一个原因是,在其他条件相同的情况下,静态方法调用应该更快。静态方法不能是虚方法,也不接受隐式this引用。

我将它们声明为静态,从而将它们标记为无状态。

对于不能导出的小操作,Java没有更好的机制,所以我认为私有静态是可以接受的。

我想澄清一些其他答案所说的事情,因为他们提供了我认为是错误的信息。

首先,因为方法是私有的,即使你声明它们是静态的,你也不能在这个类之外访问它们。其次,它们是私有的,所以你甚至不能在子类中重写,所以静态或非静态没有任何区别。第三,非静态的私有方法也可以从类的构造函数中调用,它不必是静态的。

现在回到您的问题,私有helper方法应该定义为静态还是非静态。我将采用Steve的回答,因为将私有方法标记为静态表明该方法是无状态的,因为我在编码时也遵循这一规则。

使用静态方法的一个问题是,它会使对象在单元测试中更难使用。Mockito不能为静态方法创建模拟,也不能创建该方法的子类实现。