Size: a a a

2020 September 23

Р

Руслан in symfony
Павел Г.
вот есть сущность на 10 свойств.
Прилетел первый кейс: обновление первого и третьего свойства - вы пишите свой update.
Прилетел второй кейс: обновление второго и третьего - вы обновляете свой апдейт, вместо добавленния нового функционала
И т.д. т.е. каждый раз меняете его.
Плюс странная задача сущности  "Обновить". Один метод собирает просто VOшки и сетит. Те же сетеры только под капотом одного.
Кто мешает прописать проверку 10 свойств и обновления того, что прилетело?
источник

ПГ

Павел Г. in symfony
Руслан
Именно так. А так можно пилить систему с предположением, что прилетит миллиард данных. Вот только оверинжиниринг и преждевременная оптимизация зло похуже любого говнокода
Дело не в предположении, а в том, что:
1) Совершенно не понятно что делает update и в каком кейсе
2) Он сеттер как ни крути
3) Его нормально отестировать не получится, так как почти все действия опциональны.
источник

ПГ

Павел Г. in symfony
Руслан
Кто мешает прописать проверку 10 свойств и обновления того, что прилетело?
Когда этого не нужно делать изначально?
источник

ПГ

Павел Г. in symfony
Такого даже кейса нет
источник

Р

Руслан in symfony
Павел Г.
Дело не в предположении, а в том, что:
1) Совершенно не понятно что делает update и в каком кейсе
2) Он сеттер как ни крути
3) Его нормально отестировать не получится, так как почти все действия опциональны.
2) глобальный сеттер для заполнения профиля, да
источник

Р

Руслан in symfony
Павел Г.
Такого даже кейса нет
Регистрация через 1с, например
источник

Р

Руслан in symfony
3) что там тестировать то?
источник

Р

Руслан in symfony
Функциональных тестов выше крыши - вызвал эндпоинт пут на регистрацию, прочитал пользователя, все ли заполнилось. Как работает внутри - как-то пофиг
источник

Р

Руслан in symfony
Павел Г.
вы грубо говоря запихиваете массив из 1000 данных и опционально пытаетесь из него создать VOшки
Расскажите ваш вариант добавления 1000 данных, в json пихать?
источник

ПГ

Павел Г. in symfony
Руслан
Расскажите ваш вариант добавления 1000 данных, в json пихать?
имеется ввиду когда 1000 данных и на них 100 юзкейсов по 10 данных.
источник

ПГ

Павел Г. in symfony
ваш вариант огромный метод опциональный и 1 юзкейс, мой 100 хэндлеров/юзкейсрв
источник

Р

Руслан in symfony
Павел Г.
имеется ввиду когда 1000 данных и на них 100 юзкейсов по 10 данных.
ну на 100 юзкейсов естественно разделять надо, не в один же пихать
источник

ОС

Олег Соболев... in symfony
Здравствуйте подскажите с чем связано 403 Referrer Policy: strict-origin-when-cross-origin
источник

Р

Руслан in symfony
Павел Г.
ваш вариант огромный метод опциональный и 1 юзкейс, мой 100 хэндлеров/юзкейсрв
С чего вы это взяли то? Юзкейс смена пароля - отдельный)
источник

V

Vlad in symfony
не помогло, либо я не так что то делаю..
источник

Р

Руслан in symfony
В случае данных из 1с создавать 2 команды и 2 обработчика и вызывать их последовательно никто не мешает
источник

Р

Руслан in symfony
Но мне нравится мой вариант)
источник

ПГ

Павел Г. in symfony
Руслан
Но мне нравится мой вариант)
Он неплохой, меня просто опциональность смущает, но это имхо.
источник

Р

Руслан in symfony
Павел Г.
Он неплохой, меня просто опциональность смущает, но это имхо.
Перепилить на 2 юзкейса никто не мешает, я сравнивал свой вариант с вариантом в контроллере
источник

ПГ

Павел Г. in symfony
просто чем жестче, тем проще тестировать и меньше непонятных моментов
источник