Все верно, грубо говоря мы всегда отодвигаем код который мы не тестируем за следующую абстракцию. Тестировать или не тестировать фабрики это личный выбор ;)
Если для вас все логично и удобно то наверно вы не пытались покрывать код тестами. Иначе вы зразу бы столкнулись с тем что непонятно как организовывать передачу даних между экранами, кто на кого должен держать сильные ссылки, как обеспечить изолированость слоев (привет clean architecture) и собсвенно независимость экранов друг от дружки.
В MVC в Модель засуньте как можно больше бизнес логики. Тоесть несомнено первый вариант лучше. Возможно вам также поможет вот эта статья: Do MVC like it’s 1979
Хотя вот один «show stopper»:
Колега, Антон Русаков в этой статьи расказывает весь процес переезда в Лондон, из того что может остановить его — это тест по английскому. Тест нужно сдать хотя бы на 4 бала (базовый уровень).
Если говорить о не профессиональных критериях то все просто, орагничений никаких нет, главное чтобы хотелось работать с человеком. Для меня это значит что кандидат не должен быть «ниньзя-суперстар» а командным игроком способным работать в «плоской» иерархии 15 человек.
Примеры, на самом деле, очень маленькие, и проблему в них никак не увидеть. А статья, все-таки, обзорная, и полезная, в первую очередь, тем, кто сам заметит проблемы с Епловским MVC и захочет попробовать использовать что-то другое.
Колега, Антон Русаков в этой статьи расказывает весь процес переезда в Лондон, из того что может остановить его — это тест по английскому. Тест нужно сдать хотя бы на 4 бала (базовый уровень).