DM
def change_resposebility_dude
old manager = event.old_manager
new_manage = user
change_and_notify(event, new_manager, old_manager)
end
old manager = event.old_manager
new_manage = user
change_and_notify(event, new_manager, old_manager)
end
Size: a a a
2020 March 18
DM
с колбеками 100% не стоит это делать, выглядит плохо, потом будет еще хуже
КК
может стоит написать сервис, и там указать всю эту логику?)
Может, но я пока барахтаюсь как умею, рельсы то дней 40 назад увидел %) До сервисов пока не дополз)
ТК
Кракозябр Кракозябрович
Может, но я пока барахтаюсь как умею, рельсы то дней 40 назад увидел %) До сервисов пока не дополз)
Да сервис в принципе то и не сложно написать, можно кстате внутри модели колбэком дёргать этот сервис, ну или из контроллера.
DM
Кракозябр Кракозябрович
Может, но я пока барахтаюсь как умею, рельсы то дней 40 назад увидел %) До сервисов пока не дополз)
можно сразу отходить от колбеков и логики в моделях, просто пиши отдельный класс который будет всю логику делать и возвращать результ
DM
типа EventCreator.new(все_параметры).execute
DM
почитай про ServiceObject's, тот же active_interaction гем очень приятный
КК
Да сервис в принципе то и не сложно написать, можно кстате внутри модели колбэком дёргать этот сервис, ну или из контроллера.
печалька та в том, что пока не известен полный перечень уведомлений и соответственно все случаи создания оных
КК
Сервисобжект значится
DM
меня на первой работе тимлид лапатой бы забил за колбеки)
ТК
Кракозябр Кракозябрович
печалька та в том, что пока не известен полный перечень уведомлений и соответственно все случаи создания оных
Начинай писать сервис, а в методах сервиса можно написать закоментированные пустышки
например: # здесь будет notify
и получится набросок который ты потом дополнишь
например: # здесь будет notify
и получится набросок который ты потом дополнишь
КК
меня на первой работе тимлид лапатой бы забил за колбеки)
Хорошо когда тимлид, он же сеньёр, он же всё остальное не один и не завален прочим геммороем)
DM
ну меня на код ревью били за каждую хрень) даже за отступы
КК
который случился в аккурат через неделю как приходишь студентом на галеру
КК
ну меня на код ревью били за каждую хрень) даже за отступы
Сурово. Я пока успел только свою гит ветку закосячить)
ТК
ну меня на код ревью били за каждую хрень) даже за отступы
Хорошо что он вообще есть этот код ревью, ты хотябы знаешь в правильном направлении ты двигаешься или нет)
DM
Хорошо что он вообще есть этот код ревью, ты хотябы знаешь в правильном направлении ты двигаешься или нет)
ну да повезло, шас на работе ваще лафа какая то, лишь бы косяков в логике и работе небыло кода
DM
а все остальное вообще пофиг
DM
тимлид сам пишет как хочет) я за ним потом сам рубопоком кодстайл правлю
ТК
уф не люблю рубокоп)