NO
Сценарии усложняют код ) Всё зло, что усложняет код...
Size: a a a
2020 May 02
V
Что заполняется в форме? Это регистрация, логин, редактирование CRUD, онлайн-заявка?
Настройки устройства. При создании нужно выбрать тип. При редактировании тип изменить уже нельзя, и это поле показывать и вообще как-то учитывать не нужно
V
Я бы разбил на две формы без проблем. Но не хочется копипастить attributeLabels
E
Я бы разбил на две формы без проблем. Но не хочется копипастить attributeLabels
ну так разбейте на две формы, в родительскую абстрактную вынесете все общее
NO
Значит CRUD. Сделай раздельные формы ) DeviceCreateForm и DeviceUpdateForm
V
Значит CRUD. Сделай раздельные формы ) DeviceCreateForm и DeviceUpdateForm
Та ну. Со временем атрибутов добавиться, и если нужно будет поменять текст, лазить по всем формам и менять его там?
NO
Я бы разбил на две формы без проблем. Но не хочется копипастить attributeLabels
Если Label потребуется на одной форме сделать одно, а в другой другое? ) В случае с раздельными формами даже думать не придётся )
E
Им дали наследование и трейты а они юзают сценарии
V
ну так разбейте на две формы, в родительскую абстрактную вынесете все общее
так себе решение. Если в будущем понадобится ещё что-то, как реализовать множественное наследование. Ну разве что трейты...
NO
Ок, ну тут как со всеми прочими антипаттернами ) Делайте на сценариях, наешьтесь проблем, выпилите сценарии, забудьте о сценариях как о кошмаре )
V
Ок, ну тут как со всеми прочими антипаттернами ) Делайте на сценариях, наешьтесь проблем, выпилите сценарии, забудьте о сценариях как о кошмаре )
Уговорил) вынесу лейблы в трейт. Второй раз в жизни трейты юзаю)
NO
А вот это тем более не рекомендую... Трейт усложняет сильнее сценариев
V
Всё равно неудобно две модели. Правила валидации, название полей одни и те же. Если захочется поменять название поля или rules, придётся делать это в двух моделях
V
полей со временем будет больше, валидация усложниться...
NO
Делайте как кажется вам правильным сейчас. Чужой опыт вас ничему не научит, только свой.
V
S)
кто-то юзал state machine? есть норм extension?
P
так себе решение. Если в будущем понадобится ещё что-то, как реализовать множественное наследование. Ну разве что трейты...
Сделай хиден поле с типом и все
P
При создании поле есть, при редактировании значение известно, чтоб не городить ещё одну модель просто скрой поле
ДК
полей со временем будет больше, валидация усложниться...
А со сценариями то как весело вам будет в этом случае ууух