【Ruby on Rails】N+1問題の解決

Question

選手名鑑のようなサイトを作成しており、戦績を出力する際にN+1問題が発生しています。
具体的な状況は以下の通りです。

テーブル

- id
- name

- id
- player1_id
- player2_id
- result

Controller

@player = Player.find(params[:player_id])
@games = Game.where(player1_id: @player.id).or(Game.where(player2_id: @player.id))

View

<h2>試合一覧</h2>
<% @games.each do |game| %>
  <div>
　　<%= Player.find(game.player1_id).name %><span>対</span><%= Player.find(game.player2_id).name %>//ここでN+1が発生
    <%= game.result %>
  </div>
<% end %>

補足

• gamesテーブルのデータは、大会が開催されるごとに手動でCSVファイルを作成して、DBに直接インポートしています
• 各プレイヤーがgamesテーブルのplayer1に入るかplayer2に入るかに規則性はありません

この状況でN+1問題を解決し、クエリの数を減らす手段がございましたら、アドバイスいただけると嬉しいです。
どうぞよろしくお願いいたします。

追記

class Player < ApplicationRecord
  has_many :games
end
class Game < ApplicationRecord
  belongs_to :player
end

Comments

[winterboum]

class Gameの関連定義を載せてください。
- id
- player1_id
- player2_id
- result
なのに　 game.player.name　となっているので

[d0ne1s]

コメントありがとうございます!

class Game < ApplicationRecord
belongs_to :player
end

となっています。

[winterboum]

関連定義を載せてください。
- id
- player1_id
- player2_id
- result
とは？　table定義ですか？

[d0ne1s]

わかりづらくすみません。
teble定義です。

[winterboum]

わからん、、、
gameにplayer_id　が無いのに、　belongs_to :player　が成り立ってるのか。
私の理解を超えてる

[mogya]

これたぶん、controllerのところの
```
@games = Game.where(player1_id: @player.id).or(Video.where(player2_id: @player.id))
```
が書き間違いで、gameかvideoかどっちかなんじゃないかな？混ぜてとったりはしないでしょ...

[d0ne1s]

おっしゃる通り、書き違いです。申し訳ございません。
修正させていただきました。
ご指摘いただきありがとうございます。

Answer 1

class Player < ApplicationRecord
  has_many :games
end

class Game < ApplicationRecord
  belongs_to :player1, class_name: 'Player'
  belongs_to :player2, class_name: 'Player'
end

    @player = Player.find(params[:player_id])
    @games = Game.includes(:player1, :player2).where(player1_id: @player.id).or(Game.includes(:player1, :player2).where(player2_id: @player.id))

<% @games.each do |game| %>   
  <%= game.player1.name %>対<%= game.player2.name %>
<% end %>

Answer 2

gameにplayer_id　が無いのに、　belongs_to :player　が成り立ち
<%= game.player.name %> 対 <%= game.player.name %> が動く
という所は私の理解を越えているのですが、
1+N解決としては

@games = Game.includes(:players).
         where(player1_id: @player.id).
         or(Video.where(player2_id: @player.id)

Comments

[d0ne1s]

大変申し訳ございません。
<%= game.player.name %> 対 <%= game.player.name %>の部分の記述が間違っておりました。

　　<%= Player.find(game.player1_id).name %><span>対</span><%= Player.find(game.player2_id).name %>

に修正させていただきました。

ご教授いただいた内容で今から試してみます。
ご回答いただきありがとうございます????????‍♂️

[d0ne1s]

解決しました。
誤った情報をお伝えしてしまい、ご迷惑をおかけいたしました。
ありがとうございました!