如何在Rails视图中检查Campaign的所有Design是否已验证？

首先，判断「所有Design都处于is_validated状态」的核心是确认该Campaign下不存在未验证的Design，而非仅检查已验证的Design是否存在。结合你的代码场景，常见问题点及修复方式如下：

1. 视图判断逻辑错误（最常见原因） #

如果你的视图代码类似这样：

<% if @campaign.designs.validated.present? %>
  <%= icon 'check' %>
<% end %>

这是错误的——它仅判断「存在已验证的Design」，而非「所有Design都已验证」。正确逻辑应聚焦于无未验证的Design，或直接验证每个Design的状态：

修复方式（视图层直接判断） #

<% if @campaign.designs.all? { |d| d.is_validated } %>
  <%= icon 'check' %>
<% end %>

更高效的数据库查询方式（避免加载所有Design实例）：

<% if !@campaign.designs.where(is_validated: false).exists? %>
  <%= icon 'check' %>
<% end %>

2. 模型层封装优化（推荐） #

直接在视图写查询逻辑不够优雅且易重复，建议在Campaign模型中封装判断方法：

# app/models/campaign.rb
def all_designs_validated?
  designs.where(is_validated: false).exists? == false
end

若需处理「无Design的Campaign是否视为全部验证」的场景，可调整：

def all_designs_validated?
  designs.exists? ? designs.where(is_validated: false).exists? == false : true
end

之后视图直接调用即可：

<% if @campaign.all_designs_validated? %>
  <%= icon 'check' %>
<% end %>

3. 作用域的代码风格优化 #

你的validated作用域写法：

scope :validated, -> { where('is_validated >= ?', true) }

若is_validated是布尔类型字段，更规范的写法是：

scope :validated, -> { where(is_validated: true) }

这不是判断失效的直接原因，仅为代码风格优化。

4. N+1查询问题规避 #

若在视图中使用all?(&:is_validated)，会触发N+1查询（先查Campaign，再查所有Design）。可在控制器中预加载Designs：

# app/controllers/campaigns_controller.rb
def show
  @campaign = Campaign.includes(:designs).find(params[:id])
end

或使用模型层的数据库查询方法（如前面的all_designs_validated?），直接通过数据库判断，无需加载所有实例。

内容的提问来源于stack exchange，提问作者user3074558