0

我有几个动作（我们称它们为 action_a、action_b 等）

在每个操作中，我想检查用户是否已登录，如果没有respond_to如下块

 format.js {
          if current_user.nil?
            render partial: 'some_partial', handler: [:erb], formats: [:js]
          end
        }

对于一个动作来说这很好，但对于很多动作来说就不行了，因为这段代码会有很多重复，它们会做完全相同的事情，它不是很漂亮或可维护

有没有办法把它放在某个地方，这样我就可以重用这段代码，而不是在每个需要的操作中重写它？

4

2 回答 2

2

使用before_filter (rails <4.0) 或 before_action (rails 4.0)

class YourController < ApplicationController
  before_filter :check_user

  ...
  your actions
  ...

  private
    def check_user
      redirect_to sign_in_path if current_user.nil?
    end
end

或者，如果您想要特定操作并做出响应，请使用 around_action（过滤器）：

class YourController < ApplicationController
  around_action :check_user

  ...
  your actions
  def show
    @variable = Variable.last
  end
  ...

  private
    def check_user
      yield #(you normal action without js respond)
      format.js {
        if current_user.nil?
          render partial: 'some_partial', handler: [:erb], formats: [:js]
        end
      }
    end
end
于 2013-07-31T09:09:03.377 回答
2

阅读Responders。这些是为了帮助解决这个问题。

您的具体问题是过滤器通常用于什么。请参阅动作控制器指南中的过滤器部分

更一般地说，这只是 ruby​​ 代码，因此您可以将所有内容重构为方法：

def do_stuff_with(partial_name, format)
  if current_user.nil?
    render partial: partial_name, handler: [:erb], formats: [format]
  end
end


format_js { do_stuff_with('some_partial', :js) }
于 2013-07-31T09:09:06.730 回答