親メソッドを呼ぶ時のアクションが異なる場合

Question

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

ログイン確認を行うページが複数あるため、その部分は共通化しました。
・ログイン確認処理は、基底クラスに持たせる
・各継承先で処理を呼び出す

この時、継承先ごとで呼びたいアクションが違うのです。
設定ページはindexアクションから呼びたい。
Userページは、edit、updateアクションから呼びたい。

なので下のような記述になりましたが、これで良いのでしょうか？
※一応動いてます

「railsでは通常そんな書き方はしない」などあったら教えていただきたいです。

初心者につき自身がありません。。。

親となる基底クラス

class LoginController < ApplicationController
  

  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

継承先Aは、indexアクションから呼びたい

class SettingsController < LoginController
  before_action :logged_in_user, only: [:index]
  before_action :correct_user,   only: [:index]
  
  def index
    @user = User.find(params[:id])
  end
end

継承先Bは、edit, updateアクションから呼びたい

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

Answer 1

良いと思います。
というか
それでだめか、などと考えたこともなかった　というレベルですが。