Quais são os pontos-chave que verificas primeiro ao fazer uma revisão de código?

При рецензировании кода в первую очередь обращаю внимание на:

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

Пример простого комментария в рецензии:

# Хорошо, что здесь используется try-except, но стоит уточнить, какие именно исключения мы ожидаем.