Как проходил Code Review на последнем проекте

На последнем проекте Code Review был обязательным этапом перед мержем в основную ветку. Использовали GitHub Pull Requests с обязательным approval минимум от одного коллеги.

Основные принципы:

• Проверяем соответствие код-стайлу проекта (использовали Checkstyle)
• Обращаем внимание на читаемость кода и нейминг
• Тестируем edge cases и потенциальные NPE
• Проверяем покрытие тестами (минимум 80% для новых фич)

Пример плохого кода, который часто отклоняли:

// Плохо: магические числа, хардкод
public int calc(int x) {
    return x * 100 / 20; 
}

Лучший вариант:

private static final int DISCOUNT_RATE = 100;
private static final int TAX_RATE = 20;

public int calculatePrice(int basePrice) {
    return basePrice * DISCOUNT_RATE / TAX_RATE;
}

Использовали инструменты: SonarQube для статического анализа, GitFlow для ветвления.