Какие знаешь признаки нечистого кода

1. Длинные функции/методы – если функция делает больше одной логической операции или её сложно понять за 30 секунд.
2. Повторяющийся код (DRY violation) – дублирование логики вместо выноса в отдельные функции/классы.
3. Магические числа/строки – использование "голых" значений без констант или пояснений.
4. Слишком много параметров – функции с 5+ аргументами усложняют чтение и тестирование.
5. Глобальные переменные – неочевидные зависимости между частями кода.
6. Жёсткая связность (tight coupling) – классы/модули сильно зависят друг от друга.
7. Неочевидные имена – a, temp, data вместо user_ids или calculate_tax().

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

def process(x, y, z):  # Что делают x, y, z?
    a = x * 1.23 + y  # 1.23 – магическое число
    if z > 10: 
        a += 5 
    return a

Исправлено:

TAX_RATE = 1.23
BONUS_THRESHOLD = 10

def calculate_total(base_price, discount, items_count):
    total = base_price * TAX_RATE + discount
    if items_count > BONUS_THRESHOLD:
        total += 5 
    return total