Следовал ли каким-то принципам при проведении Code Review

При проведении Code Review я ориентируюсь на несколько ключевых принципов:

1. Читаемость кода – код должен быть понятен другим разработчикам. Используются ли осмысленные имена переменных/методов? Нет ли избыточной сложности?

2. Соблюдение SOLID – проверяю, соблюдаются ли принципы единой ответственности, открытости/закрытости и другие.

3. Тестируемость – код должен быть покрыт тестами (юнит, интеграционными). Проверяю, не усложняет ли архитектура написание тестов.

4. Производительность – нет ли явных узких мест (например, N+1 запросов в цикле).

5. Безопасность – проверяю на уязвимости (SQL-инъекции, XSS и т.д.).

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

// Плохо: метод делает слишком много
public void processOrder(Order order) {
    validate(order);
    calculateTotal(order);
    applyDiscount(order);
    saveToDB(order);
    sendEmail(order);
}

Лучше разбить на отдельные методы с единой ответственностью.