使用简单注入器abd WebFormsMVP传递运行时值给构造函数

本文关键字：运行时 构造函数 WebFormsMVP 简单 注入器 abd | 更新日期: 2023-09-27 18:01:39

我试图结合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;
        }
    }
}

使用简单注入器abd WebFormsMVP传递运行时值给构造函数

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所做的非常聪明。我自己是不会想到这些的。因为我是Simple Injector背后的首席开发者，所以我可以说你所做的非常聪明:-)

关于你的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不可变，这意味着您创建原始字典的副本，添加新值，并用新值替换对旧字典的引用。然而，在这种情况下，我可能只是锁定了整个东西。

这有点离题，但只是想告诉:-)

相关文章：