Rails has_many关联查询异常：Project与ToDo时间范围查询问题

嘿，我看你在写Project和ToDo的关联查询时遇到了问题，咱们来一步步把它捋顺：

先说说你现有代码的问题 #

你直接把Time.now拼进SQL字符串里，这不仅有SQL注入风险，还可能因为时区不匹配（比如Rails用UTC、服务器是本地时区）导致时间判断出错。另外你的waiting scope没写完，我先帮你补全并修正逻辑。

修正后的active scope #

用Rails推荐的参数化查询+时区安全的Time.current，再结合更优雅的范围查询语法，代码会更安全且易读：

class Project
  has_many :todos

  scope :active, -> {
    joins(:todos)
      # 用Rails范围运算符替代手写SQL条件，可读性拉满
      .where(todos: { starts_at: ...Time.current, ends_at: Time.current... })
      .distinct # 去重，避免一个项目因多个活跃todo重复出现
  }

这里starts_at: ...Time.current等价于starts_at < Time.current，ends_at: Time.current...等价于ends_at > Time.current，完全符合你要找“有任意处于当前时间区间内的ToDo”的项目的需求。

补全并修正waiting scope #

假设你要的waiting是“没有任何活跃ToDo的项目”（包括没有ToDo的项目），可以用两种方式实现：

方式一：左连接+条件过滤 #

scope :waiting, -> {
  left_joins(:todos)
    .where("todos.id IS NULL OR NOT (todos.starts_at < ? AND todos.ends_at > ?)", Time.current, Time.current)
    .distinct
}

方式二：子查询排除活跃项目（更简洁） #

scope :waiting, -> {
  where.not(id: active.select(:id))
}

如果你的waiting是指“ToDo还未开始（starts_at晚于当前时间）”的项目，那逻辑调整为：

scope :waiting, -> {
  joins(:todos)
    .where(todos: { starts_at: Time.current... })
    .distinct
}

额外提醒 #

• 用Time.current代替Time.now：它会遵循你Rails配置的时区，避免跨时区环境下的时间偏差
• distinct必须加：因为一个Project可能关联多个符合条件的ToDo，会导致查询结果重复
• 测试边界场景：比如ToDo的starts_at刚好等于当前时间、ends_at等于当前时间的情况，确保逻辑符合你的预期

这样调整后，你的查询应该就能达到预期效果啦！

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