如何重构方法以使其更易于测试

Question

下面是我很难弄清楚如何使用JUnit进行测试的方法。
此方法很难测试，因为它取决于其他方法的结果（例如getClosestDcoumentCode）。

根据对JUnit的阅读，这表明我应该重构该方法。 但是如何？ 如果不需要重构，您如何测试依赖于其他方法的方法？

谢谢，

艾略特

private static String findPrincipal(List<DocumentKey> documentkeys_) {
    Hashtable<String, Integer> codecounts = new Hashtable<String, Integer>();
    for (DocumentKey document : documentkeys_) {
        int x = 0;
        String closestCode = getClosestDocumentCode(document.candidates);
        if (closestCode == null) continue;
        int thecount = 0;
        if (codecounts.containsKey(closestCode))
            thecount = codecounts.get(closestCode);
        if (document.hasKey)
            thecount += 2;
        else
            thecount++;
        codecounts.put(closestCode, new Integer(thecount));
        x++;

    }
    String closestCode = getClosestCode(codecounts);
    return closestCode;
}

Answer 1

好吧，首先，我想知道方法是否真的需要是静态的，以及该类在做什么。 看起来它可能是一个GOD类别，或者至少它违反了单一责任原则。 getClosestCode有什么作用？ 如果它是一个类，则可以在测试中将其与存根一起插入到测试类中。

EasyMock将让您模拟方法响应，但是我不确定如何模拟静态方法。

通常，您可能需要

1. 将长函数提取到类中
2. 使功能变为非静态
3. 维护单一责任主体

Answer 2

这听起来好像getClosestCode和getClosestDocumentCode属于一组不同的不是责任的findPrincipal方法。 因此，您首先需要将它们分为两个不同的类。 为每个要实现的类创建一个接口。 然后，实现findPrincipal方法的类可以依赖于另一个接口作为构造函数参数，如下所示：

public class PrincipalFinderImpl implements PrincipalFinder
{
    private CodeFinder codeFinder;
    public PrincipalFinderImpl(CodeFinder codeFinder) {
        this.codeFinder = codeFinder;
    }
    public String findPrincipal(List<DocumentKey> documentkeys_) {
        Hashtable<String, Integer> codecounts = new Hashtable<String, Integer>();
        for (DocumentKey document : documentkeys_) {
            int x = 0;
            String closestCode = codeFinder.getClosestDocumentCode(document.candidates);
            if (closestCode == null) continue;
            int thecount = 0;
            if (codecounts.containsKey(closestCode))
                thecount = codecounts.get(closestCode);
            if (document.hasKey)
                thecount += 2;
            else
                thecount++;
            codecounts.put(closestCode, new Integer(thecount));
            x++;

        }
        String closestCode = codeFinder.getClosestCode(codecounts);
        return closestCode;
    }
}

现在，手动或使用Mocking框架轻松创建另一个实现CodeFinder接口的类。 然后，您可以控制对getClosestCode和getClosestDocumentCode的每次调用的结果，并确保分别使用您期望使用其调用的参数来调用这些方法。

Answer 3

我没有深入了解该方法。 但是，如果需要测试私有方法，则表明您的设计有问题。 至少肯特·贝克这样认为 。

Answer 4

关于JUnit Second Edition上的存根调用的一章，如果您认为现有代码未按照测试驱动的开发标准编写，那么我建议您看一下。