在控制器中重构多个渲染

Question

在我的 rails 控制器中，我必须在获得 @group 和 before_action 后检查该组不是 system。

但是我的控制器中有很多重复。我试图变成一个单独的方法，但我得到了经典：

Render and/or redirect were called multiple times in this action. Please note that you may only call render OR redirect, and at most once per action. Also note that neither redirect nor render terminate execution of the action, so if you want to exit an action after redirecting, you need to do something like "redirect_to(...) and return".

这是我的代码的一部分，没有给我错误的单独方法。

def destroy
  if @group.is_system?
    render json: { errors: 'You can\'t delete a group system' }, status: 403
    return
  end

  ...
end

def update
  if params[:group] && !params[:group].empty?
    if @group.is_system?
      render json: { errors: 'You can\'t edit a group system' }, status: 403
      return
    end

    ...

  else
    render json: { errors: 'Missing correct parameters' }, status: :unprocessable_entity
  end
end

.....

Answer 1

您可以在父控制器中：

def render_errors(errors, status)
  render json: { errors: Array(errors) }, status: status
end

def render_403(errors)
  render_errors(errors, 403)
end

def render_422(errors)
  render_errors(errors, 422)
end

然后在你的行动中：

before_action :check_system

def check_system      
  # I assume you already defined @group
  render_403('You can\'t delete a group system') if @group.is_system?
end

请注意，我更改了您的一些代码：只有一个字符串的 errors 键非常具有误导性，应该是一个数组。