Ruby on Rails：在一个动作中是否有多个 respond_to 块好：最佳实践

Question

我需要做条件检查是否渲染这个或那个。 用respond_to 块包装条件检查是否很好？ 喜欢

   respond_to do |format|
     if @applicant.user_id.nil?
       # Some logic
       format.js
       format.html { redirect_to applicant_path(params[:id]) }
     else
       format.json { render json: {message: t('_applicants.flash.applicant_already_assigned')}, status: 400 }
     end
   end

或

我将在 if..else 内有多个 respond_to 块 喜欢

if @applicant.user_id.nil?
  # Some logic
  respond_to do |format|
    format.js
    format.html { redirect_to applicant_path(params[:id]) }
  end
else
  respond_to do |format|
    format.json { render json: {message: t('_applicants.flash.applicant_already_assigned')}, status: 400 }
  end
end

推荐的最佳方法是什么？

Answer 1

在我看来，控制器操作的代码很多。 Rails 在构建时考虑了DRY 方法，因此您可以尝试以下方法：

如果你这样说：

class SomeController < ApplicationController
  respond_to :html, :json , :js

在你的控制器的开头，你可以让你的动作更苗条：

def someaction
 if aplicant.id.nil?
  redirect_to applicant_path(params[:id])
 end
 else
    flash[:notice] = "Applicant already assigned" 
 end
end 

Read more about rails responses in this article.

直接回答您的问题： 使用单个 respond_to 块的第一个选项更合理，因为它涉及编写更少的代码。正如我之前提到的，Rails 就是为了避免重复，而条件运算符中的两个响应块与编写 Rails 应用程序的风格背道而驰。