我怎样才能干掉所有这些对 render :index 的调用？

Question

我有一堆带有渲染索引视图的方法的控制器。这导致我在大多数方法的末尾编写 render :index 。下面是一个示例

def index
  @models = Model.find(:all)
end

def new_models
  @models = Model.find_by_new(true)

  render :index
end

def old_models
  @models = Model.find_by_new(false)

  render :index
end

理想情况下，我只需将渲染代码移至后置过滤器中，但由于控制器在进入后置过滤器之前调用渲染，因此这不是一个选项。

我有大量这样的控制器，因此解决方案将删除大量重复的代码。

这个应用程序目前仍然是 Rails 2.3，但是，它将在未来一两个月内升级到 Rails 3。因此，虽然我更喜欢适用于 2.3 的技术，但仅限 Rails 3 的解决方案仍然会受到赞赏。

Answer 1

如果你真的想干掉这个动作，并且它很常见，那么你可以自己做一些元编程。首先使用此模块定义创建一个 render_with_index.rb 文件：

module RenderWithIndex
  def self.included klass
    klass.class_eval do
      def self.render_with_index * methods
        methods.each do |method|
          self.class_eval <<-EVAL
            alias :old_method :#{method}

            def #{method}
              old_method
              render :index
            end
          EVAL
        end
      end
    end
  end
end

然后将该模块包含在控制器中并定义应使用索引渲染的方法（确保在方法声明之后发生 render_with_index 调用。

include RenderWithIndex

def index
  @models = Model.find(:all)
end

def new_models
  @models = Model.find_by_new(true)
end

def old_models
  @models = Model.find_by_new(false)
end

render_with_index :new_models, :old_models

该模块现在允许您使用只需将索引模板添加到 render_with_index 调用即可。

Answer 2

对我来说看起来很干。恕我直言，如果您不想使用控制器方法特定的模板，那么提及您正在渲染哪个模板是一个好习惯。

如果您的渲染代码从一行扩展到多行代码，我会将它们干燥为单独的渲染方法。