На что обращаешь внимание при code review чужого кода?

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

• Читаемость и понятность кода. Код должен быть простым для понимания, без излишней сложности.
• Соответствие стилю и стандартам проекта. Важно, чтобы код был консистентен с остальной базой.
• Логика и корректность реализации. Проверяю, что код делает то, что заявлено, и нет логических ошибок.
• Обработка ошибок и исключений. Убедиться, что ошибки корректно обрабатываются и не приводят к падениям.
• Тесты. Наличие и качество юнит- и интеграционных тестов.
• Производительность и ресурсоёмкость. Оцениваю, нет ли узких мест или неоптимальных решений.
• Безопасность. Проверяю, нет ли уязвимостей, например, SQL-инъекций или утечек данных.

Такой подход помогает поддерживать качество и надёжность кода в проекте.