java – 是否正在创建一个用于测试不良做法的私有构造函数？

我遇到了一些java代码,其中公共构造函数调用一个包私有构造函数,其中包含一堆新的运算符来创建新对象.

public class Thing {

    //public
    public Thing(String param1,int paramm2) {
        this(param1,param2,new Dependency1(),new Dependency2());
    }

    //package-private for the sake of testing
    Thing(String param1,int param2,Dependency1 param3,Dependency2 param4) {
        this.memberVar1 = param1;
        this.memberVar2 = param2;
        this.memberVar3 = param3;
        this.memberVar4 = param4;
    }


    //...rest of class...
}

在我看来,这是错误的,因为您正在编写代码来测试它而不是编写正确的代码.我想其他两个选项(我能想到的)是创建工厂或使用PowerMockito在适用的情况下注入新对象.就个人而言,我会写如下所示.

public class Thing {

    //public
    public Thing(String param1,int paramm2) {
        this.memberVar1 = param1;
        this.memberVar2 = param2;
        this.memberVar3 = new Dependency1();
        this.memberVar4 = new Dependency2();
    }

    //...rest of class...
}

实施此方法的最佳做法/正确方法是什么？