在重载构造函数中重用代码

Question

我的 BigBlock 类需要一些重载的构造函数。它们都需要以相同的方式初始化相同的几个字段。

执行此操作的正确方法是什么？是否要创建一个函数（例如下面示例中的Initialize）来执行这些操作，并让所有构造函数都调用该函数？

public class BigBlock {
    private Thing parentThing;
    Units lengthUnit;
    LabCoordinateSystem labCoordinateSystem;

    private void Initialize(){
        lengthUnit = parentThing.getPreferredUnits(0);
        labCoordinateSystem = parentThing.getCoordinateSystem();
    }

    BigBlock(Thing myThing){
        parentThing= myThing;
        Initialize();
    }

    BigBlock(Thing myThing, double x, double y, double z){
        parentThing= myThing;
        Initialize();
        // more code involving x, y, z
    }

    // a few more constructors

}

Answer 1

与往常一样，“哪种方式是……的正确/最佳方式”的答案是“取决于”。

如果 Initialize 背后的逻辑专门与对象创建相关，并且 Thing 在所有创建方面都很常见，那么您可以通过在最简单的构造函数上插入该行为并使用它存在于其他每一处。这会将初始化行为的使用集中在一个地方。

BigBlock(Thing myThing){
    parentThing= myThing;
    Initialize();
}

BigBlock(Thing myThing, double x, double y, double z){
    this(myThing);
    // more code involving x, y, z
}

在其他情况下，将 Initialize 作为单独的方法可能会很有用。例如，如果initialize执行一些可重用逻辑，就像您可以在与对象创建不同的时刻调用对象的“重置”。

Answer 2

通常，最好将所有构造函数链接到包含最多信息的单个构造函数，例如，

BigBlock(Thing myThing) {
    this(myThing, 0, 0, 0); // Assuming 0 is the default value for x, y and z
}

如果有不同的方法来调用构造函数，而不能有效地表示相同信息的子集，那么它会变得有点奇怪- 但那时我会说无论如何都有设计的味道。

请注意，当您在单个构造函数中获得所有实际逻辑时，您就不需要 Initialize 方法（应该是 initialize 来遵循 Java 命名约定） ，顺便说一句） - 这可能也您可以将字段设为最终的，而以前您无法做到这一点。

Answer 3

只需重用当前的构造函数即可。让所有其他构造函数调用初始化所有必需值的构造函数。

BigBlock(Thing myThing){
    parentThing = myThing;
    lengthUnit = parentThing.getPreferredUnits(0);
    labCoordinateSystem = parentThing.getCoordinateSystem();

}

BigBlock(Thing myThing, double x, double y, double z){
    this(myThing);
    // more code involving x, y, z
}

如果 x、y 和 z 需要成为初始化的一部分，BigBlock(Thing myThing) 应调用 < code>BigBlock(Thing myThing, double x, double y, double z) 使用默认值。

Answer 4

否 - 您可以使用此构造函数从构造函数调用其他构造函数。

BigBlock(Thing myThing) {
  this(myThing,0,0,0); // Pass default values for other constructors
}