@symfony_ru - страница 812 - Telegram web archive

Kirill

- Избавиться от сеттеров и заменить их доменным методом у сущности это что значит это же основа доктрины или дата маперов

спорно

Kirill

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

Kirill

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

Роман

о чем ты

Kirill

о чем ты

А, это ты криво процитировал, поэтому промахнулся ответом

Kirill

Это я @bigdickisbackint0wn писал

Михаил

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

согласен, но в случае с букингом я бы заменил

Роман

приведите примеры, что это такое доменный объект

Kirill

В случае с букингом там вообще некорректная логика)

Kirill

findOrNull -> new - это ошибка гонки состояний жеж

Kirill

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

Михаил

приведите примеры, что это такое доменный объект

объект предметной области. Сделка, Продукт, Покупатель итд

Kirill

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

Михаил

findOrNull -> new - это ошибка гонки состояний жеж

да, поэтому я и говорю в 3 пункте пересмотреть флоу

Роман

модель что ли или в случее симы entity

Михаил

модель на подиуме.

Kirill

не, ну если в качестве энтитей там анемичные шняги, то почему нет?

Михаил

модель это вцелом про весь бизнес код. Ты моделируешь реальные вещи. Стараешься приблизиться к реальности на сколько это целесообразно

Kirill

я бы скорее от get/set нафиг избавился и заменил на свойства или поля хотя бы

Михаил

не, ну если в качестве энтитей там анемичные шняги, то почему нет?

в случае анемичной модели, ок

Kirill

в случае анемичной модели, ок

ну а почему нет, тупо, примитивно и просто

Роман

так они есть, но принятно писать используя get/set

Dmitriy

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

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

Роман

это же база

Kirill

так они есть, но принятно писать используя get/set

кем принято?)

Роман

всеми

Kirill

а, ну ок

Роман

используя доктрину

Kirill

чтобы что?

Михаил

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

Роман

разве вообще что-то будет работать без get/set даже dql не заработает помоему

Dmitriy

заработает )

Kirill

разве вообще что-то будет работать без get/set даже dql не заработает помоему

покажи мне тут хоть один get/set))) https://github.com/SerafimArts/packagist.app/blob/master/app/Package/Domain/Package.php

Михаил

разве вообще что-то будет работать без get/set даже dql не заработает помоему

посмотри реализацио PropertyAccessor

Роман

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

Dmitriy

изучай доку, проводи эксперименты

Михаил

покажи мне тут хоть один get/set))) https://github.com/SerafimArts/packagist.app/blob/master/app/Package/Domain/Package.php

опа, это мы копируем)

Dmitriy

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

Роман

и как в такую модель записывать данные?

Dmitriy

покажи мне тут хоть один get/set))) https://github.com/SerafimArts/packagist.app/blob/master/app/Package/Domain/Package.php

фу трейты )

Kirill

для установки значений принято писать оператор "=" вообще

Kirill

фу трейты )

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

Михаил

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

КОДОГЕНЕРАЦИЯ )))))))))

Роман

а понятно через конструктор

Dmitriy

покажи мне тут хоть один get/set))) https://github.com/SerafimArts/packagist.app/blob/master/app/Package/Domain/Package.php

уже под 8.3? )

Kirill

уже под 8.3? )

это 8.4

Dmitriy

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

твое же не мое )

Роман

эмуляция элокента)

Dmitriy

это 8.4

ой да 8.4 чой то это я

Kirill

с 8.3 придётся без свойств просто

Dmitriy

ну прикольно с get

Vlad

покажи мне тут хоть один get/set))) https://github.com/SerafimArts/packagist.app/blob/master/app/Package/Domain/Package.php

А зачем свой Packagist делаешь? Чем тебя nexus не устроил?

Роман

хорошо бы добавили авто свойства как в c#

Kirill

А зачем свой Packagist делаешь? Чем тебя nexus не устроил?

Kirill

вот этим)

Vlad

Ммм та free версия есть

Роман

ну реально это какая эмуляция актив рекорд

Vlad

И мы ее используем

Роман

лавел способ

Роман

в симфе так не принято

Kirill

а ещё я хочу плагины всякие писать

Kirill

и просто с 8.4 поиграться

Роман

а обновлять как какую то конкретную запись и конкретное поле?

Vlad

Ммм та free версия есть

Это походу облако ты смотришь

Kirill

ну возможно

Kirill

на работе понятно что артифактори

Kirill

или джейфрог

Kirill

но пофигу, тут интереснее вопрос про "не принято"

Роман

я ещё пользуюсь 8.2

Kirill

я так и не понял где там и что не принято

Kirill

и почему нельзя так писать

Роман

а обновлять как какую то конкретную запись и конкретное поле?

.

Kirill

.

какой оператор в пыхе служит за установку данных?

Роман

не видел чтоб в симфони так писали

Роман

такого нет думаю в доках даже

Kirill

ну то что ты сидишь в проекте на пхп 5 - не означает, что не вышли новые версии пыха))))

Роман

да я symfony 7 php 8.2 пишу с нуля

Kirill

гет/сет для эмуляции свойств - разумное и более чем аргументированное требование для пхп 5

Роман

это требования инкапсуляции данных

Kirill

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