1. PHP / Говнокод #12103

    +65

    1. 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') . "'");

    Opencart, catalog/model/catalog/product.php

    Они что издеваются? А я-то думаю что у меня ложит mysql сервер на колени...

    Запостил: fooser, 12 Ноября 2012

    Комментарии (12) RSS

    • блин, как теперь отредактировать пост, чтобы разбить на несколько строк?
      Ответить
      • Никак ;)

        P.S. Все равно никто не будет читать и изучать этот запрос, а его длину и так видно.
        Ответить
        • Все же - какой пентагоновский суперкомпьютер нужен чтобы отработать этот запрос, если товаров в каталоге около нескольких сотен? Этот запрос ложит на колени топовую тачку... И что с этим звиздецом делать я хз...
          Ответить
          • Для начала *, убрать после DISTINCT. Оно там не в тему. С остальными агрегатами надо разбираться.
            Ответить
    • Оно и так смотрится волшебно.
      Не очень нравится подобная технология только тем, что я считаю необоснованным использование подзапросов в селекте. Всё вырождается примерно в такую вот хрень.
      Единственное, чего я не знаю, как реализовать вменяемо без подзапроса - лимит по полю.

      По запросу.
      Не вижу совершенно, зачем нужны подселекты по полям reward, stock_status, weight_class, length_class, их можно смело выносить в джойны.
      Ответить
    • Видимо вы не видели коды битрикс.
      https://twitter.com/idenisko/status/253841754132992001/photo/1

      Запрос на весь экран
      Ответить
      • А вот автор запроса походу увидел, воодушевился, и теперь только так и пишет. Ведь в 1с работают профессионалы, а они не могут ошибаться.
        Ответить
      • Программисты битрикса хорошие, просто им надо кормить свои семьи. (q)
        Ответить
    • Может автор этого запроса решил, что подзапросы в SELECT не так грузят сервер как JOIN?
      Ответить
    • ЕБАААААААА O_O
      Ответить
    • Длина тут не при чём. Иногда лучше один раз сделать сложный запрос, чем потом foreach'ем обходить и вызывать для каждой строчки.
      Ответить
      • Открою УЖАСНУЮ тайну. В этом запросе ещё оптимизировать и оптимизировать. Огромнейший простор, короче.
        А запрос такой сложности не нужен. Проще разбить на два-три простых, если однородность данных более-менее позволяет, их можно развернуть из столбцов в строки.
        (На правах копетана) Сложный запрос хуже для понимания, хуже для переработки, не даёт простора для увеличения сложности. Наивный школьник видит, что у него его пыхосайт генерирует десять запросов на страницу и принимает соломоново решение - генерировать один. Только из этого не следует, что скорость обработки увеличится в десять раз. Правильным решением будет оптимизация по факту; отбросить ненужное, что-то закэшировать, что-то вынести в nosql. Короче - очевиднейшие вещи. Большие запросы писать НЕ НАДО. Даже если один. Но по десять.
        Ответить

    Добавить комментарий