Size: a a a

2021 April 18

IS

I Scarab in Yii Framework 2
там до датапровайдера ещё как до луны, там идёт парсинг банковской выписки.
Мне идеологически больше интересно - делать ли контроллер совсем тупым и пусть поиском занимается сервисный слой, или всё же контроллер должен сделать какие-то предварительные проверки сам.
Ну и красивее, когда сервисный метод получает объекты заданных типов, а не три инта.
источник

Д

Дмитрий in Yii Framework 2
means your controller not 'very' simple, but extends some logic, you dont know.
источник

YO

Yasiel Oria Gómez in Yii Framework 2
that must be the problem
источник

Д

Дмитрий in Yii Framework 2
learn what dataProvider is.
источник

Д

Дмитрий in Yii Framework 2
это ты про что сейчас? ни обьявление метода не запостил, нихрена..
источник

YO

Yasiel Oria Gómez in Yii Framework 2
Yes
источник

YO

Yasiel Oria Gómez in Yii Framework 2
Thank
источник

Д

Дмитрий in Yii Framework 2
естественно в  контроллер не надо логику пихать
источник

IS

I Scarab in Yii Framework 2
а проверка существования нужных экземпляров тоже считается логикой?
источник

Д

Дмитрий in Yii Framework 2
в контроллере?
источник

IS

I Scarab in Yii Framework 2
ага.
источник

IS

I Scarab in Yii Framework 2
Вообще там так получается:
источник

IS

I Scarab in Yii Framework 2
а в сервисе уже
источник

IS

I Scarab in Yii Framework 2
соответственно при тестировании сервиса я могу ему скормить тестовый Source, тестовый Account и так далее.
источник

IS

I Scarab in Yii Framework 2
Альтернатива - чтобы контроллер получил $sourceId и отдал его сервису, а тот пускай ебётся. Но тогда непонятно, как такое тестировать, да и принцип single responsibility как-то совсем нарушается.
источник

IS

I Scarab in Yii Framework 2
(это я тут себе pet-project пилю для загрузки банковских выписок в софтину для ведения домашнего бюджета :) )
источник

Д

Дмитрий in Yii Framework 2
так у тебя и в контроллере srp нарушается от слова совсем.
источник

IS

I Scarab in Yii Framework 2
а как лучше?
источник

Д

Дмитрий in Yii Framework 2
дай подумать.
источник

IS

I Scarab in Yii Framework 2
один хрен где-то же эти проверки надо делать.
источник