Есть ли возможность как-то оптимизировать следующий запрос?

Question

[klinnov]

Есть ли возможность как-то оптимизировать следующий запрос? Спасибо!

SELECT t_ps.*, cl.*, c.*, ml.description, m.name as m_name
FROM `tt_payment_settings`    t_ps
  JOIN `tt_card`                c ON t_ps.id_card = c.id_card
  JOIN `tt_card_lang`            cl ON cl.id_card = c.id_card
  JOIN `tt_module`            m USING (id_module)
  JOIN `tt_module_lang`        ml ON ml.id_module = t_ps.id_module

WHERE IF(
    (
      SELECT COUNT(t_ps.id_payment_settings)
      FROM `tt_payment_settings`    t_ps
        JOIN `tt_card`                c ON t_ps.id_card = c.id_card
        JOIN `tt_card_lang`            cl ON cl.id_card = c.id_card
        JOIN `tt_module`            m USING (id_module)
        JOIN `tt_module_lang`        ml ON ml.id_module = t_ps.id_module

      WHERE t_ps.id_shop = 651
            AND t_ps.id_season = 7
            AND cl.id_lang = 1
            AND ml.id_lang = 1
    )>0
    , t_ps.id_shop = 651
      AND t_ps.id_season = 7
      AND cl.id_lang = 1
      AND ml.id_lang = 1
    , t_ps.id_shop = 651
      AND t_ps.id_season IS NULL
      AND cl.id_lang = 1
      AND ml.id_lang = 1
)

--- UPD
Переделал так:

SELECT tt.*, cl.*, c.*, ml.description, m.name as m_name
FROM `tt_payment_settings`    ps
  JOIN `tt_card`                c ON tt.id_card = c.id_card
  JOIN `tt_card_lang`            cl ON cl.id_card = c.id_card
  JOIN `tt_module`            m USING (id_module)
  JOIN `tt_module_lang`        ml ON ml.id_module = tt.id_module

WHERE tt.id_shop = 651
      AND cl.id_lang = 1
      AND ml.id_lang = 1
      AND tt.goo_active = 1
      AND
      IF(
          EXISTS(SELECT *
                 FROM `tt_payment_settings`    ps
                 WHERE tt.id_shop = 651
                       AND tt.id_season = 7)

          ,tt.id_season = 7
          ,tt.id_season IS NULL
      )

Если можно еще как-то лучше буду рад увидеть

Answer 1

[Артём Каретников]

Как минимум за проверку наличия записи через count надо 10 суток гауптической вахты выписать. Юзай exists.

Как максимум, попробуй объяснить, что именно ты тут пытаешься сделать. Потому что выглядит эта конструкция отвратительно.
Правильный код лаконичен и краток. Если же есть дублирования одного и того же, то скорее всего там косяки.

Comments

[klinnov]

Спасибо за ответ! Пытаюсь сделать следующее:
есть таблица, в ней есть записи при выборке нужно делать проверку если есть хотя бы одна запись с id_season то выбирать по условию 1 ( id_season и id_shop) а если нет то выбирать по условию 2 ( only id_shop)

[Артём Каретников]

Надо проверять по exists. Можешь сделать либо два запроса, либо попробовать вставить exists в условие

т.е.

where id_shop = 512
and (
( exists (select * from tbl_session) and id_session = 215)
)
Т.е If в условии тебе не нужен. У тебя ж логические операции есть :)

Дальше. Одно и то же. Таблицы у тебя заджойнены? Значение id_land может быть разным?
AND cl.id_lang = 1
AND ml.id_lang = 1

Дальше:
,tt.id_season = 7
,tt.id_season IS NULL
Это что? Может, ты случайно имел ввиду это: tt.id_season = 7 OR tt.id_season IS NULL

Если со всей прямотой, май френд, тебе надо остановиться, пока не поздно и как следует обсудить со знающим человеком структуру базы. Как я понимаю, ты делаешь магазин, если на этапе старта накосячить с этим, то как более менее разовьетесь, будет жуть. А переделывать будет уже поздно, будет куча кода связанного - традиционные грабли.
Вот переменные я бы сделал еще, зачем мне два раза писать t.id_shop = 651, когда я могу использовать t.id_shop = iId_shop и сохранить запрос как процедуру?

Answer 2

[Антон Антон]

заменить if на union двух select. Про exists написали ранее.

Comments

[klinnov]

Спасибо за ответ! Сделал по другому, обновил. Можете посмотреть? Если можно еще как-то лучше буду рад увидеть

[Антон Антон]

добавьте вывод explain https://dev.mysql.com/doc/refman/5.7/en/explain.html

[klinnov]

Антон Антон: prnt.sc/ev13ot