railsでinitializeを使うメリットについて

Question

railsでインスタンスメソッドでクラスを呼び出す際にinitializeを使うメリットってありますか？
例えば下記ののようにSessionsControllerクラスでAuthenticatorクラスのauthenticateメソッドを呼び出すのですがその時に引数をメソッドではなくクラスに設定するにするメリットはありますか？

class Staff::Authenticator
    def initialize(staff_member)
        @staff_member = staff_member
    end

    def authenticate(raw_password)
        @staff_member &&
         !@staff_member.suspended? &&
         @staff_member.hashed_password &&
         @staff_member.start_date <= Date.today &&
         (@staff_member.end_date.nil? || @staff_member.end_date > Date.today) &&
         BCrypt::Password.new(@staff_member.hashed_password) == raw_password
    end
end

class Staff::SessionsController < Staff::Base
    def create
        @form = Staff::LoginForm.new(params[:staff_login_form])
        
        if @form.email.present?
            staff_member = StaffMember.find_by("LOWER(email) = ?", @form.email.downcase)
        end
        if Staff::Authenticator.new(staff_member).authenticate(@form.password)
            session[:staff_member_id] = staff_member.id
            redirect_to :staff_root
        else
            render action: "new"
        end
    end
end

ではなく

class Staff::Authenticator
    def authenticate(staff_member, raw_password)
        staff_member &&
         !staff_member.suspended? &&
         staff_member.hashed_password &&
         staff_member.start_date <= Date.today &&
         (staff_member.end_date.nil? || staff_member.end_date > Date.today) &&
         BCrypt::Password.new(staff_member.hashed_password) == raw_password
    end
end

class Staff::SessionsController < Staff::Base
    def create
        @form = Staff::LoginForm.new(params[:staff_login_form])
        
        if @form.email.present?
            staff_member = StaffMember.find_by("LOWER(email) = ?", @form.email.downcase)
        end
        if Staff::Authenticator.new.authenticate(staff_member,@form.password)
            session[:staff_member_id] = staff_member.id
            redirect_to :staff_root
        else
            render action: "new"
        end
    end
end

でも動作は変わらないと思うのですがどう違いますか？

Answer 1

メリット・デメリットというよりは、どのように使いたいのかというところがポイントになると思います。

例えば、このように使いたいと考えたら

Staff::Authenticator.authenticate(staff_member, raw_password)

こうなりますし、

class Staff::Authenticator

  def self.authenticate(staff_member, raw_password)
    ...
  end

end

インスタンスを作ってから使いたいなら最初のコードのようになります。

authenticator = Staff::Authenticator.new(staff_member)
authenticator.authenticate(raw_password)


class Staff::Authenticator

  def initialize(staff_member)
    @staff_member = staff_member
  end

  def authenticate(raw_password)
    ...
  end

end

どっちを使っても間違いではありません。

Answer 2

1つのメソッドに処理を全部書いてしまうのであれば、インスタンス変数である意味はあまりありません。

一方で、メソッドを分割するとなった場合に、インスタンス変数があれば、それは共通して使えるので、メソッドごとに引き回す必要がなく、きれいに書けます。