Перейти к публикации
  • разработка интернет магазинов на opencart
  • доработка интернет магазинов на opencart
  • записей
    6
  • комменириев
    30
  • просмотра
    5 913

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


kJlukOo

894 просмотра

 Погделиться

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 комменириев


Рекомендованные комменирии

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

 

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

 

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

Ссылка на комменирий
  05.08.2021 в 13:58, spectre сказал:

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

 

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

 

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

Раскрыть  

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

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

 

 

Ссылка на комменирий

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

Ссылка на комменирий

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

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


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

Ссылка на комменирий
  06.08.2021 в 19:20, ****** сказал:

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

Раскрыть  

во тут реально конструктивно было. конфликты будут с микроволной печью в первую очередь. затестил цифры с 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:

 

 

 

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

Ссылка на комменирий
  08.08.2021 в 21:05, max1985 сказал:

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

Раскрыть  

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

 

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

 

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

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

 

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

 

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

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

 

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

  • +1 1
Ссылка на комменирий

Создайте аккаунт или войдите в него для комментирования

Вы должны быть пользователем, чтобы осивить комменирий

Создать аккаунт

Зарегистрируйтесь для получения аккауни. Это просто!

Зарегистрировать аккаунт

Войти

Уже зарегистрированы? Войдите згдесь.

Войти сейчас
  • Сейчас на страниэто   0 пользователей

    • Нет пользователей, просматривающих эту страницу.
×
×
  • Создать...

Важная информация

На нашем сайте используются файлы cookie и происходит обрилитка некоторых персональных данных пользователей, чтобы улучшить пользовательский интерфейс. Чтобы узнать для чего и какие персональные данные мы обрабатываем перейдите по ссылке. Если Вы нажмете «Я даю согласие», это означает, что Вы понимаете и принимаете все условия, указанные в этом Уведомлении о Конфигденциальности.