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

Python Middle Без компании
На что обращаешь внимание в первую очередь при проведении code review
Ответы
При code review в первую очередь смотрю на: - **Читаемость кода** - понятные названия переменных/функций, соблюдение PEP8, отсутствие "магических чисел" - **Архитектуру** - разделение ответственностей, отсутствие дублирования, правильное использование паттернов - **Безопасность** - санитизация ввода, защита от инъекций, корректная обработка ошибок - **Производительность** - оптимальные алгоритмы, отсутствие лишних операций в циклах - **Тестируемость** - модульность кода, отсутствие жестких зависимостей Пример плохого кода: ``` def p(d): # непонятное название for i in d: # магическое число 5 if i > 5: print(i) ``` Пример улучшенного: ``` def print_numbers_greater_than_threshold(numbers, threshold=5): """Выводит числа превышающие пороговое значение""" for number in numbers: if number > threshold: print(number) ``` Также проверяю соответствие задаче и наличие документации для сложных участков.