Ruby on Railsのメソッドの引数についての初歩的な疑問

Question

メソッドの引数についての初歩的な疑問

Ruby及びRuby on Rails初心者です。
メソッドの引数について疑問があります。

HogeController以外でhoge1,hoge2を実行する場合も、必ず@userをセットするものとします。
以下のhoge1, hoge2はともに同じ結果になると思います。

しかし、やはり可読性や安全性等からhoge1のように@userを引数として渡すべきなのでしょうか。
なかなか答えにたどり着けず悩んでおります。
どうぞよろしくお願いいたします。

class HogeController < ActionController::Base
 @user = User.find(params[:id])

 hoge1(name: "太郎", user: @user)

 hoge2(name: "太郎")

end

class ApplicationController < ActionController::Base

 def hoge1(name: nil, user:)
  name.nil? ? user.name = "名無し" : user.name = name
 end


 def hoge2(name: nil)
  name.nil? ? @user.name = "名無し" : @user.name = name
 end

end

Answer 1

そもそも論ですが、特定のコントローラーから供給される@userのような変数に依存するメソッドをApplicationControllerに持たせること自体が、おすすめできません。

このようなものはmoduleあるいはConcernsとして定義して、必要な場合にincludeして使う、という方がいいと思います。

Comments

[Kaz_M]

ご回答有り難うございます。
このような場合はConsernsに切り出そうと思います。

その場合でもやはり@userを渡すべきでしょうか。重ね重ねすみません。