在构造函数中做很多事情不好吗？

Question

final一般而言，制作所有字段是一个好主意，但有时我发现自己在构造函数中做了所有事情。最近，我结束了一个类实际上做的 一切都
在构造函数中，包括读取属性文件并访问数据库。

一方面，这就是该类的用途，它封装了读取的数据，我喜欢创建完全初始化的对象。构造函数完全不复杂，因为它委派了大部分工作，因此看起来不错。

另一方面，感觉有点奇怪。此外，在大约17:58的演讲中，有充分的理由不对构造函数进行过多工作。我认为我可以通过将适当的虚拟变量作为构造函数参数来消除此问题。

问题仍然存在：在构造函数中做很多工作（甚至所有工作）是否不好？

Answer 1

我认为“在构造函数中工作”是可以的…

…只要您不违反单一责任原则（SRP）并坚持使用依赖注入（DI）即可。

我最近一直在问自己这个问题。我发现反对在构造函数中进行工作的动机是：

• 很难测试
  • 我看到的所有示例都是没有使用DI的地方。这实际上不是构造函数进行实际工作的错误。
• 您可能不需要构造函数计算出的所有结果，从而浪费了处理时间，并且很难单独进行测试。
  • 这基本上是对SRP的违反，不是构造函数每个人所做的工作的错。
• 旧的编译器在构造函数中引发异常时遇到麻烦，因此除了在构造函数中分配字段外，您不应该做任何其他事情。
  • 我认为考虑到编译器的历史缺陷，编写新代码不是一个好主意。如果我们这样做的话，我们最好也不要使用C ++ 11。

我的看法是…

…如果您的构造函数需要进行一些工作以使其遵守资源获取初始化（RAII），并且该类不违反SRP，并且可以正确使用DI；那么在构造函数中进行工作就可以了！如果您想防止使用初始化完全失败的类对象，而不是依赖用户检查某些返回值，甚至可以引发异常。