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

При code review я фокусируюсь на нескольких ключевых аспектах:

1. Качество кода - соблюдение code style (Google Java Style Guide), читаемость, отсутствие дублирования (DRY)
2. Архитектура - правильное разделение ответственностей, SOLID принципы
3. Тестируемость - наличие unit-тестов, mock-объекты где нужно
4. Безопасность - проверка на уязвимости (SQL-инъекции и т.д.)
5. Производительность - оптимальные алгоритмы, избегание N+1 проблем

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

// Плохо: нарушает SRP, нет обработки ошибок
public void processUserData(String json) {
    User user = new Gson().fromJson(json, User.class);
    userRepository.save(user);
    emailService.sendWelcomeEmail(user.getEmail());
}

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

// Хорошо: разделение ответственностей, обработка ошибок
public void processUserData(String json) throws InvalidUserDataException {
    User user = parseUser(json);
    saveUser(user);
    sendWelcomeEmail(user);
}

Использую конструктивный подход - отмечаю не только проблемы, но и предлагаю решения. Важно сохранять баланс между строгостью и поддержкой коллег.