Задача: вернуть список заказов с опциональной передачей аргументов limit
(это максимум сколько записей вернуть), и offset
(это сколько сместить с начала)
Вот мой код, он работает, но мне кажется можно записать попроще, но не приходит в голову, как?
Основная проблема в том, что их могут и не передать, или передать только один, поэтому приходится каждый случай портянками обрабатывать...
def get_queryset(self):
limit = self.request.query_params.get('LIMIT')
offset = self.request.query_params.get('OFFSET')
limit = int(limit) if limit else None
offset = int(offset) if offset else None
if limit and offset:
return Orders.objects.all()[offset:offset+limit]
if limit:
return Orders.objects.all()[:limit]
if offset:
return Orders.objects.all()[offset:]
return Orders.objects.all()
LIMIT
иOFFSET
окажутся не числами (а какой-нибудь злобный хакер вроде меня обязательно попробует подставить не-числа), тоint()
вывалит ошибкуlimit
это обычно тоже плохая идея, потому что если вдруг в базе лежит миллионы Orders'ов, то сервер будет очень сильно мучаться, пытаясь вывести все эти миллионы при отсутствующем limit (а ещё кстати не забывайте, что тот же злобный хакер может подставить отрицательные числа)limit = min(max(0, limit), 100)
иoffset = max(0, offset)
то от всех этих трёх if'ов можно будет избавиться и написать простоreturn Orders.objects.all()[offset:offset+limit]
потому что значения offset и limit будут гарантированно корректные