Типа setter injection это плохо?

это плохо и этому нет никакого оправдания. Вообще.

Типа наследники натворят чего

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

это плохо и этому нет никакого оправдания. Вообще.

Ты логер делаешь всегда обязательной зависимостью?

Ты логер делаешь всегда обязательной зависимостью?

нет

аще никогда не делаю обязательным

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

у меня есть кейс с абстрактным классом и пару методов вроде markComplete()/markInComplete() и стратегия уведомлений, пока боли особо небыло

но, чую, из симфони чатика надо сюда сообщеньки форвардить... одно и то же каждый раз...

у меня есть кейс с абстрактным классом и пару методов вроде markComplete()/markInComplete() и стратегия уведомлений, пока боли особо небыло

я не топлю за отказ от абстрактных классов, если что)

Это неудачный пример был. А что если там были бы posts. Ну и тпа какой-то setPosts в абстрактном, а в реализациях setPosts может отличаться(типа что-то докидывать) Тоже бы за такое руки оторвал?))

привет всем, чтобы system запустить и использовать сторонюю программу на сервере надо в php 7 safe mode отключать? вроде это убрали с 5,4 ?

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

и делать когда понимаешь риски

так-то у меня у самого саги, к примеру - абстрактный класс. Но у них полностью приватный стейт.

обычно разговор за сеттеры всегда плавно перетекает в мутабельность и всякие нехорошие штуки которые можно сделать с языком

не вижу ничего плохого в абстрактных классах, если человек пользуется мозгом, а не сеттит коннекты к базе данных, "что бы убрать копипасту"

выдохнул спокойно :)

Ок. смысл, вроде, понял

это плохо и этому нет никакого оправдания. Вообще.

вообще то есть

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

и я вставил костыл с сетером)))

но вообще да это полная хуйня

как по мне setter injection весьма хорошая штука, ибо мне незачем в конструкторе требовать то, что можно забить каким-нибудь Null-обджектом, то, что не сильно влияет на работу

без коннекта к бд, скорее всего класс не обойдется, а вот без какого-нибудь логера или мейлера запросто

как по мне setter injection весьма хорошая штука, ибо мне незачем в конструкторе требовать то, что можно забить каким-нибудь Null-обджектом, то, что не сильно влияет на работу

Может тогда тебе вообще этот объект не нужен в данном?

Херь какая-то

От обоих, если что)

Может тогда тебе вообще этот объект не нужен в данном?

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

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

чо мешает при надобности запилить прокси к твоей либе?

или даже декоратор

да это всё равносильные выходы, в целом

а еще есть АОП))

https://habr.com/post/165329/

https://github.com/mmasiukevich/service-bus/blob/master/src/Sagas/Configuration/SagaEventListenerProcessor.php Нужен логгер - передай. Не нужен - ничё не изменится. В симфоне чате все это уже сотню раз обсасывали

https://habr.com/post/165329/

Оч опасная штука

От обоих, если что)

мою историю нужно воспринимать с некоторой долей иронии, но тем не менее она реальная)

https://github.com/mmasiukevich/service-bus/blob/master/src/Sagas/Configuration/SagaEventListenerProcessor.php Нужен логгер - передай. Не нужен - ничё не изменится. В симфоне чате все это уже сотню раз обсасывали

нужен логгер — передай в setLogger, нехер конструктор засирать, чтобы вот такую хуйню не создать https://github.com/pepe1518/magento2/blob/master/vendor/magento/module-catalog/Model/Product.php#L386

нужен логгер — передай в setLogger, нехер конструктор засирать, чтобы вот такую хуйню не создать https://github.com/pepe1518/magento2/blob/master/vendor/magento/module-catalog/Model/Product.php#L386

Чушь собачья

ох ты ж ебана

нужен логгер — передай в setLogger, нехер конструктор засирать, чтобы вот такую хуйню не создать https://github.com/pepe1518/magento2/blob/master/vendor/magento/module-catalog/Model/Product.php#L386

хм...а мы ругали битрикс

Это ошибка проектирования. Если бы там были сеттеры, лучше бы не стало

Ащн ни капли

но вообщето сервисы должны быть или стейтлесс или иммутабельными, а setLoger просто хуй кладет на этот принцип

но вообщето сервисы должны быть или стейтлесс или иммутабельными, а setLoger просто хуй кладет на этот принцип

а setContainer ;x ?

но вообщето сервисы должны быть или стейтлесс или иммутабельными, а setLoger просто хуй кладет на этот принцип

Именно

Это ошибка проектирования. Если бы там были сеттеры, лучше бы не стало

конкретно в этом случае все зависимости обязательные, а теперь добавь ко всем зависимостям = null в кострукторе и увидишь тотальный пиздец

а setContainer ;x ?

не понял, что это? контейнер заинджектить?

не понял, что это? контейнер заинджектить?

https://api.symfony.com/master/Symfony/Component/DependencyInjection/ContainerAwareInterface.html#method_setContainer

конкретно в этом случае все зависимости обязательные, а теперь добавь ко всем зависимостям = null в кострукторе и увидишь тотальный пиздец

Сеттеры мне тут чем помогут?

Сеттеры мне тут чем помогут?

как минимум разгрузить конструктор от опционального говна

https://api.symfony.com/master/Symfony/Component/DependencyInjection/ContainerAwareInterface.html#method_setContainer

аа, ну эт хуйню я не использую

как минимум разгрузить конструктор от опционального говна

Схуяли?

аа, ну эт хуйню я не использую

там еще ContainerAwareTrait есть для этих целей

твой пример с Logger $logger = null — уже засирание сраным говном

Схуяли?

видимо размазав все это гавно, по всему файлу, и проекту в целом

которому там не место

Ясна. Ларавель девелопер

там еще ContainerAwareTrait есть для этих целей

эх соскучился я по симфе

аа, ну эт хуйню я не использую

вроде, если хочешь в сф комманду запихнуть контейнер, вариант сделать тупо и быстро отнаследоваться от какой-то байды которая внутри себя это делала upd ContainerAwareCommand во

эх соскучился я по симфе

да вы вроде не заложники, отдельные сервисы ж пофиг на чем писать

вроде, если хочешь в сф комманду запихнуть контейнер, вариант сделать тупо и быстро отнаследоваться от какой-то байды которая внутри себя это делала upd ContainerAwareCommand во

да верно, но мы даже в команды инджектили всё

да вы вроде не заложники, отдельные сервисы ж пофиг на чем писать

отдельные сервисы ))))))

есть у нас 1 отдельный сервис

вопрос: нахуя setContainer нужен? что бы потом вызывать $this->container->get?

объективно говоря я слышал на протяжении двух версий что его хотят деприкейтнуть вообще

объективно говоря я слышал на протяжении двух версий что его хотят деприкейтнуть вообще

в 4 уже не всё можно так дернуть

только то, что явно в конфиге как паблик объявлено

ERROR: S client not available

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

в 4 уже не всё можно так дернуть

вроде это не только в 4

а у меня от синтаксиса приватных сервисов бомбит, могли флаг какой-то в ямле сделать

Все любители впихнуть сеттеры для зависимостей должны кодить на мадженте. Им там самое место. Особенно с аргументацией а стиле "что б в конструкторе меьше было". Мусор под коврик замели и типа порядок, бгг. Ещё бОльший треш разведя

Все любители впихнуть сеттеры для зависимостей должны кодить на мадженте. Им там самое место. Особенно с аргументацией а стиле "что б в конструкторе меьше было". Мусор под коврик замели и типа порядок, бгг. Ещё бОльший треш разведя

будто бы ты не убирался в комнате, заметая все под кровать?))

будто бы ты не убирался в комнате, заметая все под кровать?))

У меня ребенок мелкий, не прокатит)

а у меня от синтаксиса приватных сервисов бомбит, могли флаг какой-то в ямле сделать

так там же есть public: true/false

или нет?

так там же есть public: true/false

когда я последний раз смотрел было с точкой

или нет?

Так

я вообще щас на своем отдельном проекте на пхп пишу конфиги

$services->set(DemandsElasticityAbTestService::class) ->args( [ ref(MemberTestingMapper::class), ref(GroupChooser::class), ] ) ->public();

Ток за паблик тру опять таки надо по рукам бить в 99% случаев

фигня конечно, но в отстутсвии йамла сойдет

лучше чем массивы

Ток за паблик тру опять таки надо по рукам бить в 99% случаев

с учетом, что можно через локатор разрулить, то наверное в 99.9999999% случаев

так там же есть public: true/false

действительно, хм видимо комьюнити взяло свое,а я не следил так как другую версию на работе юзают :)

с учетом, что можно через локатор разрулить, то наверное в 99.9999999% случаев

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

с учетом, что можно через локатор разрулить, то наверное в 99.9999999% случаев

Я вот даже не уверен, что публичные сервисы чем-то хуже сеттеров для зависимостей, так-то

Я вот даже не уверен, что публичные сервисы чем-то хуже сеттеров для зависимостей, так-то

сам факт того, что что-то помечено паблик — нейтральный. а вот когда кто-то начинает вызывать $this->container->get, вот тогда надо убивать с особой жестокостью

ну если первые дают нам потенциальную возможность говнокодить, то вторые это уже гавнокод, котоыре дают нам потенциальную возможность написать код с ошибками

сам факт того, что что-то помечено паблик — нейтральный. а вот когда кто-то начинает вызывать $this->container->get, вот тогда надо убивать с особой жестокостью

кек, так если помечано пабилк, знначит подразумевается что кто то и будет вызывать)))

Если у тебя зависимости в конструктор не влезают, то это проблема разделения ответственности и сеттерами она не рншается. Вооьще никак

кек, так если помечано пабилк, знначит подразумевается что кто то и будет вызывать)))

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

нет нельзя)

если ты создаешь лазейку для гавнокода, то кто нибудь обязательно ее использует

это тоже самое как принести топор в компанию алкашей

вот когда кто-то вызвал, то первым делом git blame, вторым — разогревание кочерги на плите с последующем погружением в анус того, что вызвал

а потом говорить, ну сам топор то зла не сделал, я его просто принёс и никого не убивал

это тоже самое как принести топор в компанию алкашей

а может я дровосек и по пути с работы к корешам зашел

это как "оружие убивает"

ок сойдемся на том что публик сервисы пишут дровосеки)