На что обращаешь внимание в первую очередь при проведении code review
Python
Senior
Без компании
На что обращаешь внимание в первую очередь при проведении 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)
```
Также проверяю соответствие задаче и наличие документации для сложных участков.