Поиск best practice Laravel. Размещение всех классов в силгентон хорошо или плохо?

Question

[Иван]

В поисках бест-практикс, просматривая опен соурс проекты наткнулся на одно решение в одном очень популярном проекте.
В самом решение нечего особенного нет, оно указанно в документации. Интересует вопрос на сколько такой подход хороший или плохой.
Суть решения, добавить все классы проекта в контейнер как сингелтоны через свойство public $singletons. Т.е. по факту все классы, всю бизнес логику Слой Сервис, все добавили в контейнер как сингелтоны, таким методом.
https://github.com/monicahq/monica/blob/master/app...

Есть ли какие-то последствия у такого подхода? У меня есть конечно свои рассуждения по данному подходу, очень бы хотел услышать еще чье-нибудь мнение.
Спасибо.

Comments

[JhaoDa]

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

[dmitriy]

давай свои рассуждения, может они совпадают с нашими

[JhaoDa]

Тут никто не ответил за 4 минуты и автор создал новый вопрос — Зачем добавлять классы в контейнер в Laravel? . Нормально, так и надо.

[Иван]

JhaoDa, нет. Я изначальное не хотел объединять эти вопросы, что бы получить ответ на оба и не размазать их в один ответ о контейнере и зачем он нужен. Этот вопрос конкретно про определенный подход. Видит ли кто-то в таком подходе плюсы и почему на гитхабе у него там много звезд.

[JhaoDa]

Иван, звёзды на гитхабе не из-за этого конкретного подхода, а вообще — наверное, хорошо написано.

Свои рассуждения ты озвучивать не планируешь, да?

[Иван]

JhaoDa, я конечно больше относился к этому сервису не как к форуму, а как задай вопрос - получи ответ. Но не суть конечно могу. Правда в другом вопросе мне скинули ссылку на хорошую доку php-di.org/doc/understanding-di.html и после этой доки данный подход мне вообще стал еще больше не понятен и я бы не стал его использовать в своих проектах. Вообще я сейчас хочу понять для себя на сколько часто и как мне использовать контейнер зависимостей, могу ли я использовать new или мне надо все делать через контейнер и так правильно и лучше. Первые мои рассуждения конечно мне не понятно зачем их делать всех сингелтонами, хотя с другой стороны здесь четкая и понятная логика "типа используем контейнер для всех наших классов" хотя с другой стороны контейнер мог их и сам создать и без объявления, соответственно может есть какой-то посыл их объявить сингелтонами... Я так могу долго.... но это мои рассуждения, а хотелось бы почитать ответы людей которые умеют этот правильно готовить и узнать почему так правильно.

[JhaoDa]

Иван, при чём тут форум? Слышал выражение типа «Правильно заданный вопрос содержит в себе 50% ответа»? Рассуждения надо не мне в каментах писать, а добавить в вопрос.

[Иван]

JhaoDa, на мой взгляд это усложнит вопрос, а для тех кто не уверен в свое ответе собьет его. Тут же все просто, есть опен соурс проект 7 тыс. звезд. Есть четкий и понятный подход засунуть все в контейнер как сингелтон и использовать. И очень просто вопрос хорошо это или плохо. Зачем тут мои рассуждения?

[JhaoDa]

Иван, зачем тогда ты упомянул свои рассуждения?

[Иван]

JhaoDa, это фраза тебя как-то обидела, что у меня есть свои рассуждения? Прости! Просто добавил, даже не знаю почему, лексическое соединение вопроса и просьбы людей высказать свое мнение и свои рассуждения.

[JhaoDa]

Иван, почему это могло меня обидеть? Экстраполируешь свои чувства на других? Я хочу, чтобы люди учились задавать вопросы, а они не учатся.

[Иван]

JhaoDa, мне кажется ты ко мне не справедлив. Фраза "у меня есть свои рассуждения", не делает мой вопрос плохим. Если это не так объясни и я буду задавать вопросы лучше благодаря тебе.

[vism]

Иван, Зачем кормишь Тролля то?

[Иван]

vism, хотел задобрить :) Репутация хорошая, да и в подписи у троля "LaravelRUS Team", думал совет дельный даст, мнение экспертное напишет. Но увы, оказалось он только рассуждать о правильности вопроса может.

[vism]

Иван, он тут откормился хорошо. Хотя на самом деле я его понимаю. обычно советуешь, расписываешь , а автор джун отмечает ответы такие же джуновские, а качественные ответы так и висят не принятыми.
Я пожалуй его понимаю.

Answer 1

[hack504]

нет никакого смысла описывать все синглетоны в контейнере. Если же этот контейнер умеет инжектить зависимости, то тогда появляется смысл для переопределение зависимости: изменяя свойство public $singletons для определенных классов мы можем получать другую реализацию на лету, Но опять же не вижу смысла описывать их все - это же синглетоны, можно было бы просто реализовать через метод:

$AppServiceProvider->delegate('\App\Services\Account\Company\CreateCompany',  '\App\Services\Account\Company\MyCreateCompany');

Answer 2

[dmitriy]

Данный способ говорит о том что зависимость не нужно будет создавать каждый раз при ее использовании а использовать один объект-одиночку