На что обращаешь внимание при Code Review

При Code Review проверяю:

1. Читаемость - понятные названия переменных, методов, отсутствие "магических чисел"
2. Архитектуру - соблюдение SOLID, правильное разделение ответственностей
3. Обработку ошибок - корректные try-catch блоки, валидация входных данных
4. Тестируемость - отсутствие жестких зависимостей, возможность мокировать
5. Производительность - оптимальные алгоритмы, избегание N+1 запросов

Пример плохого кода:

// Плохо: магическое число, длинный метод
public double calc(int x) {
    return x * 3.14 * x / 2 + x - 5;
}

Пример улучшенного:

// Хорошо: константа, разделение логики
private static final double PI = 3.14;
private static final int DISCOUNT = 5;

public double calculateCircleArea(int radius) {
    return PI * radius * radius;
}

public double applyDiscount(double amount) {
    return amount - DISCOUNT;
}