【Ruby on Rails】クエリの数を減らしたい

Question

Ruby on Railsでアプリを作成しています。
ユーザーに関連する情報を取得する際、クエリの数を減らしたいと思っています。
何か良い方法がございましたら、ご教授いただけますと幸いです。

現在のコード

@user = User.find(params[:id})
@comments = Comment.where(user_id: @user.id)
@posts = Post.where(user_id: @user.id)
@likes = Like.where(user_id: @user.id)
@songs = Song.where(user_id: @user.id)

よろしくお願いいたします。

Answer 1

この場合、無理に減らさなくていいかと思います。

もちろんJOINしてクエリ回数を減らすことはできますが、そうしたところでアクセスすべきテーブルの数は変わらないので、そこまで速度に大差は出ません。

Comments

[maisumakun]

それより、ビューで使うときにうっかりN+1クエリを発生させていないかのほうが問題です。

[d0ne1s]

ありがとうございます。
クエリでのパフォーマンス改善は難しいですか。
実は、comment, post, like, song以外にもアクセスすべきテーブルが7個くらいあって、クエリの投げすぎでパフォーマンスが悪く、なんとかしなきゃ、という状況です。

根本的にアプリの構造の問題としか言えないですかね。
ちなみにページはこちらになります。
いろんな大会の結果を取得する必要があって、テーブル数が多くなっています。
https://mcbattle-ch.jp/mcs/detail?mc_id=4

[d0ne1s]

N+1は一つ存在していて、今から改善するところです。

[winterboum]

＞いろんな大会の結果を取得する必要があって、テーブル数が多くなっています。
大会ごとにtableを作るのではなく、一つのtableで管理できませんか？
記録内容が全くバラバラだと難しいかも、ですが

[d0ne1s]

アドバイスありがとうございます。
確かにその手もありますね。
現状大会によって必要な情報が違う+今後も大会が増えていく予定
なので、ちょっと簡単では無い気がします。

かなり頑張ればできないことは無いので、必要になったらその方法も検討しようと思います。
参考になりました。

[winterboum]

情報がtextなら、大会に応じてlabelを変えれば済むだけ　という手もあります

[d0ne1s]

確かにおっしゃる通りですね。
ただ今回の場合は大会ごとの結果まとめページがあったり、結構いろんな場所でいろんな使われ方をしているので、難しいかと思います。

ありがとうございます。

Answer 2

各テーブルのuser_idにインデックスをつけて、1つ1つのクエリの時間を短くする方法があります。
あとは、アクセス中のuser_idの情報はキャッシュで保持しておくとか。
結合のコストがあるので、単一表を絞り込むだけの方が速いと思います。

Comments

[d0ne1s]

ありがとうございます。
確認したところ、全てのテーブルでuser_idにはインデックスが貼られていました。
キャッシュに関しても検討させていただきます。

いろいろな観点のご意見をいただくことができ、とても参考になります!