chesnikovofficial Oct 17 2021 at 22:03

Опыт внедрения многомодульности в (еще) не разросшееся приложение (Часть 1)

5 min

2.2K

Development for iOS*Development of mobile applications*Swift*

Comments 4

0xFEE1DE4D Oct 31 2021 at 16:48

modelsNetworkService.loadModels(page: 1)тесты которые лезут по настоящему в сеть не очень надежны, и не особо приветствуется. лучше замокать networkService, и проверить что при замоканном failure происходит вызов кложуры с верной ошибкой, а при замоканом успехе response не пустой, и в нем есть данные

и кажется urlRequest в ModelsAPI немного громоздкий получается, возможно стоит абстрагироваться до method и parameters, и уже в абстракции делать URLRequest

chesnikovofficial Nov 18 2021 at 12:46

Спасибо за критику, действительно стоит абстрагироваться. Тоже думаю уже над этим.

По поводу тестов тоже понял, не совсем корректное место для проверки работы сервисов продукта выбрал.

Prostor9 Nov 18 2021 at 12:48

А почему Вы в loadModels в completion-блоке не используете Result?
И Вы используете Combine? Не слежу просто за реактивными фреймворками.

Еще guard let strongSelf = self else { return } можно много где повыкидывать, т.к. все равно с nil работаете и выдаете

chesnikovofficial Nov 18 2021 at 12:50

С result начал знакомится, благодаря вашему комментарию, спасибо, удобная вещь. Обязательно нужно будет завезти.

Guard наверное остаточное у меня от прошлых мест работы где говорили его пихать везде где только можно. Надо будет при рефакторинге критично посмотреть на свой код.

Show the best of all time