ログインチェックなど共通処理の置き場所

Question

Ruby on Railsチュートリアルを一通り終えた程度の初心者です。

ログインチェックなどの処理は、色んな場所（コントローラ）で共通して使うものだと気付きました。

class UsersController < ApplicationController
  before_action :logged_in_user, only: [:edit, :update]
  before_action :correct_user,   only: [:edit, :update]


  private

    # beforeアクション ログイン済みユーザーかどうか確認
    def logged_in_user
      unless logged_in?
        store_location
        flash[:danger] = "ログインしてください"
        redirect_to login_url
      end
    end
    
    # beforeアクション 正しいユーザーかどうか確認
    def correct_user
      @user = User.find(params[:id])
      redirect_to(root_url) unless @user == current_user
    end
    
end

この場合、基底コントローラクラスである「ApplicationControllerクラス」に置くべきでしょうか？
それともconcernsに置くべきでしょうか？

Answer 1

ログインするページ・していないページが混在するような状況であれば、ログインページ用の基底クラスを作って、ログイン内のページを動かすコントローラーはそこから継承させる、という形がいいかなと思いました。

class LoggedInController < ApplicationController
  before_action :logged_in_user
  before_action :correct_user
  
end

class UsersController < LoggedInController
  # 略
end

Comments

[pecchan]

有難う御座います！
はい、利用規約ページなどはログインしてなくてもOKです。
なので混在してます。

とてもスッキリです、これでいってみます！