java - 在构造函数中做很多坏事吗？

Question

关闭。这个问题是 opinion-based .它目前不接受答案。

---

想要改进这个问题吗？ 更新问题，以便 editing this post 提供事实和引用来回答它.

关闭 7 年前。

Improve this question

使所有字段final 通常是一个好主意，但有时我发现自己在构造函数中做了所有事情。最近我在构造函数中完成了一个真正所有操作的类，包括读取属性文件和访问数据库。

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

另一方面，感觉有点奇怪。此外，在 this talk在大约 17:58 时，有充分的理由在构造函数中没有做太多工作。我想我可以通过传递适当的假人作为构造函数参数来消除这个问题。

问题仍然存在:在构造函数中做大量工作(甚至所有工作)是否不好？

Answer 1

我认为“在构造函数中工作”还可以......

...只要你不违反Single Responsibility Principle (SRP)并坚持使用Dependency Injection (DI) .

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

• 很难测试
  • 我见过的所有例子都是DI没有被使用。实际工作并不是构造函数的错。
• 您可能不需要构造函数计算的所有结果，这会浪费处理时间并且很难单独进行测试。
  • 这基本上违反了SRP ，不是构造函数的错。
• 旧的编译器在构造函数中抛出异常时遇到了问题，因此除了在构造函数中分配字段之外，您不应该做任何事情。
  • 我认为在编写新代码时考虑到编译器的历史缺陷并不是一个好主意。如果我们这样做，我们还不如取消 C++11 和所有好的东西。

我的看法是……

...如果您的构造函数需要做一些工作以使其遵守 Resource Acquisition Is Initialization (RAII)并且类(class)不违反SRP和 DI使用得当；然后在构造函数中工作就是 A-Okay!如果您想阻止使用初始化完全失败的类对象，而不是依赖用户检查某些返回值，您甚至可以抛出异常。