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

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

ただいまの
回答率

88.06%

副問合せを複数回行う複雑なSQLを、ActiveRecordで書く方法

解決済

回答 2

投稿

  • 評価
  • クリップ 0
  • VIEW 2,410

score 37

お世話になっております。
現在、複数回副問合せを行う複雑なSQLを、ActiveRecordで実現しようとしています。
当初はActiveModelで生SQLを使用していましたが、ActiveRecord依存のGemを使用する
ために、ActiveRecordで書き直しを試みています。

元となる生SQL
SELECT * FROM Model1 query1
  JOIN (SELECT * FROM Model2 query2
    JOIN (
      SELECT * FROM Model1 query3
       WHERE EXIST(SELECT * FROM Model2 query4 WHERE...(たくさん 
       AND EXIST(SELECT * FROM Model2 query5 WHERE...(たくさん))
       )
     )
  )


実際にActiveRecordに書き換えた所、ModelとService層が非常に煩雑になってしまいました。
特に、副問合せ部分と、Modelをまたがっていることが原因で、可読性が落ちてしまいました。

Service
 query5 = Model2.scope().scope().scope().scope().scope().scope...to_sql
 query4 = Model2.scope().scope().scope().scope().scope().scope.where("EXIST( #{query5})")..to_sql
 query3 = Model1.scope().scope().scope().scope().scope().scope.where("EXIST( #{query4})")..to_sql
 query2 = Model2.joins("#{query3}").to_sql
 query1 = Model1.joins("#{query2}").select(column1,column2,,,,)
Model1
scope: 
scope:...20個ほど

Model2
scope: 
scope:...50個ほど


このような、副問合せを含むような複雑なSQLを書く場合、どのようにすれば可読性を維持できるでしょうか?
当方Rails初心者のため、些細なことでもいいので、ご教示いただきたいです。
よろしくお願いいたします。

RubyOnRails 5.2
Postgresql 10

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

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

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

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 2

+3

各query内で過剰に分割しすぎないことだと思います
例えばquery1がscope6個以上繋げていますが、その各scopeは他機能でも利用するのでしょうか? 
意図のある範囲でのscope化までに控えるのはどうでしょう?

class Model2
 scope :query5, (lambda do
   where(***)  # scope1に相当
    .where(***) # scope2に相当
    .where(***) # scope3に相当
    .where(***) # scope4に相当
 end)
end

また上記方法ではscope内が複雑になりすぎる場合は Query object という実装パターンがあります
参考: Rails - ActiveRecord の scope を Query object で実装する

class Query5
  def initialize(relation = Model1.all)
    @relation = relation
  end

  def call
    @relation.where(hoge1: condition1)
                .where(hoge2: condition2)
                .where(***3)
                .where(***4)
  end

  # 複雑になる部分は関数で抜き出す(テストもしやすい)
  def condition1
  end

  def condition2
  end
end

class Model2
  scope :query5, Query5.new
end

あとはこのquery3~5に明確な名前をつけ、個別にその意図を確認するテストをするくらいでしょうか
そうしたら全体の最終SQL構造の見通しが多少落ちても、さほど問題はないかなという認識です

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/08/08 11:27

    回答ありがとうございます。
    このような考え方もあるんですね。勉強になりました。
    今回は元のSQL自体をもう少し簡潔にするように修正したいと思います。
    ありがとうございました。

    キャンセル

checkベストアンサー

+2

そもそも元のSQLが複雑怪奇です。
そこを見直した方が早道ではないでしょうか。

現れるモデルは2つなので、以下のように書き換えられるのではないかと思います。

SELECT * 
FROM Model1  
     inner join Model2
     on Model1.key=Model2.key
where (たくさん)


それが出来ないという事なのかもしれませんが・・・

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/08/01 10:13

    そもそもの案件としては、別言語で書かれた既存システムを、Railsに書き換える様な案件のため、RailsのORMで表現することが難しいという理由で、SQLを変更することは極力避けたいという気持ちです。(動きを担保するという気持ちももちろんあります。)
    しかし、saziさんの仰っているように、元を変更しないことには綺麗に書くことができないかもしれないので、変更することも検討したいと思います。ありがとうございます。

    キャンセル

  • 2018/08/03 13:11 編集

    元を変更しないならそれを一つのモデルとして扱えばどうですか。
    条件を最適化するには普通のモデルでは駄目でしょうけど、一々モデル間のつながりは考えなくて良いわけですし。

    キャンセル

  • 2018/08/08 11:25

    ActiveModelで実装するということになるかと思うのですが、ActiveRecord依存のgemを使用する場合があるため、別のモデルとして作成するのは避けたいです。
    今回は最終的に元のSQLを書き換えるような方向にしました。

    キャンセル

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

  • ただいまの回答率 88.06%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る