Как будешь проводить код ревью

Мой подход к код-ревью как Senior Android Developer

Основная цель код-ревью в моей практике — совместное улучшение качества кода, а не поиск ошибок или критика. Я рассматриваю ревью как образовательный процесс и возможность делиться знаниями внутри команды.

Подготовка к ревью

Перед началом ревью я всегда:

• Изучаю связанный тикет/задачу в трекере (Jira, GitHub Issues)
• Понимаю бизнес-контекст и требования
• Проверяю, есть ли описание изменений в PR/MR
• Убеждаюсь, что CI/CD pipeline успешно выполнился

Ключевые аспекты, которые я проверяю

1. Архитектурная корректность

// ПЛОХО: Нарушение принципа единственной ответственности
class UserManager {
    fun saveUser(user: User) { /* сохранение в БД */ }
    fun fetchUsers(): List<User> { /* сетевой запрос */ }
    fun validateEmail(email: String): Boolean { /* валидация */ }
}

// ХОРОШО: Разделение ответственности
class UserRepository { /* работа с данными */ }
class UserValidator { /* валидация */ }
class UserViewModel { /* логика представления */ }

2. Соответствие принципам SOLID и Clean Architecture

• Проверяю соблюдение инверсии зависимостей
• Контролирую размер классов и методов
• Слежу за правильным использованием слоев (domain, data, presentation)

3. Качество кода и читаемость

• Именование переменных, методов, классов (осмысленное, на английском)
• Размер методов (обычно не более 20-30 строк)
• Отсутствие магических чисел и строк
• Правильное использование Kotlin features (extension functions, sealed classes, coroutines)

4. Работа с памятью и производительность

// ПЛОХО: Утечка контекста
class MyAdapter(private val context: Context) {
    // Context может привести к утечке памяти
}

// ХОРОШО: Использование Application Context или ViewHolder pattern
class MyAdapter(private val context: Context) {
    init {
        this.context = context.applicationContext
    }
}

5. Безопасность и обработка ошибок

• Проверка null safety
• Обработка исключительных ситуаций
• Безопасное хранение чувствительных данных
• Валидация пользовательского ввода

6. Тестируемость

• Наличие unit тестов для критической логики
• Использование dependency injection для мокинга
• Отсутствие статических зависимостей

7. UI/UX аспекты для Android

• Корректная работа с жизненным циклом компонентов
• Правильное использование ViewModel и LiveData/Flow
• Оптимизация работы с RecyclerView
• Поддержка разных размеров экранов и ориентаций

Процесс проведения ревью

1. Первичный обзор (5-10 минут):

   • Беглый просмотр всех изменений
   • Понимание общего масштаба изменений

2. Детальная проверка:

   • Построчный анализ изменений
   • Проверка каждого аспекта из списка выше
   • Запись комментариев с пояснениями

3. Тестирование (при необходимости):

   • Запуск приложения на эмуляторе/устройстве
   • Проверка edge cases
   • Валидация бизнес-логики

Формат комментариев

Я всегда придерживаюсь конструктивного подхода:

• ❌ "Здесь может быть проблема с..."
• ✅ "Предлагаю рассмотреть альтернативу..."
• ❓ "Можешь пояснить, почему выбран этот подход?"
• 📚 "По этой теме есть полезная статья/документация..."

Обязательно указываю приоритет:

• BLOCKER — необходимо исправить перед мержем
• MAJOR — важное улучшение, но не блокирующее
• MINOR — рекомендация на будущее

Послемержевые действия

После принятия PR:

1. Проверяю, что все комментарии обработаны
2. Убеждаюсь, что ветка корректно влита
3. При необходимости провожу ретроспективу по сложным кейсам
4. Делаю заметки для общих знаний команды

Инструменты и автоматизация

Использую:

• GitHub/GitLab/Bitbucket для основного ревью
• SonarQube/Detekt для статического анализа
• Custom lint rules для проверки проектных стандартов
• CI/CD pipeline с автоматическими проверками

Важные принципы

1. Уважение к автору — код-ревью это диалог, а не монолог
2. Своевременность — стараюсь провести ревью в течение 4-8 часов
3. Баланс — не требую идеального кода, но настаиваю на критически важных улучшениях
4. Обучение — делюсь знаниями и принимаю feedback о своем стиле ревью

Такой подход позволяет не только поддерживать высокое качество кода, но и способствует профессиональному росту всех членов команды. Код-ревью становится не рутиной, а ценным процессом обмена опытом.