从Java中的构造函数内部调用私有方法

Question

我有以下课程：

package com.tesco.demandforecasting.group8.choprachap7;

import java.util.ArrayList;

import com.tesco.demandforecasting.group8.utils.MathOperUtils;
import com.tesco.demandforecasting.group8.utils.RegressionUtils;

import lombok.Getter;

/**
 * This class if used to find seasonality factor of each period, as explain in
 * the chapter See https://kelley.iu.edu/mabert/e730/Chopra-Chap-7.pdf for the
 * explanation
 */
@Getter
public class ChopraChap7SeasonalFactorsCalculator {

    private double[] regressionParams;

    private int sales_size;
    private int periodicity;

    private ArrayList<Integer> sales;
    private ArrayList<Double> deseasonalisedData;
    private ArrayList<Double> deseasonalisedDemandUsingRegression;
    private ArrayList<Double> seasonalityFactors;

    public ChopraChap7SeasonalFactorsCalculator() {

        this.sales = new ArrayList<>();
        this.deseasonalisedData = new ArrayList<>();
        this.deseasonalisedDemandUsingRegression = new ArrayList<>();
        this.seasonalityFactors = new ArrayList<>();

        this.sales.add(8000);
        this.sales.add(13000);
        this.sales.add(23000);
        this.sales.add(34000);
        this.sales.add(10000);
        this.sales.add(18000);
        this.sales.add(23000);
        this.sales.add(38000);
        this.sales.add(12000);
        this.sales.add(13000);
        this.sales.add(32000);
        this.sales.add(41000);

        this.sales_size = sales.size();
        this.periodicity = 4;

        calculateSeasonalityFactors();
    }


    private void calculateSeasonalityFactors() {

        .......
        .......

        this.seasonalityFactors = seasonalityFactors;
        this.deseasonalisedDemandUsingRegression = deseasonalisedDemandUsingRegression;
        this.deseasonalisedData = deseasonalisedData;

    }

}

我想使用它们各自的 getter 将类字段公开给外部类。但是，问题是这些字段只有在调用 ChopraChap7SeasonalFactorsCalculator() 方法后才能获得任何值。所以，我在这里所做的是在创建类的对象后立即调用该方法。当然，这会起作用，但这是好的设计模式吗？

假设我不会从构造函数中调用该方法。所以，如果我们有下面的代码是一些其他的类：

ChopraChap7SeasonalFactorsCalculator calc = new ChopraChap7SeasonalFactorsCalculator();
calc.getDeseasonalisedData();

这将返回给我任何空数组列表。如何确保在访问任何字段之前调用该方法？

在我的情况下，最好的设计模式是什么？

Answer 1

当然，这会起作用，但这是好的设计模式吗？

这是一个非常正确的设计。您将构造函数逻辑的一部分委托给私有方法以使事情更清晰。

这将返回给我任何空数组列表。我如何确保 在访问任何字段之前调用方法？

对于任何方法或代码块，您担心有人更改构造函数中的某些内容可能是正确的。
但是应用程序的设计并不是为了检查每个组件是否符合我们的期望。这是单元测试的角色，用于断言实际行为是预期的。

因此，为 ChopraChap7SeasonalFactorsCalculator 构造函数编写一个单元测试，并在此测试中断言，一旦创建对象，所有 getter 都会返回预期值。
如果有人以不正确的方式修改了构造函数，测试将失败，构建也会失败。您有自己的方式来确保事情现在符合预期。

Answer 2

我认为这很好。构造函数用于创建有用的对象。如果您确定不设置这些对象就无法使用该对象，那么没有理由不在构造函数中设置它们。

如果你检查https://docs.oracle.com/javase/tutorial/java/javaOO/constructors.html

一个类包含被调用来创建对象的构造函数 类蓝图。

您已经添加了字段，但是如果没有设置这些字段，您就没有工作对象，显然您已经知道这些值。最好的方法是将这些留在构造函数中。如果有一些未知的值或要求来创建该类的实例，您可以考虑工厂模式或其他东西，但在您的情况下使用构造函数就可以了。