我们应该避免在java代码中编写静态方法以获得更好的可测试性吗?

时间:2013-11-23 14:40:59

标签: java static

我更喜欢在我的java代码中使用静态方法,因为我认为它们是“functional”“无状态”并且副作用较小。所以可能有一些辅助类和方法如下:

public class MyHelper {
    public static Set<String> array2set(String[] items) { ... }
    public static List<String> array2list(String[] items) { ...}
    public static String getContentOfUrl(String url) {
        // visit the url, and return the content of response
    }
}

public class MyApp {
    public void doSomething() {
        String[] myarray = new String[]{ "aa","bb"};
        Set<String> set = MyHelper.array2set(myarray);
        String content = MyHelper.getContentOfUrl("http://google.com");
    }
}

但我的朋友说我们应该避免定义这样的静态实用程序方法,因为我们直接在我们的代码中调用它们,如果它们具有外部依赖性,则很难模拟它们或测试它们。他认为代码应该是:

public class ArrayHelper {
    public Set<String> array2set(String[] items) { ... }
    public List<String> array2list(String[] items) { ...}
}
public class UrlHelper {
    public String getContentOfUrl(String url) {
        // visit the url, and return the content of response
    }
}

public class MyApp {
    private final ArrayHelper arrayHelper;
    private final UrlHelper urlHelper;
    public MyApp(ArrayHelper arrayHelper, UrlHelper urlHelper) {
        this.arrayHelper = arrayHelper;
        this.urlHelper = urlHelper;
    }
    public void doSomething() {
        String[] myarray = new String[]{ "aa","bb"};
        Set<String> set = arrayHelper.array2set(myarray);
        String content = urlHelper.getContentOfUrl("http://google.com");
    }
}

通过这种方式,如果我们想为MyApp编写单元测试,我们可以模拟ArrayHelperUrlHelper并将它们传递给MyApp的构造函数。

我完全同意他的UrlHelper部分意见,因为原始静态代码使MyApp不可测试。

但我对ArrayHelper部分有点困惑,因为它不依赖于任何外部资源,逻辑非常简单。在这种情况下,我们是否应该避免使用静态方法?

何时使用静态方法?或者只是尽量避免使用它?


更新

我们在开发中使用“TDD”,因此课程的可测试性通常是我们最关心的问题。

我只是在第一句中用“无状态”替换“功能性”这个词,因为那是真实的意思。

7 个答案:

答案 0 :(得分:12)

您可能永远不会想要模拟将数组转换为列表(或集合)的方法,并且此方法不需要任何状态且不依赖于任何环境,因此静态方法看起来很好我。

就像标准的Arrays.asList()(您应该使用它)。

另一方面,访问外部URL通常是您希望能够轻松模拟的东西,因为不会嘲笑它

  • 使测试成为集成测试
  • 每次运行测试时都要求使用此外部URL,这可能无法保证
  • 要求让此外部URL完全返回您希望它在测试中返回的内容(如果您要测试错误事件,则包括错误)。

答案 1 :(得分:4)

请注意Java专家中常见的一种疾病:过度工程

在您的具体示例中,您执行具有可模拟性问题。如果您遇到问题,则不会提出一般性问题,因此我认为您目前没有问题。

一般的论点是,只要有选择,static方法就更简单,因此是首选。一个可能的实例方法必须首先证明自己需要是一个实例方法。

如果这是我的项目,我会将任何改造推迟到实例方法,直到需要它变得清晰和存在的那一刻。

答案 2 :(得分:1)

静态意味着您可以在不实例化类的情况下调用该方法。如果你想将你的代码打包成一个类,并且你有一个只做一些逻辑或基本的东西的功能,那就太好了。

只是不要使用静态函数来尝试和编辑类中的成员变量(显然)。

我个人认为使用静态函数很好,因为它是无状态的。

答案 3 :(得分:1)

回答问题时应该使用静态方法“这个方法是特定实例的功能吗?”

你不应该根据测试决定static方法,你应该根据设计来做。您的示例不需要实例,因为它没有意义。所以static是更好的选择。您始终可以将这些方法包装在特定的测试器类中以进行测试。

自包含功能不是静态的唯一情况就是当您想要提供多个实现时,因此您需要避免static,因为您需要继承。

答案 4 :(得分:1)

我经常使用静态方法:

  • 用于工厂方法(显式命名的构造函数)
  • 在面向对象的层上方提供功能层,以组成对象
  • 有时用于通用功能(Apache Commons有很多很好的例子)

我从不使用“单例”(静态对象)和引用静态对象的方法,因为它们是测试和重用的完全头痛。我还避免将任何东西硬编码到可能需要更改的静态方法中。有时我会提供多种方法 - 一种方法将所有依赖项作为参数,另一种方法使用较少的参数,使用一些默认(硬编码)值调用更灵活的方法。

答案 5 :(得分:0)

java.lang.Math是静态的,这是一个很好的例子。我认为静力学并没有收集垃圾,如果可能应该避免。

答案 6 :(得分:-4)

没有

正如Peter Lawrey在该问题的评论中所提到的,Java就是面向对象编程。虽然某些功能方面是可行的并且被放入例如。 Java 8,其核心Java不起作用。 static打破了学习如何做现代Java的好处 - 更不用说各种不那么有趣的范围问题了 - 没有目的使用它们 除非< / em> 你是某种Java向导,真正知道当你使用那个神奇的关键词时会发生什么。

你不是巫师。 Java不起作用。如果你想成为一名巫师,你可以学习。如果您想以功能性方式进行编程,请查看Scala或Groovy等混合语言,或者探索功能齐全的世界,例如: Clojure的。