3

考虑这段代码

module Auth

  def sign_in(user)
    #some stuff
    session[:user_id] = user.id
  end

end

现在，我想将它包含在我的应用程序控制器中。

ApplicationController < ActionController::Base
  include Auth
end

这使得该sign_in方法在我所有的控制器中都可用。现在，为了明确这不是控制器动作，我想保留名称，所以我的控制器读取

def sign_user_in
  # Some stuff
  Auth.sign_in(@user)
end

这显然是行不通的，因为 Rails 会在 Auth 模块中寻找类方法。所以问题是......是否可以在控制器中包含一个模块，保留它的名称或命名空间，但仍然可以访问与控制器相同的范围？（在这种情况下，会话变量）。

到目前为止，我想出的最不坏的方法是停止在 ApplicationController 中包含模块，而是在调用这样的 auth 方法时传递应用程序控制器实例：

def current_user(controller)
  User.find(controller.session[:user_id])
end

使用 usingself作为参数从控制器调用此方法有效。

4

2 回答 2

2

试试这个？

对所有功能使用实际类，并且控制器具有该类的可用实例；基本上与您上面的代码完全相同 - 请参阅current_user但您只需要一次传递控制器实例，而不是在每个方法调用上

module Auth
  # this method is 'mixed' into controller (self)
  def initialize_authorizer
    @authorizer = ::Auth::Authorizer(self)
  end

  # this class doesn't need to be in this namespace (module), put it where ever makes sense
  class Authorizer
    def initialize(controller)
      @controller = controller
    end

    attr_reader :controller

    def sign_in(user)
      #some stuff
      controller.session[:user_id] = user.id
    end

    def current_user
      User.find(controller.session[:user_id])
    end        
  end
end

ApplicationController < ActionController::Base
  include Auth

  before_filter :initialize_authorizer
end

def sign_user_in
  # Some stuff
  @authorizer.sign_in(@user)
end
于 2013-05-12T14:47:16.463 回答
0

我问这个问题已经9年了。与此同时，我意识到这样做不是一个好主意，因为它会与语言发生冲突。

常量有自己的self，当引用一个常量时，你会期望任何方法都是类方法。除非在方法调用期间显式传递了引用，否则您不会期望它们能够访问调用对象，在这种情况下，您将具有双向依赖关系，这会带来一系列问题。那将是代码异味，应该是重构软件设计的原因。

于 2022-02-25T13:53:24.830 回答