Войти Регистрация
Код-ревью

29. Заявка на возврат товара в Ozon

Средняя

Python · 41 строк

Покупатель выбирает в заказе товары и оформляет возврат. Окно — 14 дней с получения, бельё, косметика и цифровые товары не возвращаются. Если вернули весь заказ, возвращаем и платную доставку.

Функцию вызывает API мобильного приложения: order_id и item_ids приходят из JSON как есть. В проекте USE_TZ = True, поля цен — DecimalField. Отметьте строки с ошибками.

В работе: Раздел «Мои заказы» → «Вернуть товар»: в распродажу 11.11 сюда приходят тысячи заявок в час.

Как проверяем Нажмите на строку с ошибкой и коротко напишите, что не так. Баг засчитан, если отмечена хотя бы одна его строка. Зачёт — найдено не меньше 60 % багов с учётом серьёзности (критичный весит 3, обычный 2, мелкий 1), а лишних отметок не больше, чем багов в коде. Пустые строки, скобки и комментарии лишними не считаем. Ответ один на подход: после него показываем разбор.
Django ORMчасовые поясаденьгиSQL-инъекцияN+1

Источник: наша задача на данных Ozon: возвраты и заказы. Условие пересказано своими словами.

  1. from datetime import datetime, timedelta
  3. from django.db import connection
  5. from .models import Order, OrderItem, ReturnRequest
  6. from .notify import send_sms
  8. RETURN_DAYS = 14
  9. NON_RETURNABLE = {"underwear", "cosmetics", "digital"}
  12. def create_return(user, order_id, item_ids, reason):
  13. order = Order.objects.get(id=order_id)
  14. if order.delivered_at is None:
  15. return {"error": "Заказ ещё не получен"}
  16. deadline = order.delivered_at + timedelta(days=RETURN_DAYS)
  17. if datetime.now() > deadline:
  18. return {"error": "Срок возврата истёк"}
  20. refund = 0.0
  21. items = []
  22. for item_id in item_ids:
  23. item = OrderItem.objects.get(id=item_id)
  24. if item.product.category in NON_RETURNABLE:
  25. continue
  26. refund += item.price * item.quantity
  27. items.append(item)
  29. if order.delivery_paid and len(items) == order.items.count():
  30. refund += order.delivery_price
  32. with connection.cursor() as cur:
  33. cur.execute(
  34. "UPDATE order_items SET status = 'return_requested' "
  35. f"WHERE order_id = {order_id} AND id IN ({','.join(map(str, item_ids))})"
  36. )
  37. req = ReturnRequest.objects.create(
  38. order=order, user=user, reason=reason, refund=round(refund, 2)
  39. )
  40. send_sms(order.phone, f"Заявка на возврат {req.id}: вернём {refund} ₽")
  41. return {"id": req.id, "refund": refund}