Что хотят услышать интервьюеры:

Ожидают услышать, что изменения в коде вносятся не «на глаз», а через последовательный рефакторинг. Нужно улучшать структуру малыми безопасными шагами: переименовывать, дробить длинные функции, убирать дублирование, выделять смысловые абстракции. Важно сохранять поведение программы и подкреплять изменения тестами.

Определение:

Для улучшения структуры и читаемости кода применяют рефакторинг — изменение внутреннего устройства кода без изменения его внешнего поведения. Обычно это включает понятные имена, разделение ответственности, уменьшение вложенности, устранение дублирования и выделение повторяющейся логики в функции или методы. Хороший рефакторинг делает код проще для понимания, сопровождения и дальнейшего расширения.

Пример использования:

Было: длинная функция с проверками, вычислениями и форматированием результата в одном месте. После рефакторинга логика делится на небольшие функции с понятными именами.

def calculate_discount(price, is_member):
    if price < 0:
        raise ValueError("price must be non-negative")

    if is_member:
        discount = price * 0.1
    else:
        discount = price * 0.05

    final_price = price - discount
    return round(final_price, 2)

Здесь функция уже достаточно проста, но если бы внутри были дополнительные проверки, форматирование и работа с данными из внешнего источника, это стоило бы вынести в отдельные функции.

Пояснение кода:

В примере функция делает одну понятную вещь — считает цену со скидкой.
Сначала выполняется валидация входных данных, чтобы исключить некорректные значения. Затем выбирается размер скидки в зависимости от признака участника. После этого вычисляется итоговая цена и округляется результат.

Если бы код был сложнее, шаги рефакторинга выглядели бы так:

Найти слишком длинные и перегруженные функции.
Разбить их на части по смыслу.
Дать переменным и функциям говорящие имена.
Убрать повторяющиеся фрагменты.
Проверить, что поведение не изменилось, с помощью тестов.

Ключевые моменты:

Улучшать код лучше небольшими безопасными изменениями, а не переписывать всё сразу.
Основные техники: переименование, извлечение функции, устранение дублирования, упрощение условий.
Полезно снижать вложенность и делать код «плоским» и линейным.
Рефакторинг должен сохранять поведение программы.
После изменений важно прогонять тесты, чтобы убедиться, что логика не сломалась.
Хороший код должен быть читаемым без лишних комментариев за счёт ясной структуры и именования.

Какими методами и подходами следует применять изменения в коде для улучшения его структуры и читаемости?