Rails-PostgreSQL查询优化：合并三个Scope实现特定排序需求

你需要的是一个能同时满足「先展示库存从高到低的进行中促销，再展示未进行促销」的统一Scope，下面是解决方案：

现有代码的问题 #

• not_progress直接把pluck出的ID拼进SQL，不仅有SQL注入风险，还没法和另外两个Scope结合做整体排序
• 三个独立Scope没法实现「进行中组+未进行组」的分层排序逻辑

合并后的Scope代码 #

scope :ordered_promotions, -> {
  joins(:offers)
    .select('promotions.*, offers.current_inventory')
    .order(
      <<~SQL
        CASE WHEN #{in_progress.to_sql.match(/WHERE (.*)/)[1]} THEN 1 ELSE 0 END DESC,
        offers.current_inventory DESC
      SQL
    )
    .group('promotions.id, offers.current_inventory, offers.created_at')
}

代码逻辑说明 #

1. 关联库存表：用joins(:offers)关联到库存表，确保能获取current_inventory字段
2. 分层排序核心：
   • 用CASE语句复用你已有的in_progress Scope的判断条件，把进行中的促销标记为1，未进行的标记为0，按这个值降序，保证进行中的促销整体排在前面
   • 进行中的促销内部，再按current_inventory从高到低排序
3. 分组去重：保留你原代码中的group字段，避免出现重复数据

注意事项 #

如果in_progress Scope的SQL结构有变化，可能需要调整to_sql.match(/WHERE (.*)/)[1]这部分，确保能正确提取到进行中促销的判断条件。如果担心正则匹配出问题，也可以直接把in_progress里的条件（start_and_end_dates.in_weeks.between_hours对应的SQL）写进CASE语句里，比如：

CASE WHEN start_date <= NOW() AND end_date >= NOW() AND ... THEN 1 ELSE 0 END DESC

这样更稳妥，避免正则匹配的潜在问题。

内容的提问来源于stack exchange，提问作者Roberto da Silva