Как доработать SQL-запрос?

Question

[lexstile]

Есть запрос:

SELECT DISTINCT events.id, events.name as event_name, events.date_start, events.date_end, events.country_id, events.description, events.day_1, events.day_2, events.month_1, events.month_2, events.year_1, events.year_2, events.is_present, links.links, images.images, countries.countries, countries.country_ids, country_to_cluster.year_start FROM events
		LEFT JOIN (SELECT countries.id, GROUP_CONCAT(countries.id) as country_ids,  event_to_country.country_id as country_id, event_to_country.event_id, GROUP_CONCAT(countries.name) as countries FROM countries LEFT JOIN event_to_country ON countries.id = country_id GROUP BY event_to_country.event_id) as countries ON events.id = countries.event_id
		LEFT JOIN (SELECT event_id, GROUP_CONCAT(link) as links FROM links GROUP BY event_id) AS links ON events.id = links.event_id
		LEFT JOIN (SELECT event_id, GROUP_CONCAT(name) as images FROM images GROUP BY event_id) AS images ON events.id = images.event_id
		LEFT JOIN keywords ON events.id = keywords.event_id
		LEFT JOIN country_to_cluster ON country_to_cluster.country_id = events.country_id AND events.year_1 >= country_to_cluster.year_start AND events.year_1 <= country_to_cluster.year_end
		WHERE (country_to_cluster.cluster_id = :cluster_id) 
		ORDER BY events.year_1 ASC
		LIMIT 30

Можно ли так сделать и как?
Сейчас события выбираются по кластерам, кластер - временной промежуток, есть поля country_to_cluster.year_start и country_to_cluster.year_end.

Возникла необходимость добавления кластера без дат, то есть поля year_start и year_end равны NULL.

Можно ли переделать запрос следующим образом, чтобы он смотрел на даты только в том случае, если они не NULL?

У меня получился такой LEFT JOIN, так?

LEFT JOIN country_to_cluster ON (country_to_cluster.year_start IS NULL AND country_to_cluster.year_end IS NULL && country_to_cluster.country_id = events.country_id) OR (country_to_cluster.country_id = events.country_id AND events.year_1 >= country_to_cluster.year_start AND events.year_1 <= country_to_cluster.year_end)

Answer 1

[Максим]

Такое?

SELECT DISTINCT events.id, events.name as event_name, events.date_start, events.date_end, events.country_id, events.description, events.day_1, events.day_2, events.month_1, events.month_2, events.year_1, events.year_2, events.is_present, links.links, images.images, countries.countries, countries.country_ids, country_to_cluster.year_start FROM events
    LEFT JOIN (SELECT countries.id, GROUP_CONCAT(countries.id) as country_ids,  event_to_country.country_id as country_id, event_to_country.event_id, GROUP_CONCAT(countries.name) as countries FROM countries LEFT JOIN event_to_country ON countries.id = country_id GROUP BY event_to_country.event_id) as countries ON events.id = countries.event_id
    LEFT JOIN (SELECT event_id, GROUP_CONCAT(link) as links FROM links GROUP BY event_id) AS links ON events.id = links.event_id
    LEFT JOIN (SELECT event_id, GROUP_CONCAT(name) as images FROM images GROUP BY event_id) AS images ON events.id = images.event_id
    LEFT JOIN keywords ON events.id = keywords.event_id
    LEFT JOIN country_to_cluster ON country_to_cluster.country_id = events.country_id AND events.year_1 >= coalesce(country_to_cluster.year_start,events.year_1) AND events.year_1 <= coalesce(country_to_cluster.year_end,events.year_1)
    WHERE (country_to_cluster.cluster_id = :cluster_id) 
    ORDER BY events.year_1 ASC
    LIMIT 30

Это будет игнорировать null отдельно для каждого поля year_start и year_end. Если надо именно одновременный null, это будет чуть по другому

Comments

[lexstile]

Похоже, там одновременный NULL.
Пример:
- выбираю два кластера - у одного year_start и year_end заполнены, у второго нет (NULL).
- у первого кластера выводим записи, обращая внимание на поля - year_start и year_end, а у второго - просто проверяем принадлежность к кластеру, без упомянутых полей.

[Максим]

lexstile,

SELECT DISTINCT events.id, events.name as event_name, events.date_start, events.date_end, events.country_id, events.description, events.day_1, events.day_2, events.month_1, events.month_2, events.year_1, events.year_2, events.is_present, links.links, images.images, countries.countries, countries.country_ids, country_to_cluster.year_start FROM events
    LEFT JOIN (SELECT countries.id, GROUP_CONCAT(countries.id) as country_ids,  event_to_country.country_id as country_id, event_to_country.event_id, GROUP_CONCAT(countries.name) as countries FROM countries LEFT JOIN event_to_country ON countries.id = country_id GROUP BY event_to_country.event_id) as countries ON events.id = countries.event_id
    LEFT JOIN (SELECT event_id, GROUP_CONCAT(link) as links FROM links GROUP BY event_id) AS links ON events.id = links.event_id
    LEFT JOIN (SELECT event_id, GROUP_CONCAT(name) as images FROM images GROUP BY event_id) AS images ON events.id = images.event_id
    LEFT JOIN keywords ON events.id = keywords.event_id
    LEFT JOIN country_to_cluster ON country_to_cluster.country_id = events.country_id 
             AND ((events.year_1 >= country_to_cluster.year_start AND events.year_1 <= country_to_cluster.year_end) 
                    or (country_to_cluster.year_start is null and country_to_cluster.year_end is null ))
    WHERE (country_to_cluster.cluster_id = :cluster_id) 
    ORDER BY events.year_1 ASC
    LIMIT 30

[Максим]

lexstile, правда вот это немного неправильно

WHERE (country_to_cluster.cluster_id = :cluster_id)

правильно вот так
where events.country_id = :cluster_id

[lexstile]

Максим, нет, это неверно.
country_id - это id страны
cluster_id - это id кластера

[Максим]

lexstile, ну как неверно

country_to_cluster ON country_to_cluster.country_id = events.country_id

но при

WHERE (country_to_cluster.cluster_id = :cluster_id)

не будет работать LEFT JOIN country_to_cluster

[lexstile]

Максим, вы написали:
where events.country_id = :cluster_id
Но cluster_id != country_id

[Максим]

lexstile, сорри, мой косяк, не заметил
Но тогда у тебя не работает LEFT JOIN country_to_cluster
т.к. ты в where ссылаешься на данные таблицы и как следствие это просто join

[lexstile]

Максим, ааа, понял, у меня просто динамические условия, а запрос как пример основного.
Вот это условие добавляется, если кластеры выбраны:

$terms[] = 'country_to_cluster.cluster_id = :cluster_id';

Я благодарен, что не остались равнодушными и рассказали про возможную проблему!

[Максим]

lexstile, Да тут весь селект, одна большая проблема, особенно тот что Вы написали в соседней ветке =)

[lexstile]

Максим, я понимаю, иначе бы и проблем не было.
Но я пишу его с Божьей помощью, прошу понять.