Rails6下ActiveRecord查询将AND条件改为OR条件的实现咨询

原有代码问题说明 #

你的现有代码存在两个核心错误：

• 不要遍历已加载的@items记录拼接查询条件，ActiveRecord中多次链式调用where方法时，默认会将多个条件用AND拼接，自然会得到不符合预期的查询结果
• 你的需求是对全量查询做统一条件过滤，不需要针对单条记录做分支判断，直接构造全局的OR查询条件即可

实现代码（Rails 6 适用） #

首先确认Item模型已经定义了和Master的关联，示例：

class Item < ApplicationRecord
  belongs_to :master # 如果关联可选可加 optional: true
end

修改controller中的查询逻辑：

# 统一关联Master表，避免OR查询两边结构不一致报错
@items = @items.left_joins(:master)

# 分别构造两个分支的查询条件
accepted_cond = @items.where(status: 'accepted').merge(Master.where(name: search_params[:name]))
unaccepted_cond = @items.where.not(status: 'accepted').where(name: search_params[:name])

# 用OR拼接两个条件得到最终结果
@items = @items.where(accepted_cond.or(unaccepted_cond))

def search_params
  params.permit(:name)
end

结果验证 #

上述代码生成的WHERE条件符合需求，等价于你期望的逻辑：

WHERE (
  (`items`.`status` = 'accepted' AND `masters`.`name` = 'test')
  OR
  (`items`.`status` != 'accepted' AND `items`.`name` = 'test')
)

如果不需要status的分支判断、只需要两个name字段任意匹配，也可以直接写：

@items = @items.left_joins(:master).where("items.name = ? OR masters.name = ?", search_params[:name], search_params[:name])

更推荐使用ActiveRecord原生的or方法，可避免手动写SQL带来的注入风险。

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