На что обращаешь внимание при код-ревью?

На что обращаю внимание при код-ревью

Код-ревью — это критический процесс обеспечения качества. Я проверяю несколько слоёв: от архитектуры до мелких деталей.

1. Архитектура и дизайн

Первым делом проверяю соблюдение SOLID принципов и разделение ответственности.

2. Обработка ошибок

Ошибки должны обрабатываться правильно, не скрываться. Проверяю:

• Нет пустых catch блоков
• Нет e.printStackTrace()
• Используется структурированное логирование
• Пробрасываются осмысленные исключения

3. Null Safety

Проверяю защиту от NullPointerException:

• Защита на входе методов
• Использование Optional вместо nullable
• Null checks перед вызовом методов

4. Производительность

Проверяю сложность алгоритмов:

• Нет O(n²) где можно O(n)
• Нет N+1 query проблем
• Нет утечек памяти
• Кеширование где нужно

5. Безопасность

• Нет SQL Injection уязвимостей
• Prepared statements для БД запросов
• Нет хардкодированных секретов
• Валидация пользовательского ввода

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

Проверяю:

• Инъекция зависимостей
• Возможность мокирования
• Unit тесты на критичный код
• Test coverage >= 80%

7. Читаемость кода

• Методы короче 20 строк
• Ясные имена переменных и методов
• Нет магических чисел (константы)
• Логическое разбиение на функции

8. Логирование

• Нет System.out.println в продакшене
• Правильные уровни логирования (debug, info, warn, error)
• Структурированное логирование (параметры, не конкатенация)

9. Версионирование зависимостей

• Актуальные версии библиотек
• Нет SNAPSHOT версий без необходимости
• Совместимость версий

10. Тесты

• Тесты читаемые (Arrange-Act-Assert)
• Осмысленные assertions
• Не только happy path
• Граничные случаи покрыты

11. DRY и KISS

• Нет дублирования кода
• Переиспользование компонентов
• Простые решения вместо сложных

12. Документация

• JavaDoc для публичного API
• Комментарии для нетривиальной логики
• Нет очевидных комментариев

Чек-лист при код-ревью

• Архитектура и SOLID принципы
• Безопасность (SQL injection, XSS, secrets)
• Производительность (N+1, O(n²), утечки памяти)
• Тестируемость (DI, unit tests, coverage)
• Читаемость (короткие методы, ясные имена)
• Обработка ошибок (нет silent fails)
• Null safety (Optional, null checks)
• DRY (нет дублирования)
• JavaDoc для публичного API
• Логирование структурировано

Вывод

При код-ревью фокусируюсь на трёх уровнях приоритета:

1. Архитектура — фундамент
2. Корректность — логика и безопасность
3. Качество — стиль и производительность

Цель не критиковать, а улучшить качество и передать знания.