Rails 元编程/重构/干燥控制器

Question

我有一个有几个动作的控制器。许多人遵循这种模式：

def favorites
  @favorites = Favorite.where(organization_id: @resource.id).page(params[:page]).per(50)
end

不仅仅是收藏夹，还有下载、搜索、列表等，它们都非常相似，以至于我想创建一个可以在 before_filter 中调用的方法。像这样的：

def set_instance_variable
  subject = __method__
  class = __method__.singularize.constantize
  instance_variable = self.class.instance_variable_set("@#{subject}", "#{class}.where(organization_id: @resource.id).page(params[:page]).per(50)")
end

这里的语法可能有点偏离，但我知道这不会起作用，因为__method__ 将始终是 set_instance_variable 而不是调用它的父方法。

有没有办法根据定义实例变量的方法动态设置实例变量？上面的这个例子是否在正确的轨道上？

Answer 1

我喜欢 CanCan 库处理这个问题的方式。使用 CanCan，您可以在控制器顶部调用类方法：

load_resource

CanCan接着看：

• 您正在采取的行动来确定您想要一个集合还是单一资源，

• 确定要加载的类的控制器名称

• 添加范围的授权规则，例如您的 organization_id 限制（cancan 也是用于定义这些范围的库）

我认为分页和资源加载是分开的事情，你不应该把它们放在同一个方法中。我会为这样的界面拍摄：

class FavoritesController
  load_resource
  paginate_resource only: [:index]

  def show
    # @favorite loaded here
  end

  def index
    # @favorites loaded and paginated here
  end
end

https://github.com/CanCanCommunity/cancancan/blob/develop/lib/cancan/controller_resource.rb#L29

如果在您的应用程序中拥有非 RESTful 资源更有意义，那么您就不能重用基于约定的东西 cancan 而是必须定义自己的函数。考虑这样的事情：

def favorites
  @favorites = load_resource Favorite
end


private

def load_resource(klass)
  klass.where(organization_id: @resource.id).page(params[:page]).per(50)
end