質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.50%
Ruby on Rails 5

Ruby on Rails 5は、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

Active Record

Active Recordは、一つのオブジェクトに対しドメインのロジックとストレージの抽象性を結合するデザインパターンです。

Q&A

2回答

1428閲覧

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

nakki

総合スコア9

Ruby on Rails 5

Ruby on Rails 5は、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

Active Record

Active Recordは、一つのオブジェクトに対しドメインのロジックとストレージの抽象性を結合するデザインパターンです。

0グッド

0クリップ

投稿2019/10/29 05:14

編集2022/01/12 10:55

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

ruby

1) accounts.class => Array(Activerecordは使えません) 2 3 4 arys = accounts.map do |acct| 5 a_name= acct&.address 6 {quota: AccountInfo.find_by(name: a_name)&.quota&.hoge} 7 end 8    datas = accounts.zip(arys) 9 10 11 12

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

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

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

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答2

0

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

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

投稿2019/10/29 05:22

maisumakun

総合スコア145121

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

nakki

2019/10/29 05:32

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

0

code を見ると
datas = [[account,{quota: hoge}],[account,{quota: hoge}],,,,]
という結果を得たいと言うことのようですが、それと
「AccountInfo has_many quotaという関係性です。」というのがしっくり来ません。
account と account_info は 1:1 でそれに quota が複数あるとすると、提示されているcode、datas の構造からは2つ目以降の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

とすると早いかも。

投稿2019/10/29 13:33

編集2019/10/29 14:02
winterboum

総合スコア23284

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

まだベストアンサーが選ばれていません

会員登録して回答してみよう

アカウントをお持ちの方は

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問