Jump to content
Search In
  • More options...
Find results that contain...
Find results in...
  • Sign Up
  • entries
    5
  • comments
    29
  • views
    2,216

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


kJlukOo

579 views

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 Comments


Recommended Comments

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

 

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

 

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

Link to comment
28 минут назад, spectre сказал:

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

 

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

 

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

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

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

 

 

Link to comment

Вообще такой запрос полезен там, где нужно сделать подборку товаров аля модуль какой-нибудь вывода товаров

Link to comment

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

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


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

Link to comment
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:

 

 

 

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

Link to comment

А смысл рассматривать частный случай?

 

 

  • +1 1
Link to comment
В 05.08.2021 в 17:35, kJlukOo сказал:

окфильтр надо будет потестить

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

Link to comment
В 09.08.2021 в 00:05, max1985 сказал:

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

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

 

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

 

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

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

 

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

 

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

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

 

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

  • +1 1
Link to comment

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now
  • Recently Browsing   0 members

    No registered users viewing this page.

×
×
  • Create New...

Important Information

On our site, cookies are used and personal data is processed to improve the user interface. To find out what and what personal data we are processing, please go to the link. If you click "I agree," it means that you understand and accept all the conditions specified in this Privacy Notice.