- 1
$query = $this->db->query("SELECT DISTINCT *, pd.name AS name, p.image, m.name AS manufacturer, (SELECT price FROM " . DB_PREFIX . "product_discount pd2 WHERE pd2.product_id = p.product_id AND pd2.customer_group_id = '" . (int)$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)$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 points FROM " . DB_PREFIX . "product_reward pr WHERE pr.product_id = p.product_id AND customer_group_id = '" . (int)$customer_group_id . "') AS reward, (SELECT ss.name FROM " . DB_PREFIX . "stock_status ss WHERE ss.stock_status_id = p.stock_status_id AND ss.language_id = '" . (int)$this->config->get('config_language_id') . "') AS stock_status, (SELECT wcd.unit FROM " . DB_PREFIX . "weight_class_description wcd WHERE p.weight_class_id = wcd.weight_class_id AND wcd.language_id = '" . (int)$this->config->get('config_language_id') . "') AS weight_class, (SELECT lcd.unit FROM " . DB_PREFIX . "length_class_description lcd WHERE p.length_class_id = lcd.length_class_id AND lcd.language_id = '" . (int)$this->config->get('config_language_id') . "') AS length_class, (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 COUNT(*) AS total FROM " . DB_PREFIX . "review r2 WHERE r2.product_id = p.product_id AND r2.status = '1' GROUP BY r2.product_id) AS reviews, p.sort_order FROM " . DB_PREFIX . "product p LEFT JOIN " . DB_PREFIX . "product_description pd ON (p.product_id = pd.product_id) LEFT JOIN " . DB_PREFIX . "product_to_store p2s ON (p.product_id = p2s.product_id) LEFT JOIN " . DB_PREFIX . "manufacturer m ON (p.manufacturer_id = m.manufacturer_id) WHERE p.product_id = '" . (int)$product_id . "' AND pd.language_id = '" . (int)$this->config->get('config_language_id') . "' AND p.status = '1' AND p.date_available <= NOW() AND p2s.store_id = '" . (int)$this->config->get('config_store_id') . "'");
fooser 12.11.2012 13:03 # +1
bormand 12.11.2012 13:05 # +1
P.S. Все равно никто не будет читать и изучать этот запрос, а его длину и так видно.
fooser 12.11.2012 13:12 # 0
DBdev 12.11.2012 13:58 # +3
eth0 12.11.2012 18:09 # 0
Не очень нравится подобная технология только тем, что я считаю необоснованным использование подзапросов в селекте. Всё вырождается примерно в такую вот хрень.
Единственное, чего я не знаю, как реализовать вменяемо без подзапроса - лимит по полю.
По запросу.
Не вижу совершенно, зачем нужны подселекты по полям reward, stock_status, weight_class, length_class, их можно смело выносить в джойны.
sleeper 13.11.2012 09:04 # +1
https://twitter.com/idenisko/status/253841754132992001/photo/1
Запрос на весь экран
bormand 13.11.2012 09:43 # 0
eth0 13.11.2012 19:39 # +6
guest 23.11.2012 19:45 # +1
guest 23.11.2012 19:56 # −3
MikleSmart 26.11.2012 11:04 # −1
eth0 26.11.2012 21:06 # +2
А запрос такой сложности не нужен. Проще разбить на два-три простых, если однородность данных более-менее позволяет, их можно развернуть из столбцов в строки.
(На правах копетана) Сложный запрос хуже для понимания, хуже для переработки, не даёт простора для увеличения сложности. Наивный школьник видит, что у него его пыхосайт генерирует десять запросов на страницу и принимает соломоново решение - генерировать один. Только из этого не следует, что скорость обработки увеличится в десять раз. Правильным решением будет оптимизация по факту; отбросить ненужное, что-то закэшировать, что-то вынести в nosql. Короче - очевиднейшие вещи. Большие запросы писать НЕ НАДО. Даже если один. Но по десять.