Question

我有一个需要很多依赖项的类 - 按照我的意思，很多依赖项都是8或更多。它在IDE中看起来很难看，因为它会破坏行，在一种情况下我有3行构造函数签名：

第一步

我决定停止使用这种方法并创建依赖项字典。我取得了什么？漂亮的构造函数签名，但更容易让运行时异常。

第二步

public class FooClass
{
    public FooClass(IDependency1 dependency1, IDependency2 dependency2, ..., IDependency8 dependency8, IDependency9 dependency9)
    {
        ...
    }

    ...
}

但是注册这种类型现在很难看。作为示例，我正在使用AutoFac，但任何其他依赖项容器都具有相同的行为。

public class FooClass2
{
    private IDictionary<Type, object> dependencyDictionary;

    public FooClass2(IDictionary<Type, object> dependencyDictionary)
    {
        this.dependencyDictionary = dependencyDictionary;
        ...
    }

    ...

    public T GetObject<T>()
    {
        return (T)this.dependecyDictionary.FirstOrDefault(t => t.Key == typeof(T));
    }

    // USAGE
    public void FooMethod()
    {
        IDependency1 = this.GetObject<IDependency1>();
        ...
    }
}

当然，为了避免使用第二种方法，我可以创建一个模型，模板或者随意调用它，但它会生成许多其他类，这些类仅用于指定依赖项。

第3步

var builder = new ContainerBuilder();
builder.Register(c => new FooClass2(new Dictionary<Type, object>{
    {typeof(IDependency1), c.Resolve<IDependency1>()},
    {typeof(IDependency2), c.Resolve<IDependency2>()},
    ...
    {typeof(IDependency8), c.Resolve<IDependency8>()},
    {typeof(IDependency9), c.Resolve<IDependency9>()},
})).As<FooClass2>();

我知道有可能通过属性注入，但我想避免它。上3种方法中的哪一种被认为是一种好的做法？你知道还有其他方法可以将大量的依赖关系传递给一个类吗？

Answer 1

显然你的Step1不好，因为它有太多的依赖关系。我们还需要其他方式。

我强烈反对你的第二步。因为使用依赖注入的想法是让开发人员明白你的依赖关系。通过您的方法，您很难理解api。

第3步只是一个黑客攻击。它负责任地推动了其他一些需要丑陋构造函数的类。也不推荐它。

<小时/> 当你有太多的依赖关系时，根据定义你的类做了太多事情。这也意味着您违反了Single responsibility principle。

确定课程的责任，重构他们遵循单一责任原则。那么你的课不需要太多的依赖。

Mark Seemann在这个主题中有一篇很好的文章Refactoring to Aggregate Services。

依赖注入 - 参数的数字

1 个答案: