仅仅为了可测试性而使用额外的构造函数有多好？

Question

目前我需要写SecondClass，并决定使用这种解决方案来写，否则它就无法测试。

@Component
public class FirstClass {

    public void doStuff() {
        System.out.println("First Class stuff!");
    }

}

@Component
public class SecondClass {

    private final Random random;
    private final FirstClass firstClass;

    @Autowired
    public SecondClass(FirstClass firstClass) {
        this(new Random(), firstClass);
    }

    public SecondClass(Random random, FirstClass firstClass) {
        this.random = random;
        this.firstClass = firstClass;
    }

    public void doOtherStuff() {
        firstClass.doStuff();
        System.out.println("Second Class stuff " + random.nextInt(10));
    }

}

我的同事不喜欢我的解决方法，他们更喜欢这样实现SecondClass：

@Component
public class SecondClass {

    private Random random;
    private final FirstClass firstClass;

    @Autowired
    public SecondClass(FirstClass firstClass) {
        this.random = new Random();
        this.firstClass = firstClass;
    }

    public void doOtherStuff() {
        firstClass.doStuff();
        System.out.println("Second Class stuff " + random.nextInt(10));
    }

    public void setRandom(Random random) {
        this.random = random;
    }

}

我不同意这种解决方案，因为我认为 Random 是此类的必要部分，它不会在运行时更改，并且 setter 仅用于测试目的，这就是为什么我更喜欢两个解决方案构造函数。

我们也想出了这种构造函数：

@Autowired(required = false)
public SecondClass(FirstClasss firstClass, Random random) {
    this.random = (random == null ? new Random() : random)
    ...
}

但实际上在构造函数中注入了更多组件，因此如果所有组件都需要，那将是更可取的。

我想知道这里有没有人有过这种经历？您如何看待这个案例以及是否有更好的方法来解决这个问题？

Answer 1

您的问题源于拥有 final 字段。关注鲍勃叔叔，随时保护它们，以便您可以在测试课程中给它们写信。您不是在编写接口，而是在编写代码，因此访问修饰符并不那么重要，并且 protected 很好，因为它允许包访问。

或者您可以通过硬连线将它们自己设置在您的测试用例中，例如ReflectionUtils.setField

Answer 2

如果您在SecondClass 中声明了两个构造函数，并且仅在单元测试中使用其中一个，则意味着单元测试的构造函数可以替换为您的单元测试的更相关的技巧，例如反射到设置random 字段的值。
在可以避免的情况下，您不应该仅仅为了单元测试而打开 API。
当您别无选择时，您应该只为单元测试打开您的 API。

如果random 字段是只读的，并且在SecondClass 被实例化后即为已知值，则您应该有一个SecondClass 的构造函数。并且random 字段应该是final 并且在SecondClass 的构造函数中赋值。

Answer 3

你说得对，我的朋友。您的方法是正确的，因为根据面向对象的设计，对象必须在其任何构造函数执行后初始化为有效状态。因此，如果需要random 字段，必须在所有构造函数中初始化。此外，如果以后不更改，它必须是最终的（并且没有设置器）。所以第二种方案是不对的，因为它违反了这个原则。

我也不喜欢第三种解决方案，因为它对客户端代码来说可能是愚蠢的：虽然客户端假设 null 被设置为输入值，但实际上使用了另一个（不受控制的）值.

因此，如果您需要 SecondClass 可测试的构造函数，请添加它。因为如果一个类是不可测试的，它也是不可执行的。测试是确保类可用的正确方法。