使用Simple Injector將構造函數傳遞給構造函數abd WebFormsMVP

Question

我試圖將SimpleInjector與WebFormsMvp相結合。

爲了方便DI WebFormsMvp提供了IPresenterFactory接口。
它包含Create方法，該方法提供演示者類型以解析和視圖實例。
我需要注入的視圖實例爲的構造的主持人。
主持人也有其他依賴關係需要創建容器。

這是我到目前爲止，但它並不理想。
什麼是正確的解決方案的問題？

主講構造：

public FooPresenter(IFooView view, IClientFactory clientFactory) : base(view) 

廠：

public class SimpleInjectorPresenterFactory : IPresenterFactory 
{ 
    private readonly Container _container; 
    private IView _currentView; 

    public SimpleInjectorPresenterFactory() 
    { 
     _container = new Container(); 

     Func<Type, bool> isIView = 
      type => typeof(IView).IsAssignableFrom(type); 

     _container.ResolveUnregisteredType += (s, e) => { 
      if (isIView(e.UnregisteredServiceType)) 
       e.Register(() => _currentView); 
     }; 
    } 

    public IPresenter Create(Type presenterType, Type viewType, IView viewInstance) 
    { 
     lock (_currentView) 
     { 
      _currentView = viewInstance; 
      return _container.GetInstance(presenterType) as IPresenter; 
     } 
    } 
} 

Answer 1

WebFormsMvp迫使你採取在主持人的構造函數的觀點，但是這會觸發循環引用。如果您查看不同容器的工廠實現，您會看到，對於每個容器，他們都會在設計中採用不同的技巧來解決這個問題。例如，使用unity時，他們創建一個子容器並在子容器中註冊該視圖，並使用該子容器解析演示者。相當奇怪和性能沉重。

WebFormsMvp的設計者應該在IPresenter接口上使視圖成爲可寫屬性，而不是在呈現器的構造函數中使用視圖。這會讓主持人的觀點變得難以置信。事情是這樣的：

public IPresenter Create(Type presenterType, IView view) 
{ 
    var presenter = (IPresenter)_container.GetInstance(presenterType); 
    presenter.View = view; 
    return presenter; 
} 

遺憾的是他們並沒有這樣做，這是不可能的擴展設計，讓這個（沒有做使用反射真的討厭的東西）。

簡單噴油器不支持向GetInstance()方法提供構造函數參數。出於好的理由，因爲這通常會導致Service Locator反模式，並且您可以隨時通過更改設計來解決此問題。就你而言，你並沒有做出這種古怪的設計，所以你不能改變它。

你對ResolveUnregisteredType做了什麼很聰明。我不會想到這件事。由於我是簡單注射器背後的主要開發人員，因此我可以說您所做的確實非常聰明:-)

只需提供關於您的SimpleInjectorPresenterFactory的兩點反饋意見。

首先，你應該提供Container作爲構造函數的參數，因爲這將是非常可能的，你需要給其他註冊加入到容器中，並且你不想註冊SimpleInjectorPresenterFactory內Container。

其次，您可以通過使用System.Threading.ThreadLocal<IView>來改進代碼。這可以讓你擺脫全局鎖定。該鎖可防止任何演示者同時發生，這可能會降低您的網站速度。

所以這裏有一個重構的版本：

public class SimpleInjectorPresenterFactory : IPresenterFactory { 
    private readonly Container _container; 
    private ThreadLocal<IView> _currentView = new ThreadLocal<IView>(); 

    public SimpleInjectorPresenterFactory(Container container) { 
     _container = container; 

     _container.ResolveUnregisteredType += (s, e) => { 
      if (typeof(IView).IsAssignableFrom(e.UnregisteredServiceType)) { 
       e.Register(() => _currentView.Value); 
      } 
     }; 
    } 

    public IPresenter Create(Type presenterType, Type viewType, 
     IView viewInstance) 
    { 
     _currentView.Value = viewInstance; 

     try { 
      return _container.GetInstance(presenterType) as IPresenter; 
     } finally { 
      // Clear the thread-local value to ensure 
      // views can be disposed after the request ends. 
      _currentView.Value = null; 
     } 
    } 
} 

如果你看一下UnityPresenterFactory的實施，你看到很多緩存在那裏往前走。我不知道他們爲什麼這麼做，但從性能的角度來看，根本不需要這種簡單的噴油器。也許我錯過了一些東西，但我不明白爲什麼應該有一個緩存。

但更糟的是，UnityPresenterFactory中存在併發錯誤。看看這個方法：

private Type FindPresenterDescribedViewTypeCached(Type presenter, 
    IView view) 
{ 
    IntPtr handle = presenter.TypeHandle.Value; 
    if (!this.cache.ContainsKey(handle)) 
    { 
     lock (this.syncLock) 
     { 
      if (!this.cache.ContainsKey(handle)) 
      { 
       Type viewType = CreateType(presenter, view); 
       this.cache[handle] = viewType; 
       return viewType; 
      } 
     } 
    } 
    return this.cache[handle]; 
} 

乍一看，這個代碼看起來不錯，因爲雙重檢查鎖來實現。不幸的是，緩存（字典）是從鎖外部讀取的，而它是在鎖內更新的。這不是線程安全的。相反，開發人員應該將整個事物封裝在一個鎖中，使用ConcurrentDictionary（僅限.net 4）或考慮cache不可變，這意味着您創建原始字典的副本，添加新值並替換用新的參考舊字典。但是，在這種情況下，我可能會鎖定整個事情。

這是一個有點題外話，但只是想告訴:-)