ASP.NET MVC ContactsManager 教程中是否有解决依赖注入循环问题的好/正确方法？

Question

如果您不知道我在说什么，请阅读教程 并尝试自己添加依赖注入，或者尝试一下我对问题的解释。

注意：此问题不在 ASP.NET 原始教程的范围内。本教程仅表明所使用的模式是依赖注入友好的。

问题基本上是控制器、ModelStateWrapper 和 ContactManagerService 之间存在依赖循环。

1. ContactController 构造函数采用 IContactManagerService。
2. ContactManagerService 构造函数采用 IContactManagerRepository （不重要） 和 IValidationDictionary （由 ModelStateWrapper 实现）。
3. ModelStateWrapper 构造函数采用 ModelStateDictionary（这是控制器上名为“ModelState”的属性）。

所以依赖循环是这样的：Controller >服务>模型状态包装器> 如果您尝试向控制器

添加依赖注入，它将失败。所以我的问题是；我该怎么办？其他人已经发布了这个问题，但答案很少，各不相同，而且看起来都有点“hack-ish”。

我当前的解决方案是从 IService 构造函数中删除 IModelStateWrapper 并添加一个 Initialize 方法，如下所示：

public class ContactController : Controller
{
    private readonly IContactService _contactService;

    public ContactController(IContactService contactService)
    {
        _contactService = contactService;
        contactService.Initialize(new ModelStateWrapper(ModelState));
    }

    //Class implementation...
}

public class ContactService : IContactService
{
    private IValidationDictionary _validationDictionary;
    private readonly IContactRepository _contactRepository;

    public ContactService(IContactRepository contactRepository)
    {
        _contactRepository = contactRepository;
    }

    private void Initialize(IValidationDictionary validationDictionary)
    {
        if(validationDictionary == null)
            throw new ArgumentNullException("validationDictionary");

        _validationDictionary = validationDictionary;
    }

    //Class implementation...
}

public class ModelStateWrapper : IValidationDictionary
{
    private readonly ModelStateDictionary _modelState;

    public ModelStateWrapper(ModelStateDictionary modelState)
    {
        _modelState = modelState;
    }

    //Class implementation...
}

使用此构造，我可以像这样配置我的统一容器：

public static void ConfigureUnityContainer()
{
    IUnityContainer container = new UnityContainer();

    // Registrations
    container.RegisterTypeInHttpRequestLifetime<IContactRepository, EntityContactRepository>();
    container.RegisterTypeInHttpRequestLifetime<IContactService, ContactService>();

    ControllerBuilder.Current.SetControllerFactory(new UnityControllerFactory(container));
}

不幸的是，这意味着必须调用服务上的“Initialize”方法 <由控制器构造函数手动执行。有更好的办法吗？也许我在哪里以某种方式将 IValidationDictionary 包含在我的统一配置中？我应该切换到另一个 DI 容器吗？我错过了什么吗？

Answer 1

作为一般考虑，循环依赖表明存在设计缺陷 - 我想我可以安全地说这一点，因为您不是代码的原始作者:)

我不认为 Initialize 方法是一个好的解决方案。除非您正在处理加载项场景（您不是），否则方法注入不是正确的解决方案。您几乎已经弄清楚了这一点，因为您发现需要手动调用它并不令人满意，因为您的 DI 容器不能。

除非我完全错误，否则 ContactController 在调用其 Action 方法之前不需要 IValidationDictionary 实例？

如果这是真的，最简单的解决方案可能是定义一个 IValidationDictionaryFactory 接口并让 ContactController 构造函数采用该接口的实例。

该接口可以这样定义：

public interface IValidationDictionaryFactory
{
    IValidationDictionary Create(Controller controller);
}

控制器上需要 IValidationDictionary 实例的任何 Action 方法都可以调用 Create 方法来获取该实例。

默认实现如下所示：

public class DefaultValidationDictionaryFactory : IValidationDictionaryFactory
{
    public IValidationDictionary Create(Controller controller)
    {
        return controller.ModelState;
    }
}

Answer 2

稍微改变/改进设计，如下所示： http://forums.asp.net /t/1486130.aspx

Answer 3

每个控制器都有一个虚拟方法Initialize来执行类似的操作。

我认为没有更好的方法，因为 IValidationDictionary 是当前请求/控制器/模型状态和 IContactService 之间的抽象层。使用构造函数注入将控制器模型状态注入到服务中，然后将服务注入到控制器中是不可能的。一个必须是第一。

可能有一种使用属性注入的方法吗？但我认为这也会很复杂。