Почему запрос возвращает только запись по первому условию?

Question

[lexstile]

SELECT 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, clusters.clusters FROM events
		LEFT JOIN (SELECT countries.id, event_to_country.event_id, GROUP_CONCAT(countries.name) as countries FROM countries LEFT JOIN event_to_country ON countries.id = event_to_country.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 (SELECT country_id as c_id, GROUP_CONCAT(name) as clusters FROM clusters GROUP BY c_id) AS clusters ON countries.id = c_id
		WHERE countries.id = :country_id 
		ORDER BY events.year_1 ASC
		LIMIT 30

Вот таблица связей:

Отрабатывает странно, выводит только события с первой связью.
То есть:
Если countries.id = 2 - выводится только событие с идентификатором 142, но должен и с id = 141, т. к. связь имеется.
Событие с идентификатором 141 выводится только в том случае, если передать countries.id = 8.

Answer 1

[Alexey Gnevyshev]

Может быть, поэтому?
GROUP BY

Comments

[lexstile]

Эм, ну если не ставить GROUP BY, тоже вообще ничего не находит.

[Alexey Gnevyshev]

Где-то лишняя группировка, это точно.
Может, тогда здесь?
GROUP BY event_to_country.event_id
Но, убирая GROUP BY, смотрите чтобы остальное в подзапросе было корректно. Например, GROUP_CONCAT уже может быть лишним.

[lexstile]

Алексей Гневышев, GROUP_CONCAT нужно для вывода стран в общем списке событий:

Если я выберу одну из стран: Франция, Индия, Гватемала - выборка вернется пустой, почему-то срабатывает только поиск по первой стране.

Проблема в том, что появилась третья таблица - таблица связей, раньше было две - events (основная, в ней хранился id страны) и countries (страны).
Сейчас же добавилась третья - event_to_country, которая хранит event_id и country_id.
То есть, у одного события может быть несколько привязанных стран.

[Alexey Gnevyshev]

WHERE countries.id = :country_id
А это, так задумано?

[lexstile]

Алексей Гневышев, это один из вариантов, который я пробовал, возможно, вы намекаете на нечто нелогичное, но пока не соображу, на что именно.

[lexstile]

events (country_id здесь устаревшее, просто не убрал):


countries

[Alexey Gnevyshev]

:country_id - это bind переменная?
В ней точно все нужные значения? )

[lexstile]

Алексей Гневышев, bind - да
сейчас в ней одно значение - Россия (id = 2, должно выводить два события, но выводится одно)
Если будет несколько, сформируется условие через логическое ИЛИ

[Alexey Gnevyshev]

GROUP BY event_to_country.event_id
Думаю, эта группировка делает так, что к одному событию привязывается одна страна, в подзапросе.
Поэкспериментируйте с этим.

[Alexey Gnevyshev]

lexstile, а таблица связей, скриншот которой в самом вопросе, как называется?

[lexstile]

Алексей Гневышев, называется: event_to_country

[Alexey Gnevyshev]

lexstile, а что если это WHERE countries.id = :country_id
заменить на это:
WHERE event_to_country.country_id = :country_id

[lexstile]

Алексей Гневышев, я так понимаю, что можно просто countries.country_id - так я пробовал - результат тот же.
А если цепляться за таблицу event_to_country в условии, то нужно отказывать от as countries
Или делать еще один JOIN к таблице event_to_country, но почему-то результат тот же...

[Alexey Gnevyshev]

Или делать еще один JOIN к таблице event_to_country

Возможно, да.

Давайте проверим всё-таки, что даёт тот подзапрос.
Выполните и напишите, что выводит:

SELECT countries.*
	FROM (
		SELECT countries.id, event_to_country.event_id, GROUP_CONCAT(countries.name) as countries
		FROM countries
		LEFT JOIN event_to_country ON countries.id = event_to_country.country_id
		GROUP BY event_to_country.event_id
	) as countries
	WHERE countries.id = 2
	LIMIT 30

[lexstile]

Алексей Гневышев, хм, вот:

[Alexey Gnevyshev]

lexstile, вот. Запись Одна. А надо, чтобы было хотя бы 2 )

[Alexey Gnevyshev]

Ещё один Join - этой же таблицы, но без GROUP BY

[lexstile]

Алексей Гневышев, извините, пожалуйста, а JOIN с каким условием сделать?
А то у меня либо одна запись возвращается, либо запрос падает(

[Alexey Gnevyshev]

Попробуйте так, добавьте после первого LEFT JOIN (то есть третьей строчкой)

LEFT JOIN event_to_country ON event_to_country.country_id = countries.id

[lexstile]

Алексей Гневышев,
у меня получился вот такой запрос:

SELECT 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, clusters.clusters FROM events
		LEFT JOIN (SELECT countries.id, event_to_country.event_id, GROUP_CONCAT(countries.name) as countries FROM countries LEFT JOIN event_to_country ON countries.id = event_to_country.country_id GROUP BY event_to_country.event_id) as countries ON events.id = countries.event_id
        LEFT JOIN event_to_country ON event_to_country.country_id = countries.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 (SELECT country_id as c_id, GROUP_CONCAT(name) as clusters FROM clusters GROUP BY c_id) AS clusters ON countries.id = c_id
		WHERE countries.id = 2
		ORDER BY events.year_1 ASC
		LIMIT 30

Но возвращает он две одинаковых записи:

[Alexey Gnevyshev]

А если так добавить:

LEFT JOIN event_to_country etc ON etc.country_id = countries.id AND etc.event_id = events.id

Если будет не то, что надо, объясните, что ожидается получить от запроса - смысл его. Тогда смогу глубже поразмыслить.

[lexstile]

Есть два события, у которых есть совпадение по стране (Россия):

Я хочу, чтобы после добавления страны "Россия", остались оба события(сейчас выводится одно, если бы страна "Россия" стояла первой у второго события, оно бы вывелось тоже):

Если я выберу страну "Франция" или "Индия", или "Гватемала"... - первое событие тоже сейчас не выводится.

Но если я выберу у второго события первой страной Россию - оно выведется.

[Alexey Gnevyshev]

SELECT 
		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, clusters.clusters 
	FROM events
	LEFT JOIN event_to_country etc ON etc.event_id = events.id
    LEFT JOIN (
		SELECT countries.id, event_to_country.event_id, GROUP_CONCAT(countries.name) as countries
		FROM countries
		LEFT JOIN event_to_country ON countries.id = event_to_country.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 (
		SELECT country_id as c_id, GROUP_CONCAT(name) as clusters
		FROM clusters
		GROUP BY c_id
	) AS clusters ON countries.id = c_id
    WHERE
		etc.country_id = :country_id 
    ORDER BY events.year_1 ASC
    LIMIT 30

[lexstile]

Алексей Гневышев, спасибо большое! Дико извиняюсь! Можете не отвечать, на это сообщение, я и так потратил много вашего времени.
вывелись ровно те события, которые и требовались:

но, к сожалению, не работает с множественным условием, например - если брать 2 и более стран через логическое И (то есть, отобрать события, где страна равна России и Италии, например)

[Alexey Gnevyshev]

Рад, что победили )
Наводка: WHERE etc.country_id IN (2, 3) - так точно должно работать
А, это для ИЛИ..

[Alexey Gnevyshev]

WHERE etc.country_id = 2 AND etc.country_id = 3

[lexstile]

Алексей Гневышев,

WHERE etc.country_id = 2 AND etc.country_id = 3

я как раз про это и говорил - возвращает пусто

[Alexey Gnevyshev]

Именно так пробовали, с цифрами? Или с переменными? (в которых может быть и что-то не то)

[lexstile]

Алексей Гневышев, вот запрос:

SELECT 
    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, clusters.clusters 
  FROM events
  LEFT JOIN event_to_country etc ON etc.event_id = events.id
    LEFT JOIN (
    SELECT countries.id, event_to_country.event_id, GROUP_CONCAT(countries.name) as countries
    FROM countries
    LEFT JOIN event_to_country ON countries.id = event_to_country.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 (
    SELECT country_id as c_id, GROUP_CONCAT(name) as clusters
    FROM clusters
    GROUP BY c_id
  ) AS clusters ON countries.id = c_id
   WHERE etc.country_id = 2 AND etc.country_id = 8
    ORDER BY events.year_1 ASC
    LIMIT 30

Вот ответ:

[Alexey Gnevyshev]

А у них точно есть общее событие?
В той таблице, скриншот которой в Вопросе, не видно общего события, с одним event_id.
А, увидел..

[Alexey Gnevyshev]

Судя по всему, ещё один Join и Where

SELECT 
    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, clusters.clusters 
  FROM events
  LEFT JOIN event_to_country etc ON etc.event_id = events.id
LEFT JOIN event_to_country etc2 ON etc2.event_id = events.id
    LEFT JOIN (
    SELECT countries.id, event_to_country.event_id, GROUP_CONCAT(countries.name) as countries
    FROM countries
    LEFT JOIN event_to_country ON countries.id = event_to_country.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 (
    SELECT country_id as c_id, GROUP_CONCAT(name) as clusters
    FROM clusters
    GROUP BY c_id
  ) AS clusters ON countries.id = c_id
   WHERE etc.country_id = 2
   AND etc2.country_id = 8
    ORDER BY events.year_1 ASC
    LIMIT 30

[lexstile]

Алексей Гневышев, работает, я в шоке... БЛАГОДАРЮ!!! Попробую привязать к своей логике.

А если будет три страны? Не будет же еще JOINов?

Но уже написал ответ на предыдущий вопрос, поэтому прикрепляю:

[Alexey Gnevyshev]

А если будет три страны? Не будет же еще JOINов?

Если надо с условием И - будет :)
Сколько пунктов И (стран- участниц одного события), столько и Join-ов + where

[Alexey Gnevyshev]

Добавлю, мне кажется, многие могут не понять логику с И.
Часто в таких интерфейсах, если в поле выбрано несколько значений, то подразумевается логика ИЛИ.
Например, в товаро-учетной программе я могу выбрать в поле несколько нужных контрагентов и буду рассчитывать, что выведутся данные по каждому из них, т.е. с логикой ИЛИ.

Но не исключаю, конечно, что Вам надо именно И - тогда JOIN-ы.

[lexstile]

Алексей Гневышев, спасибо большое еще раз!
Вы правы, пересматриваю все со свежей головой.
Конечно, нужна логика ИЛИ.

[lexstile]

Алексей Гневышев, извините, что снова поднимаю тему.
А вы не подскажете, пожалуйста, если я условие не передаю в виде стран, у меня получаются дубли в выводе, это можно как-то исправить или только отдельным запросом?

[Alexey Gnevyshev]

Здравствуйте, lexstile

если я условие не передаю в виде стран

А при этом убираете LEFT JOIN, который мы добавляли (etc) ?

[lexstile]

Алексей Гневышев, нет, не убираю, запрос один, но условия формируются динамически

[Alexey Gnevyshev]

Попробуйте убрать добавленный нами LEFT JOIN. И потом результат вывода - без указания страны.
Думаю, дублей не будет.
Этот LEFT JOIN надо тоже динамически подставлять.
Что-то типо:
$count - число выбранных стран
if ($count) $left_join = '...';
else $left_join = '';
Это - если условие ИЛИ.

А если И, как Вы хотели ранее, тогда LEFT JOIN надо подставлять для каждой страны, т.е. выводить в цикле for, к примеру.

[lexstile]

Алексей Гневышев, не додумался через условие добавить.
Спасибо в очередной раз!

[lexstile]

Алексей Гневышев, я завел новый вопрос, почему-то появляются дубли, если будет интересно, вот ссылка.