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

При код-ревью обращаю внимание на:

1. Читаемость кода

   • Имена переменных/функций должны быть осмысленными (calculate_total вместо calc).
   • Отступы, PEP 8 (например, snake_case для функций).

2. Эффективность

   • Нет ли избыточных циклов или сложности O(n²) там, где можно O(n).
   • Использование встроенных функций (sum(), map()).

3. Обработка ошибок

   • Проверка крайних случаев (пустые списки, None).
   • Использование try-except для ожидаемых исключений.

4. Тестируемость

   • Функции делают одну вещь и не зависят от глобальных переменных.

5. Безопасность

   • Нет SQL-инъекций, санитизация вводов.

Пример плохого кода:

def calc(a, b):
    return a + b  # Неочевидное имя, нет проверки типов

Улучшенная версия:

def add_numbers(num1: int, num2: int) -> int:
    if not isinstance(num1, int) or not isinstance(num2, int):
        raise ValueError("Arguments must be integers")
    return num1 + num2