Railsで「ユーザーの保存しているレストランを検索する」ようなメソッドはどのモデルに記述すべきか

Question

Ruby on Railsで飲食店の情報を管理するようなサービスを作っています。

話に関係するモデルは、以下の写真のように、User, Restaurant, Clip, Station, Category, Clip_category, Boardと言ったところです。

この中で、保存したレストランを検索するメソッドを作っています。

controllerで

station = Station.search params[:query]
board = Board.search params[:query]

だいたい上のように各モデルへの操作を書いています。

ここで質問が2つあるのですが、

1. current_userのクリップ(保存したレストラン)一覧を取得するようなモデルを跨ぐようなメソッドをモデル層に記述したいときはuser.rbやclip.rb、restaurant.rbなど、どのモデルに、また、クラスメソッド、もしくはインスタンスメソッドのどちらで作るべきなのでしょうか？

現状は、
Clip.search(params[:query], current_user.id)のようにクリップクラスからメソッドを生やすか
current_user.search_clips(params[:query])のようにユーザーのインスタンスメソッドを作るか
の2つで迷い、後者を採用しています。

1. 今は、あるモデルについて検索するメソッドをモデルに書き、それをcontrollerで使っていますが、そもそもClip.get_with_queryのようなメソッドを作り、現在controllerで行っているような処理を全てmodelで行ってしまった方がいいのでしょうか？

controllerとmodelをどう使い分けるべきかがよくつかめていません。

以上2点です。
よろしくお願いいたします。

Answer 1

まず、Railsガイドの「Active Record の関連付け (アソシエーション)」を読んでみてください。
https://railsguides.jp/association_basics.html

User、Clip、Restaurantの関係は、ここで出てくる「2.4 has_many :through関連付け」のPhysician、Appointment、Patientに相当すると思います。

そのため、

class User < ApplicationRecord
  has_many :clips
  has_many :restaurants, through: :clips
end
 
class Clip < ApplicationRecord
  belongs_to :users
  belongs_to :restaurants
end

といったようになっていれば、current_user.restaurantsでクリップしたレストランが取得できるかと思います。

2つ目の質問ですが、モデルを検索するメソッドはモデルにあった方がよいですが、コントローラーで使いたい処理はまずはコントローラーに置いてもよいでしょう。ただ、他の箇所からも同様の処理を行いたくなった場合は、モデルのメソッドにリファクタリングした方がよいです。
最初から一発で正解を求めるのは難しいので、多少の回り道は覚悟した上で、少しずつアプリを育てていくつもりで書いていく方が現実的かもしれません。

Comments

[tomohiroo]

回答ありがとうございます！！
確かに、徐々にアプリを育てて行く意識は大切かもしれません、ご指摘ありがとうございます。
1つ目の質問についてなのですが、現在、リレーションはちゃんと敷いており、current_user.restaurantsでもuserがクリップしたレストランを全件取得できるのですが、全件取得ではなく、queryや、ジャンル(「ラーメン」、「イタリアン」など)など色々な条件で絞り込むようなメソッドについては、どのモデルに実装すべきか、ということで悩んでおります。

[takahashim]

なるほど、アソシエーションは設定されてるんですね。
件数が多くなければ全件取得してから絞り込みを行ってもいいかと思いますが、もう少しDBへのクエリに処理を寄せたいのであれば、「そのメソッドが返して欲しいオブジェクトのクラス」に置く、というのが分かりやすいかもしれません。
例えば「current_userの、何かの条件で絞り込んだクリップ(保存したレストラン)一覧を取得する」といった場合、返してほしいのはクリップではなくレストラン（の一覧）だと思うので、Restaurantのメソッドとして実装することにして、引数にユーザや絞り込み条件を指定するとスッキリするかと思います。