Rails项目如何正确实现用户选中房间的单房间信息展示

你遇到的路由异常、固定返回第一个房间的问题由三个错误共同导致：

• 路由未配置接收房间ID的动态段，跳转时传入的ID被识别为请求格式后缀，才会出现/single_show.2这类错误路径，此时params[:id]实际为nil
• find_by方法使用不符合语法规范，传入nil值时会直接返回数据表的第一条记录，和你选中的房间完全无关
• 原有show动作的渲染顺序错误，先执行的render语句会直接终止后续逻辑，房间判断代码永远不会生效

1. 修正路由配置 #

修改config/routes.rb，给single_show路由添加:id动态段，用于接收房间ID参数：

Rails.application.routes.draw do
  resources :rooms
  # 替换原有single_show路由配置
  get 'single_show/:id', to: 'rooms#single_show', as: 'single_show_room'
end

配置完成后，页面跳转时使用对应路由助手生成正确路径：

<%# 视图层跳转链接写法，传入房间实例即可生成/single_show/1这类标准路径 %>
<%= link_to '进入房间', single_show_room_path(room) %>

之前出现.2后缀的原因是：没有定义:id动态段时，传入的ID值会被Rails解析为请求格式（format）参数，对应params[:format] = "2"，无法被动作正确读取。

2. 修正single_show动作的查询逻辑 #

主键ID查询直接使用find方法即可，不需要用find_by，修改app/controllers/rooms_controller.rb中对应的动作代码：

def single_show
  # find方法会按主键查询对应记录，找不到时自动返回404错误，符合常规业务逻辑
  @room = Room.find(params[:id])

  # 如果要使用find_by，必须明确指定查询字段，写法为：
  # @room = Room.find_by(id: params[:id])
end

3. （可选）优化原有show动作逻辑 #

如果需要保留原来的show动作，调整渲染语句的顺序，避免逻辑提前终止：

def show
  @rooms = Room.all
  begin
    @room = Room.find(params[:id])
  rescue ActiveRecord::RecordNotFound
    render 'new'
    return # 加return终止后续执行，避免重复渲染报错
  end
  render 'index'
end

按照Rails资源路由的约定，resources :rooms自动生成的show动作本身就是用来展示单个房间详情的，不需要额外新建single_show动作，把自定义逻辑写到标准show动作中更符合框架开发规范。

内容的提问来源于stack exchange，提问作者Andrew Kancev