我正在尝试从抽象基类中删除服务定位器，但我不确定用什么来替换它。这是我得到的伪示例:

public abstract class MyController : Controller
{
    protected IKernel kernel;
    public MyController(IKernel kernel) { this.kernel = kernel); }

    protected void DoActions(Type[] types)
    {

        MySpecialResolver resolver = new MySpecialResolver(kernel);
        foreach(var type in types)
        {
            IMyServiceInterface instance = resolver.Get(type);
            instance.DoAction();
        }
    }
}

problem with this是派生类的实例化器不知道内核必须具有哪些绑定(bind)才能防止 MySpecialResolver 抛出异常。

这可能本质上是棘手的，因为我不知道从这里我必须解决哪些类型。派生类负责创建 types 参数，但它们并未在任何地方进行硬编码。 (类型基于属性在派生类的组合层次结构深处的存在。)

我试图通过延迟加载委托(delegate)来解决这个问题，但到目前为止我还没有想出一个干净的解决方案。

更新

这里确实有两个问题，一个是IoC容器被传递给 Controller ​​，充当服务定位器。这很容易删除——您可以使用各种技术将位置向上或向下移动到调用堆栈中。

第二个问题是困难的，当需求直到运行时才公开时，如何确保 Controller 具有必要的服务。从一开始就应该很明显:你不能!您将始终依赖于服务定位器的状态或集合的内容。在这种特殊情况下，再多的摆弄也无法解决 this article 中描述的问题。具有静态类型的依赖项。我认为我最终要做的是将惰性数组传递到 Controller 构造函数中，并在缺少所需的依赖项时抛出异常。

最佳答案

我同意@chrisichris 和@Mark Seemann 的观点。

从 Controller 中删除内核。我会稍微改变您的解析器组合，以便您的 Controller 可以消除对 IoC 容器的依赖，并允许解析器成为唯一担心 IoC 容器的项目。

然后我会让解析器传递到 Controller 的构造函数中。这将使您的 Controller 更易于测试。

例如:

public interface IMyServiceResolver
{
    List<IMyServiceInterface> Resolve(Type[] types);
}

public class NinjectMyServiceResolver : IMyServiceResolver
{
    private IKernal container = null;

    public NinjectMyServiceResolver(IKernal container)
    {
        this.container = container;
    }

    public List<IMyServiceInterface> Resolve(Type[] types)
    {
        List<IMyServiceInterface> services = new List<IMyServiceInterface>();

        foreach(var type in types)
        {
            IMyServiceInterface instance = container.Get(type);
            services.Add(instance);
        }

        return services;
    }
}

public abstract class MyController : Controller
{
    private IMyServiceResolver resolver = null;

    public MyController(IMyServiceResolver resolver) 
    { 
        this.resolver = resolver;
    }

    protected void DoActions(Type[] types)
    {
        var services = resolver.Resolve(types);

        foreach(var service in services)
        {
            service.DoAction();
        }
    }
}

现在您的 Controller 未耦合到特定的 IoC 容器。此外，您的 Controller 更易于测试，因为您可以模拟解析器并且根本不需要 IoC 容器来进行测试。

或者，如果您无法控制实例化 Controller 的时间，您可以稍微修改它:

public abstract class MyController : Controller
{
    private static IMyServiceResolver resolver = null;

    public static InitializeResolver(IMyServiceResolver resolver)
    {
        MyController.resolver = resolver;
    }

    public MyController() 
    { 
        // Now we support a default constructor
        // since maybe someone else is instantiating this type
        // that we don't control.
    }

    protected void DoActions(Type[] types)
    {
        var services = resolver.Resolve(types);

        foreach(var service in services)
        {
            service.DoAction();
        }
    }
}

然后您将在应用程序启动时调用它来初始化解析器:

MyController.InitializeResolver(new NinjectMyServiceResolver(kernal));

我们这样做是为了处理在 XAML 中创建的元素，这些元素需要解决依赖关系，但我们想像请求一样删除服务定位器。

请原谅任何语法错误:)

我正在写一篇博文系列，主题是在 View 模型中使用服务定位器调用重构 MVVM 应用程序，您可能会感兴趣。第 2 部分即将推出:)

http://kellabyte.com/2011/07/24/refactoring-to-improve-maintainability-and-blendability-using-ioc-part-1-view-models/

关于c# - 如何避免服务定位器反模式？，我们在Stack Overflow上找到一个类似的问题： https://stackoverflow.com/questions/6833995/