Какой у вас опыт проведения кросс-ревью кода (code review)?

Опыт включает регулярное участие в процессе code review как в роли ревьювера, так и автора. Основная цель — улучшение качества кода, обмен знаниями и предотвращение дефектов.

Фокус при ревью:

1. Корректность и надежность:

   • Логические ошибки и обработка edge-кейсов (нулевые значения, пустые коллекции, граничные условия).
   • Правильность алгоритмов.
   • Обработка исключений и ошибок.

2. Производительность и безопасность:

   • Неоптимальные запросы к БД (N+1 проблема).
   • Утечки ресурсов (незакрытые стримы, соединения).
   • Возможные уязвимости (инъекции, неправильная валидация).

3. Читаемость и поддерживаемость:

   • Соблюдение соглашений по стилю кода (code style) проекта.
   • Понятные имена переменных, методов, классов.
   • Адекватная сложность методов (цикломатическая сложность).
   • Наличие понятных комментариев для нетривиальной логики.

4. Архитектурная согласованность:

   • Следование принятым в проекте паттернам и принципам (SOLID, DRY).
   • Избегание излишнего связывания (coupling).

Пример конструктивного комментария:

// Было: Использование индекса в цикле может быть менее читаемым и подвержено ошибкам.
for (int i = 0; i < items.size(); i++) {
    process(items.get(i));
}

// Предложение: Использовать for-each для улучшения читаемости.
for (Item item : items) {
    process(item);
}
// Это также исключает потенциальную ошибку IndexOutOfBoundsException.

Подход: Комментарии формулируются в конструктивном ключе, с объяснением "почему" и, где возможно, с предложением альтернативного решения. Важно уважать время автора и концентрироваться на значимых улучшениях, а не на субъективных предпочтениях.