Question

我正在尝试将 SimpleInjector与WebFormsMvp 结合使用。

为方便DI WebFormsMvp提供IPresenterFactory接口它包含Create方法，该方法提供演示者类型以解析和视图实例。
我需要将视图实例 构建器 演示者。演示者还有其他依赖，需要通过容器创建。

这是我到目前为止所得到的，但它并不理想该问题的正确解决方案是什么？

Presenter构造函数：

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接口上使View成为可写属性，而不是在演示者的构造函数中获取视图。这将使在演示者上设置视图变得非常容易。像这样：

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

不幸的是他们没有这样做，并且不可能扩展设计以允许这样做（没有使用反射做真正令人讨厌的事情）。

Simple Injector不支持向GetInstance()方法提供构造函数参数。有充分的理由，因为这通常会导致Service Locator反模式，并且您可以通过更改设计来解决这个问题。在你的情况下，你没有做那个古怪的设计，所以你不能改变它。

你对ResolveUnregisteredType的所作所为非常聪明。我自己也不会想到这个。由于我是Simple Injector背后的领导者，我能够说你所做的事情非常聪明： - ）

关于SimpleInjectorPresenterFactory。

的两点反馈意见

首先，您应该提供Container作为构造函数参数，因为您很可能需要向容器添加其他注册，并且您不想注册{{1}在Container内。

其次，您可以使用SimpleInjectorPresenterFactory来改进代码。这允许您摆脱全局锁定。锁可以防止任何演示者同时发生，这可能会减慢您的网站速度。

所以这是一个重构版本：

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; } } }的实现，你会看到很多缓存正在进行中。我不知道为什么他们这样做，但从性能的角度来看，你根本不需要Simple Injector这样的东西。也许我错过了什么，但我不明白为什么应该有缓存。

但更糟糕的是，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]; }（仅限.net 4）或考虑ConcurrentDictionary不可变，这意味着您创建原始字典的副本，添加新值，并用新的替换旧字典的引用。但是，在这种情况下，我可能只是锁定了整个事情。

这有点偏离主题，但只是想告诉： - ）

使用Simple Injector和WebForms MVP将运行时值传递给构造函数

1 个答案: