Как лучше оптимизировать SQL?

Question

[Saip]

Есть код на Джанго, выводит на страницу 3 категории товаров в отдельные табы. Можно ли как то ещё оптимизировать код?

def catalog(request):
    wallets = Products.objects.filter(gender="men", categories=1).select_related("gender", "categories").only("id", "title", "price", "image")
    bags = Products.objects.filter(gender="men", categories=2).select_related("gender", "categories").only("id", "title", "price", "image")
    belts = Products.objects.filter(gender="men", categories=3).select_related("gender", "categories").only("id", "title", "price", "image")
    contex = {"wallets": wallets, "bags": bags, "belts": belts}
    return render(request, 'catalog.html', context)

Comments

[chemdev]

Можно сделать один запрос и разобрать на 3 списка уже на питоне

[alexalexes]

Что-то не масштабируемо.
Что вы будете делать, когда появится categories=4 ?
Будете править код - так не пойдет!

[Saip]

alexalexes, Как лучше сделать?

[maksam07]

Сайпи Курбанов, как минимум цикл по всем категориям, чтобы не было проблем добавить/удалить любую другую. Но если говорить о лучшем варианте, то вряд ли этот будет считаться хорошим

[Saip]

maksam07, Может получать в функции все категории Categories.objects.all(), а потом в шаблоне через цикл вывести категории и к ним товары через _set.all ?

Answer 1

[nnn Xion]

можно сделать лишь один запрос в бд вместо n количество зависящее от количества категорий.

from django.db.models import Q
from collections import defaultdict

def catalog(request):
    categories = {
        1: "wallets",
        2: "bags",
        3: "belts",
...
    }

    products = Products.objects.filter(gender="men").filter(categories__in=categories.keys()). select_related("gender", "categories").only("id", "title", "price", "image")

    categorized_products = defaultdict(list)

    for product in products:
        category = categories[product.categories_id]
        categorized_products[category].append(product)

    context = categorized_products
    return render(request, 'catalog.html', context)

Comments

[Saip]

Как это в шаблон выводится?
{% for i in categorized_products[1] %} типо такого?

[nnn Xion]

Сайпи Курбанов, yes

можно типо такого

{% for category, products in categorized_products.items() %}
{% for product in products %}
...

[Saip]

nnn Xion, Спасибо за ответ. Всё получилось, только ругается на скобки в шаблоне. И без них ничего не выводит

Answer 2

[Antonio Solo]

1) зачем нужен `select_related("gender", "categories")` ? ведь и то и другое не являются relations

2) всегда ли gender="man" ?

в итоге я бы сделал так

GENDER_CATEGORIES = [
    ("men", 1),  
    ("men", 2),  
    ("men", 3)   
]

from django.db.models import Q

def catalog(request):
    filters = [Q(gender=gender, categories=category_id) for gender, category_id in GENDER_CATEGORIES]
    query = Q()
    for q_filter in filters:
        query |= q_filter

    products = Products.objects.filter(query).only("id", "title", "price", "image")

    products_by_category = {category[1]: [] for category in GENDER_CATEGORIES}
    for product in products:
        for gender, category_id in GENDER_CATEGORIES:
            if product.gender == gender and product.categories == category_id:
                products_by_category[category_id].append(product)
                break

    context = {"products_by_category": products_by_category}
    return render(request, 'catalog.html', context)

ну и еще в тапл добавить имя 3-м параметром

Comments

[Saip]

У меня в проэкте Categories это модель, сделанно для того чтобы администратор мог сам добавлять и удалять категории через панель, а не в коде