Исключение определённых ID с условием ИЛИ работает некорректно?

Question

[Vitali]

Предположим что у меня есть база фильмов с жанрами и я хотел бы сделать возможность исключать определённые жанры с условием ИЛИ т.е. если у фильма есть хотя бы один из исключаемых жанров, его следует исключить из выборки. Казалось бы тривиальная задача. Окей. Т.к. мы будем использовать Ruby on Rails, мы ведь не будем писать raw sql запросы, правда ведь? Поэтому для этих целей я нашёл соответствующий Gem и смог казалось бы вполне логично составить запрос, не буду вас утруждать реализацией, если интересно, можете посмотреть мой вопрос на Stackoverflow, где мне пока к сожалению не спешат отвечать.



Скажу лишь что конечный запрос MySQL который формирует active record получается примерно таким

SELECT `movies`.* FROM `movies` 
INNER JOIN `movie_genres` ON `movie_genres`.`movie_id` = `movies`.`id` 
WHERE ((`movie_genres`.`genre_id` != 10 OR `movie_genres`.`genre_id` != 12)) 
GROUP BY `movies`.`id`;

Но не очень рабочий. См. скриншот





Несмотря что запись казалось бы попадает под условие (у фильма есть жанр 10), он всё равно показывается.



В чём может быть проблема и где стоит начать копать?



Спасибо большое.

Answer 1

[edogs]

Одна из тех ситуаций, когда M2M не айс.

Делайте exclude грубо говоря

SELECT `movies`.* FROM `movies` 
INNER JOIN `movie_genres` ON `movie_genres`.`movie_id` = `movies`.`id` 
WHERE movies.id not in 
(
select movies.id from movies left join movie_genres on movies.id=movie_genres.movie_id 
where movie_genres.genre_id in (10,12)
)
GROUP BY `movies`.`id`;

Comments

[Vitali]

Фууух, ещё бы вы мне подсказали как этот запрос на Rails AR переписать, цены бы вам не было) Не хочется городить огород из raw sql запросов. А так, ваш запрос работает корректно, да.

[Vitali]

Всё, разобрался.

Кому интересно, сделал scope в модели

scope :excluding_genres, lambda { |genres| { :conditions => ['movies.id NOT IN (SELECT movies.id FROM movies LEFT JOIN movie_genres on movies.id=movie_genres.movie_id
where movie_genres.genre_id in (?))', genres] }}

Большое вам спасибо, всё работает, хабравой вам кармы и реальной, с праздниками :)

[sl_bug]

scope :excluding_genres, -> (genres) {
  where('movies.id NOT IN (SELECT movies.id FROM movies LEFT JOIN movie_genres on movies.id=movie_genres.movie_id where movie_genres.genre_id in (?))', genres)
}

чтобы не deprecated фичу использовать (conditions)

Answer 2

[nochkin]

А какой вопрос? Если интересно что не так с запросом на RoR, то было бы полезно его увидеть.
Сам sql запрос смотрится слишком странно, так как genre_id по-любому не сможет быть равен 10 и 12 одновременно. Если я правильно понял задачу (тут она тоже не описана), то надо делать AND вместо OR. То есть, если фильм не имеет жанр 10 и не имеет жанр 12.

Comments

[Vitali]

Как раз таки задача в том что не показывать фильм если у него нет жанра 10 ИЛИ жанра 12. По сути, не показывать все фильмы которые имеют хотя бы один из этих жанров. По поводу того что задача не описана как следует, моё упущение, сейчас дополню вопрос.

[Toseter]

Все правильно вам говорят, условие долно быть, Жанр != 10 и Жанр != 12, для Жанр = 9, оба эти условия выполняются и общий результат «истина», для 10 первое не выполняется, и результат «ложь». В случае с Or `movie_genres`.`genre_id` != 10 OR `movie_genres`.`genre_id` != 12 вернет при любых значениях «истина».

Answer 3

[Aleks_ja]

SELECT `movies`.* FROM `movies` 
INNER JOIN `movie_genres` ON `movie_genres`.`movie_id` = `movies`.`id` 
WHERE ((`movie_genres`.`genre_id` != 10 AND `movie_genres`.`genre_id` != 12)) 
GROUP BY `movies`.`id`;

Вот так

Comments

[Vitali]

Неа, это условие будет работать только если у фильма будeт все эти жанры, а мне нужно что если нет хотя бы одного жанра.

[Aleks_ja]

Конечно, сейчас поздно и я могу ошибаться, но, походу, всё просто — где жанр не равен такому-то, а также не равен такому-то.

А в изначальном запросе — он либо не равен такому-то либо не равен такому-то. Т.е. у одного жанра одна айдишка, так что он по-любахе не будет одновременно и 10 и 12ым. Т.е. могли бы и не писать where совсем.

[Vitali]

Звучит логично, тут чуть ниже edogs написал корректный запрос на mysql, теперь надо придумать как его на activerecord перенести.

[Aleks_ja]

Попробуйте, а вдруг и мой сработает :)

[Vitali]

Уж пробовал) Этот запрос работает но делает немного не то что мне было нужно, он срабатывает только при условии если у фильма есть все жанры, а мне нужно если подходит хотя бы один) В в любом случае, и вам спасибо, и вам кармы)