Dependency Injection (Какой способ лучше использовать)?

Question

[BRONNER]

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

Handler(Main main) {
        this.main = main;
        this.service = main.getService()
    }
//или
   Handler(Main main, Service service) {
        this.main = main;
        this.service = service;
    }

Answer 1

[Сергей Протько]

1-ый вариант валиден если бы было так:

Handler(ServiceFactory factory) {
        this.factory = factory;
        this.service = factory.createService()
    }

если оба сервиса существуют в единственном экземпляре - то второй вариант. Если нужно делать шаблоны типа Registry и хранить там пул сервисов, то тогда опять же как и в случае с фабриками - первый вариант.

Comments

[BRONNER]

Ага, примерно понял мысль)

[Сергей Протько]

@BRONNER ну и да, основная мысль сервисов-фабрик, если сервис зависит от кучи всего и лишь чуть чуть от данных которые должны предоставить вы (например сервис хеширования паролей зависит от конкретной имплементации модели пользователя) то логично сделать сервис, который имеет все необходимое для конструирования и лишь запрашивает остаток у пользователя. В этом случае мы не нарушаем основных принципов и все хорошо покрывается тестаим.

[BRONNER]

@Fesor да, думаю вся беда и камни, как раз в недопонимании принципов и проектирования приложения... иначе обычно всё идёт, как по маслу, пока опять не начнется отклонение

[Сергей Протько]

@BRONNER почитайте про GRASP и SOLID. Ну и к здравому смыслу прислушивайтесь тоже.

[BRONNER]

@Fesor благодарю, обязательно почитаю, про первое даже не слышал, кошмар! как же любят все эти сокращения на западе, одно краше другого)))

Answer 2

[Павел Соловьёв]

второй вариант, так как в первом сервис зависит от Main, во втором случает можно подставить любой нужный сервис

Comments

[BRONNER]

У меня вообще Main, как и сервис существуют в единственном экземпляре, и сервис создается в конструкторе Main, как и Handler, в который передаются Main и свежесозданный Service, вот решил не злоупотреблять синглтоном, да и вообще, мне он как-то мало понятен, затруднят чтение только

[BRONNER]

@pavel_salauyou ну и да, благодарю за поддержку моих размышлений)

Answer 3

[FoxInSox]

Второй лучше конечно, но в обоих случаях это "ручной" DI. Как ни крути, но где-то есть зависимость от конструкторов Main и Service. В идеале DI должны заниматься не вы, а какой-либо фреймворк и/или утилиты генерирующие код и тогда будет примерно так:

@Inject Main main;
@Inject Service service;

Handler(){

}

Comments

[Сергей Протько]

DI а не DIC, это Dependency Inversion, так что все хорошо. Это просто принцип.

[BRONNER]

@Fesor Спасибо, не знал про это, почитаю

[Сергей Протько]

@BRONNER заодно прочитайте про принцип единой ответственности из которого как раз и вытикает что созданием объектов должен заниматься кто-то один. Ну и еще шаблон информационный эксперт.