嘿，很高兴看到你重新拾起Rails！咱们一步步把这个问题理顺，其实你已经搭好了核心的关联基础，只需要调整几个细节就能实现需求：

1. 修正Buildings控制器的逻辑 #

你的show动作里有两个小问题：一是多余的@buildings = Building.all（这个在index动作里用就够了），二是没利用好Rails的关联特性来获取居民。因为你已经在模型里设置了has_many :residents，直接通过建筑实例就能拿到它的所有居民，不用手动写where查询。

修改后的控制器代码：

class BuildingsController < ApplicationController
  def index
    @buildings = Building.all # 这里把变量名改成复数@buildings，对应“所有建筑”的集合，更符合Rails约定
  end

  def show
    @building = Building.find(params[:id])
    @residents = @building.residents # 直接通过关联获取该建筑的所有居民，Rails会自动帮你生成正确的SQL查询
  end
end

2. 修复index视图的链接语法 #

你原来的link_to写法有语法错误，而且手动拼接URL不是Rails的最佳实践。Rails会根据RESTful路由自动生成路由助手方法，用它更可靠。

修改后的app/views/buildings/index.html.erb（注意后缀是.html.erb，不是.rb）：

<% @buildings.each do |b| %>
  <div>
    <%= link_to b.name, building_path(b) %>
  </div>
<% end %>

building_path(b)会自动生成类似/buildings/1的URL，对应buildings#show动作，Rails会自动把b.id作为params[:id]传递给控制器。

3. 创建show视图展示居民列表 #

接下来需要创建app/views/buildings/show.html.erb页面，用来展示当前建筑的信息和它的居民：

<h1><%= @building.name %></h1>

<h2>该建筑的居民</h2>
<% if @residents.empty? %>
  <p>暂时没有居民入住~</p>
<% else %>
  <ul>
    <% @residents.each do |resident| %>
      <li><%= resident.name %></li> <!-- 这里替换成你Resident模型里的实际属性，比如email、phone等 -->
    <% end %>
  </ul>
<% end %>

<!-- 加个返回按钮，方便回到建筑列表 -->
<%= link_to '返回所有建筑', buildings_path %>

额外的小提醒 #

• 确保你的config/routes.rb里有resources :buildings，这是Rails RESTful路由的核心，会自动帮你生成index、show等动作的路由规则。
• 变量名尽量遵循Rails约定：复数变量对应集合（比如@buildings），单数变量对应单个实例（比如@building），这样代码可读性会高很多。
• 用关联查询（@building.residents）比手动写Resident.where(building_id: @building.id)更简洁，而且Rails会自动处理查询缓存，性能更好。

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