Как правильно составить запрос?

Question

[bismoney]

Привет друзья, каша в голове как это объединить сократить?

Выборка для одного блока
@posts_main = Post.where('...').limit(1)

Дальше мне надо выбрать 15 последних записей и исключить post_main

Post.where('id != ?', @posts_hot.map(&:id)).limit(15).order('created_at DESC')

Вот тут начинаются вопросы, тут мы выбираем 15 записей, но мне надо выводить не всегда 15

1) К примеру мне надо проверить эти 15 записей на quote IS NOT NULL и посчитать сколько их из 15.

Post.joins(:post_asset).where('quote IS NOT NULL').limit(15).order('created_at DESC').count

2) А так-же мне надо посчитать в этих же 15 записях сколько с post_block_id = 2

Post.where('post_block_id = 2').limit(15).order('created_at DESC').count

Так же эти две выборки надо делать исключив 'id != ?', @posts_hot.map(&:id)) новость из блока post_main
Теперь мне надо значения количества 1 и 2 вычесть из 15
То есть там где мы в самом начале делали выборку, вместо 15 надо выводить 15 минус числа тех двух запросов.

К примеру

test1 = Post.where('post_block_id = 2').limit(15).order('created_at DESC').count
test2 = Post.joins(:post_asset).where('quote IS NOT NULL').limit(15).order('created_at DESC').count
test = 15 - test1 - test2

Post.where('id != ?', @posts_hot.map(&:id)).limit(<b>test</b>).order('created_at DESC')

Как все это должно выглядеть правильно?

Answer 1

[Андрей Сидоров]

либо фильтровать сразу в запросе

Post
  .where.not(id: @posts_hot.map(&:id))
  .where.not(id: Post.where(post_block_id: 2)) # magic number anipattern. лучше заменить на константу
  .where.not(id: Post.joins(:post_asset).where.not(post_asset: { quote: nil }))
  .order(created_at: :desc)
  .limit(15)

либо отфильтровать после запроса в коде

Post
  .includes(:post_asset)
  .where.not(id: @posts_hot.map(&:id))
  .order(created_at: :desc)
  .limit(15)
  .select { |post| post.post_block_id != 2 }
  .select { |post| post.post_asset.try :quote }

Comments

[bismoney]

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

Единственно вот так выдавало ошибку
.where.not(id: Post.joins(:post_asset).where.not(post_asset: { quote: nil }))
ERROR: missing FROM-clause entry for table "post_asset"
Вот так заработало
.where.not(id: Post.joins(:post_asset).where('quote IS NOT NULL'))

Чуть позже почитаю что за ошибка, сделаю как у вас.

И если можно еще вопрос, что все таки лучше фильтровать сразу или после в коде?
Как лучше по производительности чтоли? Мне кажется выгоднее последнее?

[bismoney]

Поторопился чуток) сейчас попробую переделать, логика не та не много, .limit(15) нужен не фиксированный. То есть если в выбранных 15 есть 1 с post_block_id 2 то надо выводить на 1 меньше. А в данному случае если Post_block_id 2 то он его просто фильтрует, тоже самое с quote. А надо просто что бы к примеру есть в эти 15, 3 с Post_block_id 2 и 2 c quote не nil, то выводим не 15 а 10

[bismoney]

Как никрути у меня получает два запроса) Первый который считает сколько в этих 15, block_post_id и quote и во втором запросе уже используется лимит изходя из первого. В одно никак не получается.

[Андрей Сидоров]

> Единственно вот так выдавало ошибку
да, post_asset надо заменить на post_assets в where. там пишется не имя ассоциации, а имя таблицы
.where.not(id: Post.joins(:post_asset).where.not(post_assets: { quote: nil }))

> И если можно еще вопрос, что все таки лучше фильтровать сразу или после в коде?
> Как лучше по производительности чтоли? Мне кажется выгоднее последнее?

всё зависит от объёма данных. когда нужно всего 15 записей, и затем от них отрезать лишнее, то в руби отфильтровать и проще, и код понятнее. если бы, например, нужно было отфильтровать тысячи записей, и только затем выбрать 15, то это нужно было бы делать в базе.

[Андрей Сидоров]

> Поторопился чуток) сейчас попробую переделать, логика не та не много, .limit(15) нужен не фиксированный. То есть если в выбранных 15 есть 1 с post_block_id 2 то надо выводить на 1 меньше. А в данному случае если Post_block_id 2 то он его просто фильтрует, тоже самое с quote. А надо просто что бы к примеру есть в эти 15, 3 с Post_block_id 2 и 2 c quote не nil, то выводим не 15 а 10

тут как раз второй вариант подходит с фильтрацией в коде. сначала выбираем 15 постов, затем их фильтруем в руби. два варианта я написал, чтобы показать, как вообще можно делать.

[Андрей Сидоров]

> а как во втором случае оперировать .limit(15)?
не понял вопроса. отобразить ведь нужно 15 постов за вычетом постов, не подходящих по каким-то критериям?
соответственно выбираем 15 постов и их фильтруем
posts = Post.where(критерии_выборки).limit(15).select { |post| условия_фильтрации }

возможно, вас смущает метод select, т.к. select из Enumerable и select из ActiveRecord имеют одинаковое название? написанный мной select - это не select активрекорда, это фильтрация массива.

можно ещё вот так написать, код будет эквивалентен
posts = Post.where(критерии_выборки).limit(15).to_a
posts = posts.select { |post| условия_фильтрации }

[bismoney]

morr: В общем не суть разберусь) Вы и так помогли, спасибо огромное