服务定位器比依赖注入更容易使用？

Question

我正在开发的应用程序依赖 Autofac 作为 DI 容器，让我决定使用它的原因之一是委托工厂功能（请参阅 here)

这适用于我需要多次重新创建相同元素的所有情况，就像某些报告的情况一样和相关屏幕。一些报告（甚至是相同类型的报告）是同时执行的，但它们仅通过用户定义的参数进行更改，因此（我认为）注入工厂以创建实例、传递自由参数并将其余部分留给应用。

问题在于，每个报告都由数量可变的子报告（任务）组成，并且每个任务都实现一个 ITask 接口。每个报告最多可以有 50 个不同的任务可供使用，每个任务都封装了精确的业务操作。我的一种选择是注入委托工厂并在需要时创建它们。

这些任务必须由工厂动态生成，例如：

var myTaskA = _taskFactoryConcreteTaskA();
var myTaskB = _taskFactoryConcreteTaskB();
var myTaskC = _taskFactoryConcreteTaskC();
...
var myTaskZZ = = _taskFactoryConcreteTaskZZ();

需要大量手动接线（委托、构造函数、支持字段等），而类似的

var myTaskA = _taskFactory.Create<ConcreteTaskA>();
var myTaskB = _taskFactory.Create<ConcreteTaskB>();
var myTaskC = _taskFactory.Create<ConcreteTaskC>();
...
var myTaskZZ = _taskFactory.Create<ConcreteTaskZZ>();

工作量会少得多，特别是如果 _taskFactory 包装容器，如 这篇文章，但这基本上意味着我正在使用服务定位器创建我的任务。

我还有哪些其他选择可能适合解决此问题？

（注意：我很有可能完全偏离了轨道，我必须阅读更多有关 DI 的内容，在这种情况下，任何贡献都将更加重要）

Answer 1

由于问题中指出的工厂不接受任何参数，因此使用工厂有一种泄漏抽象的味道。正如 Nicholas Blumhardt 在他的回答中指出的那样，更好的方法可能是简单地将每项任务注入消费者。

在这种情况下，由于所有任务都实现相同的接口，因此您可以组合它们，而不是注入最多 50 个不同的 ITask 实例：

public class MyConsumer
{
    private readonly IEnumerable<ITask> tasks;

    public MyConsumer(IEnumerable<ITask> tasks)
    {
        this.tasks = tasks;
    }

    public void DoSomething()
    {
        foreach (var t in this.tasks)
        {
            // Do something with each t
        }
    }
}

或者，您可以组合 ITask 序列> 变成 Composite，这实际上是我首选的解决方案：

public CompositeTask : ITask
{
    private readonly IEnumerable<ITask> tasks;

    public CompositeTask(IEnumerable<ITask> tasks)
    {
        this.tasks = tasks;
    }

    // Implement ITask by iterating over this.tasks
}

这将简化消费者并转变事实有一个实施细节中需要执行多个任务：

public class MyConsumer
{
    private readonly ITask task;

    public MyConsumer(ITask task)
    {
        this.task = task;
    }

    public void DoSomething()
    {
        // Do something with this.task
    }
}

Answer 2

一种值得研究的方法是将问题分解为使用一组相关任务的“工作单元”：

public class WorkItem1 : ISomeWork
{
    public WorkItem1(Task1 t1, Task2 t2...) { }
    public void DoSomething() { ... }
}

然后，您对工厂的使用将趋向于 someWorkFactory().DoSomething()，可能用于几种不同种类的“东西”。

一个类对工厂或其他任何东西有大量的依赖关系，通常表明有更小、更集中的类等待被发现来分解工作。

希望这有帮助。