Д
ну, с пьяного взгляда, кмк, нужно, когда ты конструируешь statement, нужно проверки в конструкторе инкапсулировать
Size: a a a
2021 April 18
Д
и не писать их в экшене
Д
экшн вообще не должен отвечать за логику.
Д
сделай билдер какой..
IS
нее.
Statement - это форма, фактически. В момент конструирования она вообще ещё ничего ни о ком не знает.
Заполняется она в момент сабмита и, соответственно $statement->load()
Statement - это форма, фактически. В момент конструирования она вообще ещё ничего ни о ком не знает.
Заполняется она в момент сабмита и, соответственно $statement->load()
IS
ну теоретически можно в неё вкрутить метод, где бы она проверяла наличие переданного ей...
Д
а это??
IS
$manager - это как раз сервис, класс StatementManager, который инжектится в экшн.
IS
в принципе, можно и ему скармливать id-шники, да. А он пускай проверяет и в приватные поля себе складывает...
IS
интересная мысль.
Д
я мысль не уловил, но, видимо, да who knows
NO
Ну просто прокидывай в сервис готовые объекты Source, Account, File, готовь их в контроллере.
IS
вот я так и делал, но Дмитрий меня раскритиковал )
NO
StatementManager ужасное название безотносительно кода
Д
а я чо, я ничо. мне ненравится логика в контроллерах.
NO
Название должно давать информацию о том что класс делает, StatementManager даëт ноль информации
NO
Взять по айдишнику объект и передать в следующий слой это не логика.
IS
Понял. Подумаю.
Д
и тем не менее со мной пить никто не хочет
IS
Я квас пью. Пиво вчера пил, позавчера пил, всю неделю пил. Надо сделать перерыв)