Rails whereで取得した２つのデータを繰り返す

Question

piyo_id をそれぞれリレーションに持つ

Hoge Fuga というクラスがあります。

whereで取得したアクティブレコードをそれぞれ繰り返したいです。

hoges = Hoge.where(piyo_id: 1)
fugas = Fuga.where(piyo_id: 1)

そもそもやりたいこととしては

hoges = Hoge.where(piyo_id: 1)

hoges.each do |hoge|
  fuga = Fuga.find_by(fuga_name: hoge.name)
  ~~~処理~~~
end

上記のように

取得した hoges をもとに hoge というブロック変数を使って
fuga のデータを取得していたのですが
SQLがその都度発行されて N+1 となるので、
whereを使用して、１回で発行しようと考えた次第です。

hoges = Hoge.where(piyo_id: 1)
fugas = Fuga.where(piyo_id: 1)

hoges.each do |hoge|
  ~~~hogeを使った処理を１回実行~~~
  fugas.each do |fuga|
    ~~~fugaを使った処理を１回実行~~~
  end
end

のような処理をしたいです。

ブロックのなかの情報をもとに、別の繰り返し処理をするやり方をご教示頂きたいです。

Comments

[退会済みユーザー]

> ですが、この順番だとうまくいきません。
うまくいかないとはどういうことですか？

[退会済みユーザー]

全体的に間違えてしまったので修正しました

[退会済みユーザー]

> ブロックのなかの情報をもとに、別の繰り返し処理をするやり方をご教示頂きたいです。
結局何がしたいのでしょうか？
「hogeを使った処理」「fugaを使った処理」を具体的に記載してください。
それが記載されないことにはどうすれば良いのかわかりません。
そして現状では何がだめなのかを教えて下さい。

[winterboum]

piyo、hoge,fuga の関係が不明確です。モデルの関連を定義している所を見せて下さい。
とくに、hogeとfugaが1:1なのか？
というのは
「そもそもやりたいこととしては 」で書かれているのは hoge一つにfuga一つの処理ですが「whereを使用して、１回で発行しようと考えた次第です。 」の次に書かれているのはhoge一つ毎にfugaを全部繰り返しています。
hogeひとつにfuga一つ処理をしたいのなら、hogeとfugaの関係がわからないとコメントできないので

[退会済みユーザー]

winterboum 様
Piyo.rb
has_many :hoge, dependent: :destroy
has_many :fuga, dependent: :destroy

Hoge.rb
belongs_to :piyo
Fuga.rb
belongs_to :piyo

ですね

>hogeひとつにfuga一つ処理をしたい
処理としてはそうしたいです

[winterboum]

hoge一つ選んだ時、どのfugaを選んだら良いのでしょう

[退会済みユーザー]

fugaの fuga_nameという属性が、hoge.nameと一致するものを選びたいです。

返答がおかしかったら申し訳ないです。

Answer 1

データベースの変更が許されるのならば
nameではなくidを記述し関連付けしてやった方が高速です。

Comments

[winterboum]

かつ、関連の記述も簡潔見やすくなります。
それがベストに思います。

Answer 2

ということですと脊椎反射的には３つ
1)
Hoge　と　Fuga に各々
has_one :fuga, foreign_key: :fuga_name,primary_key: :name
belongs_to :hoge,foreign_key: :name,primary_key: :fuga_name
して、
hoges = Hoge.includes(:fuga).where(piyo_id: 1)
として、

hoges.each do |hoge|
  ~~~hogeを使った処理を１回実行~~~
  ~~~hoge.fugaを使った処理を１回実行~~~
end

関連付けがどうしても嫌だ、出来ないという事情があるなら

2)hoges = Hoge.where(piyo_id: 1)
fugas = Fuga.where(piyo_id: 1)
のあと
2-1)

hoges.each do |hoge|
  ~~~hogeを使った処理を１回実行~~~
  fuga = fugas.select{|f| f.name == hoge.name}.first
    ~~~fugaを使った処理を１回実行~~~
end

2-2)

fuga_hash = fugas.map{|fuga| [fuga.name,fuga]}.to_h
hoges.each do |hoge|
  ~~~hogeを使った処理を１回実行~~~
  fuga = fuga_hash[hoge.name]
    ~~~fugaを使った処理を１回実行~~~
end

1)がrailsっぽい。
私の好みは １、２−２、２−１の順

Comments

[退会済みユーザー]

詳細にありがとうございます。
勉強になりました。

[退会済みユーザー]

select.firstの所はfindを使った方がスマートな気がします。

[winterboum]

ですね。