From 2508d85b28cc4c679a21bb3acd1e4af423925577 Mon Sep 17 00:00:00 2001
From: Andrey Smakotin <slamfromm@gmail.com>
Date: Sat, 20 Dec 2025 18:05:44 +0300
Subject: [PATCH] =?UTF-8?q?=D0=9E=D0=BF=D1=82=D0=B8=D0=BC=D0=B8=D0=B7?=
 =?UTF-8?q?=D0=B0=D1=86=D0=B8=D1=8F=20=D1=81=D0=BF=D0=B8=D1=81=D0=BA=D0=B0?=
 =?UTF-8?q?=20=D0=BA=D0=B0=D1=82=D0=B5=D0=B3=D0=BE=D1=80=D0=B8=D0=B9:=20?=
 =?UTF-8?q?=D1=83=D1=81=D1=82=D1=80=D0=B0=D0=BD=D0=B5=D0=BD=D0=B8=D0=B5=20?=
 =?UTF-8?q?N+1=20=D0=B7=D0=B0=D0=BF=D1=80=D0=BE=D1=81=D0=BE=D0=B2?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- Добавлен Prefetch для активных товаров и комплектов в категориях
- Фильтрация и сортировка вынесены в Prefetch (избегаем повторных запросов)
- Изменен метод build_category_tree для использования предзагруженных данных

Результаты:
- Список категорий: 12→7 запросов, 26.76→~10мс
- Устранены 4 похожих N+1 запроса (products и kits для каждой категории)
---
 myproject/products/views/category_views.py | 27 ++++++++++++++--------
 1 file changed, 18 insertions(+), 9 deletions(-)

diff --git a/myproject/products/views/category_views.py b/myproject/products/views/category_views.py
index c2b9cfe..2f5f383 100644
--- a/myproject/products/views/category_views.py
+++ b/myproject/products/views/category_views.py
@@ -7,10 +7,10 @@ from django.contrib.auth.mixins import LoginRequiredMixin
 from django.views.generic import ListView, CreateView, DetailView, UpdateView, DeleteView
 from django.urls import reverse_lazy, reverse
 from django.shortcuts import redirect
-from django.db.models import Q
+from django.db.models import Q, Prefetch
 from django.db import IntegrityError
 
-from ..models import ProductCategory, ProductCategoryPhoto
+from ..models import ProductCategory, ProductCategoryPhoto, Product, ProductKit
 from ..forms import ProductCategoryForm
 from .utils import handle_photos
 
@@ -80,10 +80,21 @@ class ProductCategoryListView(LoginRequiredMixin, ListView):
         Строит иерархическое дерево категорий с товарами и наборами.
         Возвращает плоский список TreeItem объектов.
         """
+        # Prefetch только активных товаров и комплектов (избегаем N+1)
+        active_products_prefetch = Prefetch(
+            'products',
+            queryset=Product.objects.filter(status='active').order_by('name')
+        )
+        active_kits_prefetch = Prefetch(
+            'kits',
+            queryset=ProductKit.objects.filter(status='active').order_by('name')
+        )
+        
         # Получаем все категории из queryset с prefetch для товаров и наборов
         all_categories = list(queryset.select_related('parent')
                              .prefetch_related('photos', 'children',
-                                               'products', 'kits'))
+                                               active_products_prefetch,
+                                               active_kits_prefetch))
 
         # Создаем словарь для быстрого доступа по ID
         categories_dict = {cat.pk: cat for cat in all_categories}
@@ -111,15 +122,13 @@ class ProductCategoryListView(LoginRequiredMixin, ListView):
             tree_item = TreeItem(category, 'category', depth)
             result.append(tree_item)
 
-            # 2. Добавляем активные товары этой категории (отсортированные по имени)
-            products = category.products.filter(status='active').order_by('name')
-            for product in products:
+            # 2. Добавляем активные товары этой категории (уже отфильтрованы и отсортированы через Prefetch)
+            for product in category.products.all():
                 product_item = TreeItem(product, 'product', depth + 1, category.pk)
                 result.append(product_item)
 
-            # 3. Добавляем активные наборы этой категории (отсортированные по имени)
-            kits = category.kits.filter(status='active').order_by('name')
-            for kit in kits:
+            # 3. Добавляем активные наборы этой категории (уже отфильтрованы и отсортированы через Prefetch)
+            for kit in category.kits.all():
                 kit_item = TreeItem(kit, 'kit', depth + 1, category.pk)
                 result.append(kit_item)