10 промтов для рефакторинга legacy кода: от хаоса к порядку с помощью AI

Проанализируй следующий код на Python. Найди:
1. Неиспользуемые импорты
2. Переменные, которые объявлены, но нигде не используются
3. Функции, которые никогда не вызываются
4. Закомментированные блоки кода старше 3 месяцев (предположи)

Верни только исправленный код с краткими пояснениями, почему удалил каждый фрагмент.

Код:
[вставьте ваш код]

# Было:
import os
import sys
from datetime import datetime

def old_func():
    pass

def main():
    x = 10
    y = 20  # не используется
    print(x)

# Стало:
import os  # оставлен, если используется ниже по коду

def main():
    x = 10
    print(x)
# Примечание: удалён импорт sys (не используется), datetime (не используется), функция old_func (не вызывается), переменная y.

В коде ниже найди все магические числа (числовые литералы, кроме 0, 1, -1, True, False). Замени их на константы с понятными именами в формате UPPER_SNAKE_CASE. Константы размести в начале файла или в отдельном конфиг-словаре.

Код:
[вставьте ваш код]

# Было:
def calculate_discount(price, user_type):
    if user_type == 'vip':
        return price * 0.8
    else:
        return price * 0.95

# Стало:
VIP_DISCOUNT_RATE = 0.8
REGULAR_DISCOUNT_RATE = 0.95

def calculate_discount(price, user_type):
    if user_type == 'vip':
        return price * VIP_DISCOUNT_RATE
    else:
        return price * REGULAR_DISCOUNT_RATE

Класс UserManager выполняет следующие функции: аутентификация, работа с БД, отправка email, валидация данных, логирование. Раздели его на несколько классов в соответствии с принципом единственной ответственности (SRP). Используй паттерны: Repository для БД, Service для логики, Logger для логирования. Сохрани обратную совместимость через фасад, если необходимо.

Код:
[вставьте ваш код]

# До: один класс UserManager с 500+ строк
# После:
class UserRepository:
    def save(self, user): ...

class EmailService:
    def send_welcome(self, email): ...

class AuthService:
    def login(self, credentials): ...

class UserFacade:
    def __init__(self):
        self.repo = UserRepository()
        self.email = EmailService()
        self.auth = AuthService()
    # фасадные методы для обратной совместимости

Функция process_payment содержит 7 вложенных условий для разных типов платежей (credit_card, paypal, crypto, apple_pay, google_pay, bank_transfer, gift_card). Рефакторинг: создай интерфейс PaymentMethod и конкретные классы для каждого типа. Используй фабричный метод для выбора стратегии.

Код:
[вставьте ваш код]

from abc import ABC, abstractmethod

class PaymentMethod(ABC):
    @abstractmethod
    def pay(self, amount): ...

class CreditCardPayment(PaymentMethod):
    def pay(self, amount): ...

class PayPalPayment(PaymentMethod):
    def pay(self, amount): ...

class PaymentFactory:
    methods = {
        'credit_card': CreditCardPayment,
        'paypal': PayPalPayment,
    }
    @staticmethod
    def get_method(type):
        return PaymentFactory.methods[type]()

Дан следующий процедурный код на PHP (или Python/JS). Он представляет собой 1000+ строк с глобальными переменными, функциями и прямыми вызовами БД. Рефакторинг: перепиши его в ООП-стиле с использованием Dependency Injection, Repository Pattern и Service Layer. Сохрани ту же бизнес-логику, но сделай код тестируемым. Добавь интерфейсы.

Код:
[вставьте ваш код]

// До: глобальная переменная $db, функция get_users()
// После:
interface UserRepositoryInterface {
    public function findAll(): array;
}

class MySQLUserRepository implements UserRepositoryInterface {
    private PDO $db;
    public function __construct(PDO $db) { $this->db = $db; }
    public function findAll(): array { ... }
}

class UserService {
    public function __construct(private UserRepositoryInterface $repo) {}
    public function getActiveUsers(): array { ... }
}

В коде ниже используется цикл для загрузки связанных сущностей, что вызывает N+1 запрос. Перепиши его с использованием eager loading (JOIN или IN-запрос). Если возможно, используй batch-загрузку. Для Django — select_related/prefetch_related, для Laravel — with(), для SQLAlchemy — joinedload().

Код:
[вставьте ваш код]

# Django: Было
for order in Order.objects.all():
    print(order.customer.name)  # N+1 запросов

# Стало
orders = Order.objects.select_related('customer').all()
for order in orders:
    print(order.customer.name)  # всего 1 запрос

Дан legacy-код без тестов. Он содержит зависимости на внешние API и БД. Напиши для него unit-тесты, используя:
- Mock/Stub для внешних зависимостей
- Golden Master Testing (сначала записать вывод, потом сравнивать)
- Покрытие ключевых ветвей (min 80%)

Код:
[вставьте ваш код]

import pytest
from unittest.mock import Mock

def test_process_order_golden_master():
    # Arrange
    mock_api = Mock()
    mock_api.fetch_data.return_value = {"status": "ok"}
    service = OrderService(mock_api)

    # Act
    result = service.process(123)

    # Assert
    assert result == {"order_id": 123, "status": "processed"}
    mock_api.fetch_data.assert_called_once_with(123)

Монолитное приложение имеет 50 000 строк в одном файле. Предложи архитектурный рефакторинг: выдели минимум 5 модулей по бизнес-доменам (например: каталог, корзина, пользователи, заказы, платежи). Для каждого модуля определи:
- Границы ответственности
- Интерфейсы для взаимодействия
- Схему зависимостей (избегай циклических)
- Пример структуры папок

Код прилагается (если объёмный — укажи ключевые классы).

project/
├── modules/
│   ├── catalog/        # Товары, категории
│   ├── cart/           # Корзина (зависит от catalog)
│   ├── user/           # Пользователи, роли
│   ├── order/          # Заказы (зависит от cart, user, payment)
│   └── payment/        # Платежи
├── shared/             # Общие утилиты, DTO
└── main.py             # Точка входа

Код написан на jQuery/CodeIgniter 3. Предложи план миграции на React + Laravel 11. Конвертируй следующие функции:
- AJAX-запросы (заменить на fetch/axios)
- Манипуляции DOM (заменить на Virtual DOM)
- Роутинг (заменить на Laravel routes + React Router)
- Работа с формами (заменить на Formik или Livewire)

Давай пошагово: сначала бэкенд-API, затем фронтенд-компоненты.

// jQuery: $.ajax({ url: '/users', method: 'GET' })
// React: useEffect(() => { fetch('/api/users').then(...) }, [])

Проведи аудит безопасности следующего legacy-кода. Ищи:
1. Конкатенацию строк в SQL-запросах (SQL-инъекции)
2. Неэкранированный вывод пользовательских данных в HTML (XSS)
3. Отсутствие CSRF-токенов
4. Использование insecure deserialization (pickle, eval, unserialize)
5. Хранение паролей в открытом виде

Для каждой уязвимости укажи: строку кода, тип уязвимости, риск (Critical/High/Medium), рекомендацию по исправлению.

Код:
[вставьте ваш код]