As a result of the refactoring, a helper class has turned out, with a thick constructor, in which a lot of computations take place based on the data obtained from other standard helper classes.

The constructor does not have paramters and their introduction is counterproductive, since the class is instantiated by a framework whose DI cannot pass paramters to the constructor .

Is the constructor "thick" design acceptable?

  • It all depends on the case. You can even do ultra-thick methods - if you are sure that no programmer-user of your methods will need to slightly change the way the method works without touching your code (this is done by inheritance). - Goncharov Alexander
  • In my opinion, the designer, this is not exactly the place where the "bunch of calculations" is placed. Constructors are usually designed to initialize the values ​​required by the behavior implemented by the class. - SergeyEgorov
  • so the matter is that there is no place to put logic - hellboy

0