Each、Mapの中でActiveRecordを効率的に実行したい（件数文のクエリが実行されてしまう）

Question

以下のようなコードを効率的に実行するにはどうしたらよいかお聞きしたいです。

注) accounts.class => Array(Activerecordは使えません)


        arys = accounts.map do |acct|
            a_name= acct&.address
            {quota: AccountInfo.find_by(name: a_name)&.quota&.hoge}
        end
　　　　datas = accounts.zip(arys)

accountsには1万件のデータがあり、acctブロック変数を使って1万回のfind_byを実行してhoge属性の値を取得しています。

AccountInfo has_many quotaという関係性です。

1万回のクエリが行われるのはとても非効率だと思っていますが、
アドバイスなどお聞きしたく思います。
何卒よろしくお願いします。

Answer 1

自分であれば、以下のような処理を行うと思います。

1. names = accounts.map(&:address).compactとして、AccountInfoの検索条件を作っておく
2. infos = AccountInfo.includes(:quotas).where(name: names)と、AccountInfoを一気に引く
3. あとはEnumerableのメソッドを駆使して集計を行う

Comments

[nakki]

maisumakun　さん
迅速な回答ありがとうございます。
いただいた回答を参考に咀嚼しながら挑戦してみます。
うまくできてから、回答を乗せてみます。

Answer 2

code を見ると
datas = [[account,{quota: hoge}],[account,{quota: hoge}],,,,]
という結果を得たいと言うことのようですが、それと
「AccountInfo has_many quotaという関係性です。」というのがしっくり来ません。
account と　account_info は １：１ でそれに quota が複数あるとすると、提示されているcode、datas の構造からは２つ目以降のquotaが得られません。何がひとつ目になるかはDB次第

それでやりたいことは合って居ますか？

余談ですが
dataは複数形です。単数形は datum。ので、datasはちょっと違和感あります。
####################
で、やりたいことが合っているとして、
DBを引くのはmaisumakunさんの方法で行って、そのあとですが、単純には

arys = accounts.map do |acct|
            a_name= acct&.address
            {quota: infos.find{|info| info.name ==  a_name}&.quota&.hoge}
        end

ですが、これですと accounts.map で1万回、infos.find で平均5000回、計5千万回のloopになります。
遅いかも。
早くなるかどうか未確認ですが
info_by_name = infos.map{|info| [info.name,info&.quota&.hoge]}.to_h
としておいて

arys = accounts.map do |acct|
            {quota: info_by_name[acct&.address]}
        end

とすると早いかも。