David-S-IT / yadjango

Образовательный проект по Django от Интенсивы Академии Яндекса
1 stars 1 forks source link

Улучшить логику менеджера и проанализировать вопросы оптимизации #67

Closed drowsycoder closed 1 year ago

drowsycoder commented 1 year ago
  1. Посмотри в записи лекции (23:33), как можно ещё больше улучшить менеджер Данила там упоминал, что это как раз на домашнюю работу

  2. Если посмотреть в DjDT аналитику по запросам на некоторых страницах с несколькими изображениями, наблюдается такая картина с похожими запросами: Screenshot 2023-03-10 at 15-15-01 Товар Гранат (SQL) Посмотри, может ли её поменять какое-то изменение в менеджере, в работе с моделями или в самом шаблоне

Для начала обрати внимание на item_detail

Большая подсказка, чтобы сэкономить время: Можно сначала проработать общий менеджер модели published() для использования в обеих (!) вью-функциях. А дальше в item_detail доуточнить queryset, с которым будем работать Либо определить вообще всё в менеджерах, что будет ещё красивее (только постараться соблюдать принцип DRY и не дублировать код лишний раз)

При правильном подходе это заодно решит проблему из соседнего issue про ошибочно отображаемый неопубликованный товар

drowsycoder commented 1 year ago

В подсказке был намёк, что items_queryset - не самое говорящее имя. Это можно исправить быстро И что можно дооптимизировать - но это дольше. Оставим как задачку на будущее