Как правильно пользоваться фабрикой?

Question

[LightSouls]

Часто в различных проектах, которые используют DI, встречаю примерно такую реализацию фабричного метода с использованием enum :

class VehicleFactory
{
    public Vehicle Create(VehicleType type)
    {
        switch (type)
        {
            case Car:
                return new Car();
            case Truck:
                return new Truck();
        }
    }
}

Такой класс придется менять если появится какой-то новый тип. Но если добавить в базовый Vehicle метод GetType() и воспользоваться DI, то класс менять не придется сколько бы новых типов не появилось:

class VehicleFactory
{
    private readonly List<Vehicle> vehicles;  // внедряется через конструктор

    public Vehicle Create(VehicleType type)
    {
         vehicles.FirstOrDefault(vehicle => vehicle.GetType() == type);
    }
}

Правильно ли так делать и есть ли в этом решении какие - либо минусы ? На мой взгляд это лучше чем свич на несколько сотен строк, но не видел чтобы кто-то так делал, не понимаю почему

Comments

[Adamos]

внедряется через конструктор

И вам придется либо иметь под рукой этот полный список каждый раз, когда вы вызываете фабрику, либо, внезапно, менять ее класс (только не метод Create, а конструктор - шило на мыло) каждый раз при изменении этого списка.

[LightSouls]

Adamos, ну этим и будет заниматься DI фреймворк. Пример с Vehicle я выбрал неудачный

[Adamos]

LightSouls, чем "этим"? Поддержанием списка классов где-то в атсрале, лишь бы не там, где он реально используется (а это, внезапно, именно этот метод)?

[LightSouls]

Adamos, под 'этим' имел ввиду внедрять ,существующие в DI контейнере объекты в этот список. Но с Vehicle как я уже сказал пример неудачный, я имел ввиду сервисы которые есть в DI контейнере

[Adamos]

LightSouls, если у вас уже есть экземпляры классов и вы их собираете в контейнер - при чем тут вообще фабрика?
Напоминаю смысл фабрики: она используется, чтобы создавать объекты дочерних классов кодом, который ничего не знает о дочерних классах и обращается с созданным, как с объектами родительского класса.
У вас же код уже сам насобирал дочек... и что вы еще собрались инкапсулировать?

[LightSouls]

Adamos, Понимаю, но как тогда быть в ситуациях когда нужно вернуть конкретный сервис, по какому нибудь параметру ?

[Adamos]

LightSouls, создать его из какого-нибудь класса или достать из какого-нибудь кэша ;)
Вы же понимаете, что вопрос настолько некорректен, что вообще не имеет смысла?

[LightSouls]

Adamos, ну, думаю в этом и был смысл второго примера

[Adamos]

LightSouls, ваш второй пример - это банальный контейнер.
Который не решает ничего вообще и просто не заслуживает отдельного класса.

[LightSouls]

Adamos, Ну, если в нескольких классах понадобился такой метод, придется дублировать код.Но я понял, это не фабрика

[Adamos]

LightSouls, не в этом дело. Просто нет смысла выносить извлечение из контейнера в класс, отдельный от заполнения этого контейнера.

Answer 1

[Сергей Водаков]

Если я правильно понял ход ваших мыслей, то у вас в классе будет храниться уже готовый список инициализированных объектов, и ваша "фабрика", при обращении к методу Create, вместо того чтобы создавать объект нужного типа, будет каждый раз отдавать один и тот же, лежащий в списке.
Нужно объяснять, какие будут последствия?

Comments

[Adamos]

Ну, все не так страшно.
В вопросе метод Create во втором варианте вообще ничего не возвращает, так что никаких последствий ;)

[Сергей Водаков]

Adamos, и то правда, не смотрел на проблему с этой стороны =)

[LightSouls]

Да, я неправильно показал первый случай, имелась ввиду фабрика возвращающая не новые объекты, а конкретные сервисы

[Adamos]

LightSouls, которые вы в конструкторе будете инициализировать всей кучей независимо от того, понадобятся они вообще приложению или нет. Волшебно.

[LightSouls]

Adamos, Действительно, об этом я не подумал