Railsでビジネスロジック（クエリメソッド）を書く場所やそのレシーバについて

Question

例えば以下のような２つの関連Model(テーブル)があるとします
User,Book
でUserとBookは１対多関連してるとして

class User < ApplicationRecord
  has_many :books
end

class Book < ApplicationRecord
  belongs_to :user
end

というようなmodelクラスが生成されているとして。
例えば/users/indexアクションに、およびviewにユーザーを表示したい場合

User.includes(:books).where(id : user_id)

というロジックを書くべきなのは

class User < ApplicationRecord
  has_many :books

　def show_user(user_id)
      self.includes(:books).where(id : user_id)
  end

end

もしくは

class Book < ApplicationRecord
  belongs_to :user

　def show_user(user_id)
      User.includes(:books).where(id : user_id)
  end
end

のどちらに記述するべきなのでしょうか？

Userの一覧を見せたい為、単一責任の原則に照らし合わせるとUserモデルに書くべきなのかなと思いますがいかがでしょうか？

またUserと紐づいているがBookとは関係ないユーザーが持つ何かのステータス一覧を表示したい場合、

Status.includes(:users).where(users: {id: user_id})

のようなメソッドを作成した場合は、Statusモデルに書くべきなのか、Userモデルにメソッドを書けばいいのか分かりません。(Userをincludesのレシーバにするというのは一端無の方向で。。。)

これをStatusに書いてしまうと、/users/indexにステータスと、ユーザーを表示したい場合、

User.new
Status.new

と二つのモデルクラスのインスタンスを作成しなければならず冗長になってしまう気がします。

クラス設計で悩むことがおおいのですがどう解決すればよいでしょうか？
よろしくお願い致します。

Answer 1

どちらに記述するべきなのでしょうか？

1行程度なら、わざわざモデルに持たせるまでもありません。コントローラー直書きでいいでしょう。

Comments

[widget11]

申し訳ございません。例として出した為1行程度のものになりましたがちなみに3行～4行に及ぶ場合はどうすればいいですかね？例えばUserに対して様々な関連モデルをincludesし、且つorderメソッドやwhereメソッドを書いている場合です。

[maisumakun]

> Userに対して様々な関連モデルをincludesし、且つorderメソッドやwhereメソッドを書いている場合

whereの条件が複雑化する場合、モデルにscopeを作るのが第一選択です。

[widget11]

ご回答ありがとうございます！！
scopeは勝手なイメージで何度も使いまわす用なクエリメソッドを定義するものと考えていたのですが、
長くなる場合でもscope作成は有用ということでしょうか？
また作成したscopeはコントローラー側で呼び出すものなのでしょうか？それともmodelやservice層で呼び出しそれをメソッド化してコントローラー側で呼び出すべきなのでしょうか？
いかんせん企業レベルで運用されているrailsのコードをみたことがなく。。。

[maisumakun]

> 長くなる場合でもscope作成は有用ということでしょうか？

はい、1度しか使わなくても、複雑なクエリを組み立てる場合には便利です。

> また作成したscopeはコントローラー側で呼び出すものなのでしょうか？それともmodelやservice層で呼び出しそれをメソッド化してコントローラー側で呼び出すべきなのでしょうか？

それは状況次第です。scope複数を使うscopeを作ることもありえます。

[widget11]

非常に為になりました！
ありがとうございます。