sqlの複数発行について

いつもお世話になってます。
今回もどうぞよろしくお願いします。

sqlが複数回発行されているか

以下のコードにおいて、↓のslimの3行を指して上司から「sqlが表示部分と合わせて2回発行されているのではないか。1回で済ませられるようにしましょう」とレビューを受けました。（Railsです）

controller
1@reservations = Reservation.where(date: Date.current)

slim
1- if current_user.admin? || current_user.adviser?
2  - if @reservations.exists?
3    = render "reservations", reservations: @reservations

試した事

exists?を消して、sqlが発行されないようにしてみましたが、それだとif @reservationsが毎回trueになってしまい、望む挙動になりません。
以下のように、pを打って出力を確認した所、---5---と---6---の間には

Reservation Exists? (0.3ms) SELECT 1 AS one FROM "reservations" WHERE "reservations"."date" = $1 LIMIT $2 [["date", "2020-08-06"], ["LIMIT", 1]]
が出力されましたが、

---1---と---2---の間には何も出力されないので、2回発行されていないと自分は考えています。

ruby
1p "---1---"
2@reservations = Reservation.where(date: Date.current)
3p "---2---"
4
5= p "---4---"
6- if current_user.admin? || current_user.adviser?
7= p "---5---"
8- if @reservations.exists?
9= p "---6---"
10= render "reservations", reservations: @reservations
11= p "---7---"

ですが、この確認方法が合っているかは自信がありません。

上司からは「DBからデータを取ってきて、rubyでデータがあるかどうかのチェックし、データを表示すればSQL発行は1回で済む」と追加でアドバイスがありました（原文ママ）。

これの意味をずっと考えていますが、まだ分かりません。
whereはActiveRecord::RelationなのでSQLの発行はしていないと思うのですが、、

どうぞよろしくお願いします。

行動規範の内容に同意します

回答2件

ベストアンサー

mingos さんの言われる通り、log に SELECT が一回しかなければ１回です。
ただし、

---1---と---2---の間には何も出力されないので、2回発行されていないと自分は考えています。

は間違いです。
@reservations = Reservation.where(date: Date.current)
は DBにアクセスに行くことは指示していません。「アクセスするときにはこういう条件で」という ActiveRecord_Relation　を定義しているだけです。
その後の @reservations.exists? や @reservations.count や　@reservations.each などDBにアクセスしないと結果が得られないmethodが来て初めて SELECT　が発行されます。

ですので、載っているcodeの範囲ではでは　5,6の間だけではなく、6,7の間でもSELECTしているのではないか、と思いますが、どうでしょう。

exist?のアクセスは軽いから、有っても良いだろうと思うのですが、、、
でないとすると、
@reservations = @reservations.to_a if @reservations.present?
などとしなくてはならなくなります。

投稿2020/08/06 22:36

winterboum

総合スコア23549

YamauchiSadaka

2020/08/07 07:55 編集

なるほど、ActiveRecord::Relationをarrayにしてしまえばsqlは1回のみ実行ですね、、確かに render "reservations", reservations: @reservations の先でeachが回っています。上司からは以下の2行を特に強調して指摘されたのでスルーしてました。 - if current_user.admin? || current_user.adviser? - if @reservations.exists? ありがとうございます。

行動規範の内容に同意します

「推測するな、計測せよ」という格言があります。
まずは、SQLが実際に何回発行されているかを確認してください。
ターミナルで以下を実行しつつ、
tail -f log/development.log |　grep SELECT
ブラウザで該当ページにアクセスします。
ターミナルでSELECT文が何回実行されているのか分かります。
それでSQLの回数が問題なければ、上司にそれを見せれば良いだけです。

もし2回発行されているなら、それから改善策を考えても良いと思いますよ。

投稿2020/08/06 14:26

mingos

総合スコア4190

YamauchiSadaka

2020/08/06 14:57

コメントありがとうございます。すいません、大事な事を書いていませんでした。以下のようにして確認したんですが、`---5---`と`---6---`の間には `Reservation Exists? (0.3ms) SELECT 1 AS one FROM "reservations" WHERE "reservations"."date" = $1 LIMIT $2 [["date", "2020-08-06"], ["LIMIT", 1]]` が出力されましたが、 `---1---`と`---2---`の間には何も出力されないので、2回発行されていないと自分は考えています。 ```ruby p "---1---" @reservations = Reservation.where(date: Date.current) p "---2---" = p "---4---" - if current_user.admin? || current_user.adviser? = p "---5---" - if @reservations.exists? = p "---6---" = render "reservations", reservations: @reservations = p "---7---" ``` 頂いたコマンドももう一度試して、もう少し調べてみようと思います。