Ruby on Rails按关联表记录数排序查询及Scope冲突问题求助

看起来你遇到的核心问题是字段歧义和查询顺序的问题，我们一步步来解决：

1. 修正for_month Scope，消除字段歧义 #

你之前的Scope里用了survey.created_at，但Rails默认的表名是复数形式surveys，而且当你关联survey_responses后，数据库无法确定created_at属于哪个表（如果survey_responses也有这个字段的话）。另外，只判断月份会导致跨年度的数据被错误筛选，所以我们需要：

• 明确指定surveys表的created_at
• 同时过滤年份，避免选中往年同月份的Survey

推荐两种写法：

写法一：SQL字符串（清晰直观） #

# app/models/survey.rb
scope :for_month, ->(date) {
  where(
    "EXTRACT(MONTH FROM surveys.created_at) = ? AND EXTRACT(YEAR FROM surveys.created_at) = ?",
    date.month,
    date.year
  )
}

写法二：Arel语法（更安全，避免拼写错误） #

如果你想更符合Rails的"优雅"风格，用Arel生成条件，完全避免字符串拼接：

# app/models/survey.rb
scope :for_month, ->(date) {
  where(
    arel_table[:created_at].month.eq(date.month).and(arel_table[:created_at].year.eq(date.year))
  )
}

2. 组合查询：筛选→关联→排序→限制 #

你之前的查询顺序有问题，应该先筛选当月的Survey，再关联响应表，最后排序并限制数量。另外，用left_joins代替joins，这样没有任何响应的Survey也会被保留（只是排在最后），而joins会直接排除无响应的Survey：

@sorted_active_surveys = Survey.for_month(Date.today)
                               .left_joins(:survey_responses)
                               .group(:id) # 用Survey的主键分组，比survey_id更清晰
                               .order(Arel.sql('COUNT(survey_responses.id) DESC')) # 用Arel.sql避免Rails的SQL注入警告
                               .limit(5)

为什么之前会报错？ #

• 表名错误：你用了单数survey.created_at，但实际数据库表名是复数surveys，数据库无法识别这个表
• 字段歧义：当你joins(:survey_responses)后，SQL查询包含两个表，如果survey_responses也有created_at字段，数据库不知道你指的是哪个表的created_at，必须明确指定surveys.created_at
• 查询顺序错误：如果先分组再筛选，会导致不必要的计算，效率更低，而且可能筛选出不符合条件的数据

验证生成的SQL #

最终生成的SQL大概是这样的，完全没有歧义：

SELECT surveys.*, COUNT(survey_responses.id) AS survey_responses_count
FROM surveys
LEFT OUTER JOIN survey_responses ON survey_responses.survey_id = surveys.id
WHERE EXTRACT(MONTH FROM surveys.created_at) = 10 AND EXTRACT(YEAR FROM surveys.created_at) = 2024
GROUP BY surveys.id
ORDER BY COUNT(survey_responses.id) DESC
LIMIT 5

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