服務定位器比依賴注入更容易使用？

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 
    } 
}