Как анализировать и улучшать чужой код во view controller?

Работа с чужим кодом — типичная задача сеньор-разработчика, и здесь важно не ломать систему резкими изменениями.

С чего начинать анализ

Перед изменениями нужно понять текущее состояние кода.

1. Проверка ответственности

Контроллер не должен:

1. выполнять сетевые запросы;

2. хранить бизнес-логику;

3. управлять навигацией напрямую;

4. форматировать данные.

Если он это делает — есть точки для рефакторинга.

2. Анализ зависимостей

Стоит обратить внимание:

• на прямые инициализации сервисов;

• на отсутствие протоколов;

• на сложные графы владения.

Это часто источник проблем с тестированием и памятью.

Поиск технического долга

Типичные сигналы:

1. Большие методы.

2. Дублирующаяся логика.

3. Сильная связанность с UIKit.

4. Сложные if и switch.

Как улучшать код

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

1. Вынос логики

• бизнес-логика → ViewModel / Presenter;

• форматирование → отдельные объекты;

• навигация → координаторы.

2. Упрощение асинхронности

• убрать вложенные completion;

• централизовать обработку ошибок;

• явно управлять состояниями.

3. Улучшение читаемости

• говорящие имена;

• уменьшение методов;

• логическое разделение кода.

Практический вывод

Анализ чужого view controller — это поиск лишней ответственности. Чем меньше контроллер знает и делает, тем проще его поддерживать и развивать.