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

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包装了容器，如this other post所示，但它也意味着我正在使用服务定位器来创建我的任务。

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

（注：有一个很好的机会，我完全偏离轨道，并且我要读了很多有关DI，在这种情况下，任何的贡献将更为重要）

Answer 1

一种方法值得研究的是打破工作的问题into'units”使用一组相关任务：

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

然后，你的工厂的使用将回落向someWorkFactory().DoSomething()，可能是几个不同类型的‘东西’。

有大量的相关性，在工厂或其他任何一类，通常指向那里正在变小，等待更集中的类被发现，打破了工作。

希望这会有所帮助。

Answer 2

由于问题中提到的工厂没有采取任何论据，所以使用工厂的气味漏泄抽象。正如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 
     } 
    } 
} 

或者，您也可以撰写的ITasks序列划分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 
    } 
}