java - 我应该在构造时加载 Java 对象的数据，还是通过方法调用显式加载？

Question

这是一道设计题。我正在尝试在 2 种实现之间做出决定。

为了正确解释这一点，我需要一个例子。因此，为了举例:

我有一个生成报告的类，比方说，特定日期的某些股票市场值。我创建了一个 StockMarketReportGenerator 对象，将今天的日期传递给它，它会根据今天的市场值(value)生成一份报告。

StockMarketReportGenerator“有一个”StockMarketData 对象。 StockMarketData 对象的目的是包含存储在数据库表(可能称为 StockMarket :) )中的所有股票市场值，以及根据表数据计算的一些其他值。它具有连接数据库、检索数据、进行必要的计算并将最终值存储在对象的成员变量中的私有(private)方法。 (然后它有 getter 方法来公开这些值，但没有 setter 方法。) StockMarketData 类基本上是股票市场数据值的“持有者”。我有一个名为“calculateStockMarketData()”的中央函数，它调用所有这些私有(private)辅助方法并设置对象。 (我知道所有这些处理实际上可以通过 Hibernate 等框架更轻松地处理；但决定手动完成，因为它是一个非常小的临时项目，不值得设置。)

我的问题是 - 在我的 ReportGenerator 类中，我只需要 StockMarketData 对象来访问它的属性/成员变量 - 后处理和后计算。这意味着真的，我想得到预填充的对象与数据。因此，我将 calculateStockMarketData 方法保密，并从 StockMarketData 中自动调用它构造函数。但是我对在构造函数中完成所有处理然后没有任何公共(public)方法感到有些不安。这是设计缺陷吗？或者这真的是最合乎逻辑的方法吗？基本上，以下 2 个实现中哪个更好？

1)(我当前的实现)将中央 calculateStockMarketData() 方法设为私有(private)并从 StockMarketData 方法构造函数调用它(传递今天的日期)，这样无论何时您有一个 StockMarketData 对象，它已经被填满了。因此，在我开始使用对象属性之前，我需要从 ReportGenerator 类中获取以下行:

StockMarketData myData = new StockMarketData(someDate);

2) 将中央calculateStockMarketData() 方法公开，这样为了设置StockMarketData 对象，您需要显式调用该方法。所以从 ReportGenerator 类我会编码:

StockMarketData myData = new StockMarketData(someDate);
myData.calculateStockMarketData();

第一个让我印象深刻的是更好的设计，特别是因为在初始化之前不可能使用对象属性。但是我也不确定关于从构造函数执行大量代码的标准......我应该选择哪个？

Answer 1

Martin Fowler 在 dependency injection (Constructor vs. Setter injection) 上写了一篇很好的文章.他的建议是“尽可能在构建时创建有效的对象”。

IMO，如果可能的话，最好构建有效的对象，因为这样可以更容易地阅读/观察代码的行为，因为您可以假设对象是正确构建的，并且与对象不正确相关的错误更少正确填写。 setter 与构造函数注入(inject)的问题与您所问的不同，这是在何处执行业务逻辑的问题。我认为最好使用构造函数来创建一个有效的对象，然后在另一个公共(public)方法(你的#2)中执行实际的业务逻辑，这样对象的构造就可以在与实际业务逻辑不同的时间发生。