具有许多子类的依赖注入

Question

我在 Guice 中有一个类，它使用依赖注入在构造函数中获取 ~10 个参数。

这个类有很多派生类。 所有派生类的构造函数只是将所有参数传递给super。

向基类的构造函数添加新参数需要将此参数添加到所有派生类的所有构造函数。

    class MyBaseClass {
        @Inject
        MyBaseClass(arg1,arg2,arg3, ..., argn) {
            this.arg1 = arg1
            ....
        }

    }

    class MyDerivedClass1 extends MyBaseClass{
        @Inject
        MyDerivedClass1(arg1,arg2,arg3, ..., argn) {
            super(arg1,arg2,arg3, ..., argn)
        }    
    }

    class MyDerivedClass2 extends MyBaseClass{
        @Inject
        MyDerivedClass2(arg1,arg2,arg3, ..., argn) {
            super(arg1,arg2,arg3, ..., argn)
        }    
    }

我的一个解决方案是将所有参数包装在一个类中，并将该类注入基类和所有派生类。这样，当向注入的类添加新参数时，它将被注入到所有派生类。

类似：

    class MyBaseClassSettings {
        @Inject
        MyBaseClassSettings(arg1,arg2,arg3, ..., argn) {
         this.arg1 = arg1
         ...
        }
    }

    class MyBaseClass {
        @Inject
        MyBaseClass(MyBaseClassSettings settings) {
            this.settings = settings;
        }

    }

    class MyDerivedClass1 extends MyBaseClass{
        @Inject
        MyDerivedClass1(MyBaseClassSettings settings) {
            super(settings)
        }    
    }

    class MyDerivedClass2 extends MyBaseClass{
        @Inject
        MyDerivedClass2(MyBaseClassSettings settings) {
            super(settings)
        }    
    }

假设 args 彼此并不真正相关（一个是与数据库的连接，另一个是为任务分配线程的助手，另一个负责部分实际逻辑，另一个负责地理配置班级...（只是示例））这个解决方案是否可以考虑？

如果是这样，包装类的好命名是什么？

Answer 1

务实地说，您的解决方案是可以的。您可以通过这种方式解决您的问题，而且不会花费您很多时间。

话虽如此，如果你有时间，你可能应该重构代码并将类拆分为多个类。

有 10 个依赖项表明你的类做得很多，并且被认为是代码异味（查看this）。你的班级似乎有太多的责任，因此违反了Single Responsibility Principle。考虑Aggregated Services 来解决您的问题。

根据您的情况，您可以选择现在进行建议的修复，然后在有时间时进行重构。看看Technical Debt的概念。

至于课程名称，您建议的名称就可以了。另一个建议是MyBaseClassDependencies。

Answer 2

是的，完全合理的解决方案和层次结构的常用习语。

你的名字就好了。另一种命名模式是BaseParams 或BaseClassParams 等。