Ответ
При код-ревью фокусируюсь на нескольких ключевых аспектах:
- Архитектура - соответствие SOLID, MVVM/MVI, отсутствие god-object'ов
- Производительность - утечки памяти, главный поток, оптимизация списков
- Безопасность - хранение данных, HTTPS, ProGuard правила
- Тестируемость - мокируемость зависимостей, чистые функции
- Читаемость - именование, KDoc, сложность методов
Пример плохого кода:
fun process(data: String) { // 100 строк логики + сетевые запросы + БД }Лучше:
fun parseInput(data: String): Model = ...
suspend fun fetchRemote(model: Model): Result = ...
fun saveToDb(result: Result): Unit = ...Использую GitHub/GitLab ревью-инструменты, оставляю конкретные комментарии с предложениями по улучшению. Важно сохранять баланс между строгостью и конструктивностью.