Перейти до вмісту
Пошук в
  • Детальніше...
Шукати результати, які ...
Шукати результати в ...

блог для натуралов

  • записів
    9
  • коментаря
    33
  • перегляд
    7 781

кастомный getProducts для категории


kJlukOo

1 109 переглядів

catalog/model/catalog/product.php

метод getProducts имеет следующий цикл

foreach ($query->rows as $result) {
	$product_data[$result['product_id']] = $this->getProduct($result['product_id']);
}

те на каждый товар категории у нас создается отдельный запрос с кучей условий. как избавится от порождения кучи новых запросов?

получить все в одном запросе. без цикличного вызова метода getProduct

foreach ($query->rows as $result) {
	$product_data[$result['product_id']] = $result;
}

в родном запросе мы получим: id, рейтинг, акцию и дискаунт

в моем случае для категории мне нужно:  количество, изображение, название товара, id , цена

//исходный
$sql = "SELECT p.product_id, (SELECT AVG(rating) AS total FROM " . DB_PREFIX . "review r1 WHERE r1.product_id = p.product_id AND r1.status = '1' GROUP BY r1.product_id) AS rating, (SELECT price FROM " . DB_PREFIX . "product_discount pd2 WHERE pd2.product_id = p.product_id AND pd2.customer_group_id = '" . (int)$this->config->get('config_customer_group_id') . "' AND pd2.quantity = '1' AND ((pd2.date_start = '0000-00-00' OR pd2.date_start < NOW()) AND (pd2.date_end = '0000-00-00' OR pd2.date_end > NOW())) ORDER BY pd2.priority ASC, pd2.price ASC LIMIT 1) AS discount, (SELECT price FROM " . DB_PREFIX . "product_special ps WHERE ps.product_id = p.product_id AND ps.customer_group_id = '" . (int)$this->config->get('config_customer_group_id') . "' AND ((ps.date_start = '0000-00-00' OR ps.date_start < NOW()) AND (ps.date_end = '0000-00-00' OR ps.date_end > NOW())) ORDER BY ps.priority ASC, ps.price ASC LIMIT 1) AS special";

 

заменим

SELECT p.product_id

на

SELECT p.product_id, p.image, p.price, p.quantity, pd.name

 

$sql = "SELECT p.product_id, p.image, p.price, p.quantity, pd.name, (SELECT AVG(rating) AS total FROM " . DB_PREFIX . "review r1 WHERE r1.product_id = p.product_id AND r1.status = '1' GROUP BY r1.product_id) AS rating, (SELECT price FROM " . DB_PREFIX . "product_discount pd2 WHERE pd2.product_id = p.product_id AND pd2.customer_group_id = '" . (int)$this->config->get('config_customer_group_id') . "' AND pd2.quantity = '1' AND ((pd2.date_start = '0000-00-00' OR pd2.date_start < NOW()) AND (pd2.date_end = '0000-00-00' OR pd2.date_end > NOW())) ORDER BY pd2.priority ASC, pd2.price ASC LIMIT 1) AS discount, (SELECT price FROM " . DB_PREFIX . "product_special ps WHERE ps.product_id = p.product_id AND ps.customer_group_id = '" . (int)$this->config->get('config_customer_group_id') . "' AND ((ps.date_start = '0000-00-00' OR ps.date_start < NOW()) AND (ps.date_end = '0000-00-00' OR ps.date_end > NOW())) ORDER BY ps.priority ASC, ps.price ASC LIMIT 1) AS special";

 

//тесты

 

10 срезов по ttfb показали

категория 50 товаров. 62.8 => 23.5 ms

категория 30 товаров: 45.8 => 22.7 ms

 

 

$startTime = microtime(true);

$query = $this->db->query($sql);
foreach ($query->rows as $result) {
	$product_data[$result['product_id']] = $result;
	//$product_data[$result['product_id']] = $this->getProduct($result['product_id']);
}

$msec = (microtime(true) - $startTime)*1000;

if($_SERVER['REMOTE_ADDR']=='ваш айпи')
{
	echo round($msec,2);
	exit();
}

 

 

 

  • +1 1

10 коментарів


Recommended Comments

а как к этому отнесутся фильтры? 

 

мне кажется подобные модели трогать категорически запрещено не модами 

 

я бы сделал чтоб getProduct и getProducts принимали параметры типа не считать скидку, не считать рейтинг и тп

Надіслати
28 минут назад, spectre сказал:

а как к этому отнесутся фильтры? 

 

мне кажется подобные модели трогать категорически запрещено не модами 

 

я бы сделал чтоб getProduct и getProducts принимали параметры типа не считать скидку, не считать рейтинг и тп

дримфильтр на такой кастомизации работает исправно. окфильтр надо будет потестить

про то, что надо делать модом согласен

 

 

Надіслати

Никогда такого не было и вот опять.
Все результаты запросов актуальны в формате вывода SELEC SQL_NO_CACHE.

Где EXPLAIN запросов  с планом запроса ?


Что за цифры с потолка, как же group by и sort по сторонним таблицам.
Кто то тут крепко свистит, да еще и предлагает какую то дичайшую несовместимую дичь, которая приведет к конфликтам со всем!

  • +1 1
Надіслати
1 час назад, Yoda сказал:

Кто то тут крепко свистит, да еще и предлагает какую то дичайшую несовместимую дичь, которая приведет к конфликтам со всем!

во тут реально конструктивно было. конфликты будут с микроволной печью в первую очередь. затестил цифры с sql_no_cache и разрыв в первом байте стал еще вкуснее на каких-то 5-10мс

 

Отображение строк 0 - 49 (50 всего, Запрос занял 0.0276 сек.)
SELECT SQL_NO_CACHE p.image, p.price, p.quantity, pd.name , (SELECT price FROM oc_product_special ps WHERE ps.product_id = p.product_id AND ps.customer_group_id = '1' AND ((ps.date_start = '0000-00-00' OR ps.date_start < NOW()) AND (ps.date_end = '0000-00-00' OR ps.date_end > NOW())) ORDER BY ps.priority ASC, ps.price ASC LIMIT 1) AS special, p.product_id FROM oc_product_to_category p2c LEFT JOIN oc_product p ON (p2c.product_id = p.product_id) LEFT JOIN oc_product_description pd ON (p.product_id = pd.product_id) LEFT JOIN oc_product_to_store p2s ON (p.product_id = p2s.product_id) WHERE pd.language_id = '1' AND p.status = '1' AND p.date_available <= NOW() AND p2s.store_id = '0' AND p2c.category_id = '118' GROUP BY p.product_id ORDER BY (p.quantity>0) DESC,p.sort_order ASC, LCASE(pd.name) ASC LIMIT 0,50


     

Отображение строк 0 - 49 (50 всего, Запрос занял 0.0200 сек.)
SELECT SQL_NO_CACHE p.product_id, (SELECT price FROM oc_product_special ps WHERE ps.product_id = p.product_id AND ps.customer_group_id = '1' AND ((ps.date_start = '0000-00-00' OR ps.date_start < NOW()) AND (ps.date_end = '0000-00-00' OR ps.date_end > NOW())) ORDER BY ps.priority ASC, ps.price ASC LIMIT 1) AS special, p.product_id FROM oc_product_to_category p2c LEFT JOIN oc_product p ON (p2c.product_id = p.product_id) LEFT JOIN oc_product_description pd ON (p.product_id = pd.product_id) LEFT JOIN oc_product_to_store p2s ON (p.product_id = p2s.product_id) WHERE pd.language_id = '1' AND p.status = '1' AND p.date_available <= NOW() AND p2s.store_id = '0' AND p2c.category_id = '118' GROUP BY p.product_id ORDER BY (p.quantity>0) DESC,p.sort_order ASC, LCASE(pd.name) ASC LIMIT 0,50

 

как полагаешь разница в 0.0076 секунды сможет компенсировать 50 таких запросов(ниже)?

Отображение строк 0 - 0 (1 всего, Запрос занял 0.0016 сек.)
SELECT SQL_NO_CACHE DISTINCT *, pd.name AS name, p.image, (SELECT md.name FROM oc_manufacturer_description md WHERE md.manufacturer_id = p.manufacturer_id AND md.language_id = '1') AS manufacturer, (SELECT price FROM oc_product_discount pd2 WHERE pd2.product_id = p.product_id AND pd2.customer_group_id = '1' AND pd2.quantity = '1' AND ((pd2.date_start = '0000-00-00' OR pd2.date_start < NOW()) AND (pd2.date_end = '0000-00-00' OR pd2.date_end > NOW())) ORDER BY pd2.priority ASC, pd2.price ASC LIMIT 1) AS discount, (SELECT price FROM oc_product_special ps WHERE ps.product_id = p.product_id AND ps.customer_group_id = '1' AND ((ps.date_start = '0000-00-00' OR ps.date_start < NOW()) AND (ps.date_end = '0000-00-00' OR ps.date_end > NOW())) ORDER BY ps.priority ASC, ps.price ASC LIMIT 1) AS special, (SELECT points FROM oc_product_reward pr WHERE pr.product_id = p.product_id AND customer_group_id = '1') AS reward, (SELECT ss.name FROM oc_stock_status ss WHERE ss.stock_status_id = p.stock_status_id AND ss.language_id = '1') AS stock_status, (SELECT wcd.unit FROM oc_weight_class_description wcd WHERE p.weight_class_id = wcd.weight_class_id AND wcd.language_id = '1') AS weight_class, (SELECT lcd.unit FROM oc_length_class_description lcd WHERE p.length_class_id = lcd.length_class_id AND lcd.language_id = '1') AS length_class, (SELECT AVG(rating) AS total FROM oc_review r1 WHERE r1.product_id = p.product_id AND r1.status = '1' GROUP BY r1.product_id) AS rating, (SELECT COUNT(*) AS total FROM oc_review r2 WHERE r2.product_id = p.product_id AND r2.status = '1' GROUP BY r2.product_id) AS reviews, p.sort_order FROM oc_product p LEFT JOIN oc_product_description pd ON (p.product_id = pd.product_id) LEFT JOIN oc_product_to_store p2s ON (p.product_id = p2s.product_id) LEFT JOIN oc_manufacturer m ON (p.manufacturer_id = m.manufacturer_id) WHERE p.product_id = '2510842' AND pd.language_id = '1' AND p.status = '1' AND p.date_available <= NOW() AND p2s.store_id = '0'

 

зачем тебе explain? смешной. условия запроса никак не менялись. экплейн идентичен для двух этих запросов, я думал это само собой понятно... но не тебе :grin:

 

 

 

не убирать дискаунт, чтобы добиться хорошей совместимости будет максимально логично, но мне интересно сделать легче и быстрее

Надіслати
В 09.08.2021 в 00:05, max1985 сказал:

отпишитесь пожалуйста после тестов

Должен работать как и прежде.

 

По поводу этой разработки.

 

getProduct не просто так сделан в цикле. Это единственный и правильный путь для стандартизации структуры данных. Один метод на все случаи.

Логика такая: наборы product_id ищем любым удобным способом, а сами конечные элементы выбираем одним (!) на всю систему методом с нужными нам полями. То есть не плодим себе кучу мест, где нужно добавлять новые поля или форматировать их, следить за всем этим.

 

Чего не хватает в этом методе, так это кэширования с ограниченным временем жизни. И будь-то вызов товаров из блока "Похожие", или "Просмотренные",  любой другой модуль, всегда нужно искать product_id максимально простым запросом без кэширования, а сам product брать из кэша.

 

Также в бэкенде изменяя товар не нужно удалять весь кэш товаров, достаточно delete('product.' . $product_id) и все остальные останутся целыми.

Ну и ко всему прочему, сам по себе запрос выборки одного товара по primary key со всем навесным не такой уж тяжелый. 

 

Ваш метод тоже имеет свои преимущества, но кроме списка товаров в категории есть же еще масса других страниц, где можно всегда обращаться только к одному методу получения товара, а внутри него уже делаем что хотим, кэшируем или разбиваем большой запрос на разные (может со своими условиями), форматируем и т.д.

  • +1 1
Надіслати

Створіть аккаунт або увійдіть для коментування

Ви повинні бути користувачем, щоб залишити коментар

Створити обліковий запис

Зареєструйтеся для отримання облікового запису. Це просто!

Зареєструвати аккаунт

Вхід

Уже зареєстровані? Увійдіть тут.

Вхід зараз
  • Зараз на сторінці   0 користувачів

    • Ні користувачів, які переглядиють цю сторінку

×
×
  • Створити...

Important Information

На нашому сайті використовуються файли cookie і відбувається обробка деяких персональних даних користувачів, щоб поліпшити користувальницький інтерфейс. Щоб дізнатися для чого і які персональні дані ми обробляємо перейдіть за посиланням . Якщо Ви натиснете «Я даю згоду», це означає, що Ви розумієте і приймаєте всі умови, зазначені в цьому Повідомленні про конфіденційність.