如何利用ActiveRecord关联查询归属指定城市的Listing记录

让我来一步步帮你解决这两个ActiveRecord查询的问题：

这个问题得根据你的模型关联关系分情况处理，给你列几种常见场景：

• 如果你的模型属于单个关联对象（比如 Listing belongs_to :suburb，即一条Listing对应一个郊区），直接通过外键字段查询就好：

  # 示例：找suburb_id为123的所有Listing
  Listing.where(suburb_id: 123)
  

• 如果你的模型拥有多个关联对象（比如 Suburb has_many :listings，一个郊区对应多条Listing），可以先定位到目标关联对象，再获取它的关联记录：

  # 先找到id为123的郊区，再拿它的所有Listing
  target_suburb = Suburb.find_by(id: 123)
  listings = target_suburb.listings
  

• 如果是嵌套关联的情况（比如要通过两层关联查询），可以结合joins和where指定关联表的条件，这个逻辑和你的第二个问题类似，后面会详细展开。

先给你提个小细节：你的现有代码里用了joins(:suburbs)复数，但如果Listing和Suburb是belongs_to关联（一条Listing属于一个郊区），关联名应该是单数suburb，复数写法可能会导致查询出错或者不符合预期哦。

下面给你几种修正后的写法，都是更符合Rails最佳实践的：

1. 基础修正版（明确指定关联表条件）

listings = Listing.joins(:suburb)
                  .where('listings.updated_at >= ?', 2.months.ago)
                  .where(suburbs: { city_id: city_id })

这里用suburbs: { city_id: city_id }代替裸SQL字符串，既避免了SQL注入风险，也比直接写SQL更清晰。

2. 更优雅的merge写法（个人推荐这个）

# 先把Suburb的查询逻辑单独拎出来
suburbs_in_target_city = Suburb.where(city_id: city_id)
# 再把这个条件合并到Listing的查询里
listings = Listing.joins(:suburb)
                  .where('listings.updated_at >= ?', 2.months.ago)
                  .merge(suburbs_in_target_city)

这种写法的好处是把关联模型的查询逻辑分离出来，代码可读性更强，后续如果要修改Suburb的查询条件，直接改前面的变量就行，不用动Listing的查询部分。

3. 嵌套关联join（如果后续要用到City的其他字段）
   如果你的查询需要用到City表的其他字段，可以直接join到City模型：

listings = Listing.joins(suburb: :city)
                  .where('listings.updated_at >= ?', 2.months.ago)
                  .where(cities: { id: city_id })

这些写法都比你原来的代码更安全、更易维护，你可以根据实际需求选合适的~

内容的提问来源于stack exchange，提问作者rake-write-code