Промпт для технического ревью и проверки качества кода

Для демонстрации я выбрал реалистичный бизнес-кейс: написание функции на Python для расчета скидки в интернет-магазине.

Входные данные от пользователя: «Напиши функцию на Python, которая принимает сумму заказа и возвращает размер скидки. Если сумма больше 1000 — скидка 5%, если больше 5000 — 10%. Добавь проверку на отрицательные числа».

Ревью-отчет

1. Соответствие цели и требованиям

Запрос выполнен частично. Код будет работать, но условия в ТЗ допускают неоднозначность (порядок проверок) и отсутствие обработки типов данных.

2. Дефекты и риски

• Логическая ошибка (Medium): Если сначала проверить > 1000, то условие > 5000 никогда не выполнится. Нужно соблюдать порядок от большего к меньшему.

• Типизация (Low): Входные данные могут быть строкой или None, что вызовет ошибку TypeError.

• Масштабируемость (Low): Хардкод порогов скидок затрудняет изменение правил маркетинга.

3. Улучшенная версия (Best Practice)

def get_discount(amount: float) -> float:
    if not isinstance(amount, (int, float)):
        raise ValueError("Сумма должна быть числом")
    if amount < 0:
        raise ValueError("Сумма не может быть отрицательной")

    if amount > 5000:
        return amount * 0.10
    elif amount > 1000:
        return amount * 0.05
    return 0.0

4. Ответ модели

Вот оптимизированный код функции. Я учел порядок условий: сначала проверяем самый высокий порог (5000), иначе заказы дороже 5000 всегда будут попадать под ветку 5% скидки. Также добавлена обработка исключений для отрицательных значений.

5. Чеклист финальной проверки

• [x] Обработка отрицательных чисел? (Да)

• [x] Верная логика ветвления (5000 перед 1000)? (Да)

• [x] Соответствие типам возвращаемых значений? (Да)

• [x] Читаемость кода (PEP8)? (Да)

6. Оценка готовности

Статус: GO. Код безопасен и логически корректен.

...