Блог

В моем коде есть классы som factory, и мне не нравится вызов Structermap. Как мне сделать это правильно?

 class ManagerBaseFactory
{
    public ManagerBase GetInstance(SomeEnum e)
    {
        Type t;

        switch (e)
        {
            case SomeEnum.A:
                t = typeof(Manager1);
            case SomeEnum.B:
                t = typeof(Manager2);
            case SomeEnum.C:
                t = typeof(Manager3);
        }
        return (ManagerBase)StructureMap.ObjectFactory.GetInstance(t);
    }
}
  

Я не вижу проблемы в использовании structure map таким образом. Что вам не нравится в текущем решении?

Вы могли бы использовать именованные экземпляры, для которых вам даже фабрика больше не понадобится. ссылка на structuremap

 public class ServicesRegistry : Registry
{
    public ServicesRegistry()
    {
        For< ManagerBase >().Use< Manager1 >().Named("A");
        For< ManagerBase >().Use< Manager2 >().Named("B");
        For< ManagerBase >().Use< Manager3 >().Named("C");
    }
}
  

Вы можете получить правильный, просто вызвав

 SomeEnum e = SomeEnum.A;
ObjectFactory.GetNamedInstance<ManagerBase>(e.ToString());
  

Если вы абсолютно не хотите, чтобы ваш ManagerBaseFactory использовал контейнер, вы могли бы предоставить ему доступ ко всем реализациям ManagerBase из контейнера и заставить его вернуть правильную. Когда StructureMap видит IEnumerable типа в конструкторе, он вводит все экземпляры этого типа, о которых он знает.

 class ManagerBaseFactory
{
    private readonly IEnumerable<ManagerBase> _managers;

    public ManagerBaseFactory(IEnumerable<ManagerBase> managers)
    {
        _managers = managers;
    }

    public ManagerBase GetInstance(SomeEnum e)
    {
        Type t;

        switch (e)
        {
            case SomeEnum.A:
                t = typeof(Manager1);
                break;
            case SomeEnum.B:
                t = typeof(Manager2);
                break;
            case SomeEnum.C:
                t = typeof(Manager3);
                break;
            default:
                return null;
        }
        return _managers.FirstOrDefault(m => m.GetType().Equals(t));
    }
}
  

И, конечно, вам нужно убедиться, что все ваши реализации ManagerBase загружены в контейнер:

 var container = new Container(x =>
{
    x.Scan(scan =>
    {
        scan.TheCallingAssembly();
        scan.AddAllTypesOf<ManagerBase>();
    });
});
  

Не ясно, против какой части вы возражаете. Одно из улучшений, которое вы могли бы внести, — это иметь ManagerBaseFactory take в IContainer своем конструкторе, который вы можете использовать вместо ObjectFactory статического шлюза. Затем вы можете извлечь ManagerBaseFactory из контейнера, и контейнер сам внедрится в фабрику. Фабрику будет проще тестировать без статической зависимости.

Вы также могли бы поместить операторы return непосредственно в switch, чтобы вы могли воспользоваться преимуществами generics:

 case SomeEnum.A:
  return _container.GetInstance<Manager1>();