На что обращаете внимание при проведении код-ревью?

При код-ревью обращаю внимание на несколько ключевых аспектов:

• Читаемость и поддерживаемость кода: чтобы код был понятен другим разработчикам, использовались понятные имена переменных и функций, отсутствовали излишние сложности.
• Соответствие архитектурным и стилевым стандартам: проверяю, что код соответствует принятым в проекте паттернам и гайдлайнам.
• Логика и корректность реализации: убеждаюсь, что алгоритмы работают правильно и покрывают все необходимые кейсы.
• Обработка ошибок и исключений: проверяю, что ошибки корректно обрабатываются и не приводят к падениям.
• Тесты: наличие и качество юнит- и интеграционных тестов, покрытие важных сценариев.
• Производительность и оптимизация: оцениваю, нет ли узких мест или избыточных операций.

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