Size: a a a

2021 September 16

M2

Mrx 26 in Laravel Pro
Спасибо
источник

T

ThisCrazy in Laravel Pro
Переслано от ThisCrazy
Ребят, всем привет!
На сколько плохо, что у меня внутри сервиса (контроллер) будет происходить валидация?
Проблема в том, что названия атрибутов у меня динамические, и в форм-реквесте не могу валидацию сделать

Это плохая практика вообще, или такое встречается?
источник

T

ThisCrazy in Laravel Pro
Переслано от ThisCrazy
Если кто не понял:

1-й скрин - сама форма с динамическими изменениями аттрибутов

2-й скрин - сама валидация этих полей
источник

T

ThisCrazy in Laravel Pro
Переслано от ThisCrazy
источник

А

Антон in Laravel Pro
Норм
источник

T

ThisCrazy in Laravel Pro
Просто когда-то где-то читал, что делать валидацию внутри самого контроллера - бед практис
источник

AK

Alex Kovalchuk in Laravel Pro
а что мешает аналогично сформировать массив в форм-реквесте?
источник

А

Антон in Laravel Pro
А чо тогда контроллер делать будет?
источник

AK

Alex Kovalchuk in Laravel Pro
все что делал раньше но без валидации
источник

А

Антон in Laravel Pro
Что он делал раньше?
источник

T

ThisCrazy in Laravel Pro
Я не показываю весь код, это лишь та часть, которую ещё можно запихнуть в форм реквест, но вот при добавлении новых значений атрибута в уже существующем атрибуте, там ещё каша больше становиться
Названия атрибутов становиться какими-то таким: values.new.1.2.title
источник

AK

Alex Kovalchuk in Laravel Pro
на скриншоте не видно все тело контрлера
думаю обновлял значения
источник

T

ThisCrazy in Laravel Pro
Это сервис, сам контроллер у меня максимально худой
источник

А

Антон in Laravel Pro
Вот это как раз контроллер делать и не должен.
источник

T

ThisCrazy in Laravel Pro
Сам контроллер
источник

А

Антон in Laravel Pro
Оч плохой контроллер.
источник

T

ThisCrazy in Laravel Pro
Почему?
источник

А

Антон in Laravel Pro
Интересно, что у тебя делает attributeService. Что он там обновляет?
источник

T

ThisCrazy in Laravel Pro
Обновляет все эти поля
источник

T

ThisCrazy in Laravel Pro
Включая другие вкладки
источник