rails active record 子要素の条件にマッチする親レコードを取り出したい

Question

＜状況＞

・ユーザー（user）と本(book)があり、１対多の関係

・bookはread(boolean)カラムを持っている

＜やりたいこと＞

・子（book）要素内の条件(read: false)を一つも持っていない親(user)レコードを取り出したい

＜目的＞
ユーザーがたくさんの本を持っている状態で、本を持っているが、一つも読んでいないユーザーを見つけたいです。一つでも読んでいたら除外したいです。

＜やったこと＞
joinなどで
・読んでいない本を持っているユーザー
・読んだ本を持っているユーザー

left_outer_joinで
・本を持っていないユーザー

↑これらはできそうですが、

・読んでいない（read: false）本しか持っていないユーザーの取り出し方が分かりません。。。
よろしくお願いいたします。

Answer 1

タグにSQLを追加して達人を召喚することをお薦めします。
GROUP BY を使ってやればきれいなのができるのかな、とは思いますが私にはできそうにないので、とりあえず「これならでるかな」というのを
・読んでいない本を持っているユーザー から ・読んだ本を持っているユーザー を除いてください
User.distinct.joins(:books).where(books: {read: false}) - User.distinct.joins(:books).where.(books: {read: true})
美しくない、２回アクセスするし。
これですと paging できないです。
paging が必要なくらいおおくなりそうでしたら、読んでない ids と 読んだ ids を引いた ids でもう一度アクセスすることになってしまうです。

なお
User.distinct.joins(:books).where(books: {read: false}).where.not(books: {read: true})
はだめです。
where(books: {read: false}) は必ず where.not(books: {read: true}) ですから。

生SQLで書けば MINUS とか EXCEPT で差集合が求められるかも、ですが DBエンジンによって有ったりなかったりVersion依存したりするかも。

Comments

[toki1234]

ありがとうございます！
SQLで書いた方がいいのですかね。。。

『where(books: {read: false}) は必ず where.not(books: {read: true}) ですから。』
そうですよね。ここで期待した結果にならずで悩んでました。

[winterboum]

SQLにするか、は ここの重さ次第、重さが我慢できるか 次第 ですね。

重さが気になる頃はページングが必要になるかも、もありますからもしそうなると 今２回のDBアクセスが３回になって輪を書けてしまう。
SQLならうまく行くと１回のアクセス。
かなぁ、、、
Railsからは１回でもDBの方はやはり２，３回分の負荷になる作業かも。

ただ
このSQL作るにはかなりの技術力必要かと。そういう人に出会ったら「かなぁ。。。」のところも含めてご相談を。

Answer 2

「子（book）要素内の条件(read: false)」に当てはまるuser_idを抽出して、それをwhere.notで適用してはどうでしょうか。

User.where.not(
  id: Book.where(read: false).select(:user_id)
)

Comments

[winterboum]

これですと、全部読んだか、一冊も持っていない人 が選ばれるのでは

[maisumakun]

「子（book）要素内の条件(read: false)を一つも持っていない親(user)レコードを取り出したい」という条件にそって実装したのですが、この条件そのものが誤っていたようですね。

[toki1234]

ありがとうございます。
私の書き方が不十分でしたが、
条件でないもののIDを取り出して、where.notでそれ以外を取り出す方法でうまくいきました！