Rails报undefined method 'id' for ActiveRecord::Relation错误排查

访问Products#index页面时触发NoMethodError，错误信息为：

undefined method 'id' for #<ActiveRecord::Relation>

无法直接定位问题根源，相关代码如下：

• Products控制器代码

class ProductsController < ApplicationController
  def index
    if params[:query].present?
      @products = Product.search_by_name_and_category(params[:query])
    else
      @products = Product.all
    end
  end

  def new
    @product = Product.new
    @product.user = current_user
  end

  def show
    @product = Product.find(params[:id])
  end
end

• Product模型代码

class Product < ApplicationRecord
  belongs_to :user
  has_many :bookings
  validates :name, presence: true
  validates :category, presence: true
  has_one_attached :photo
  include PgSearch::Model
  pg_search_scope :search_by_name_and_category,
  against: [ :name, :category ],
  using: {
    tsearch: { prefix: true }
  }
end

• product-card局部视图代码

<div class="card-product-container">
  <div class="cards">
    <div class="card-product">
      <%= link_to product_path(@products.id) do %>
      <img src="https://source.unsplash.com/random/?<%= product.name %>" />
        <div class="card-product-footer">
          <div>
            <h2><%= product.name %></h2>
            <p><%= product.category %></p>
          </div>
        <h2><%= product.price %></h2>
    </div>
    <% end %>
  </div>
</div>

错误核心是类型不匹配：

1. 控制器index动作中，不管是搜索返回的结果还是Product.all，返回值都是ActiveRecord::Relation类型——也就是多条Product记录组成的集合对象，不是单个Product实例，集合对象本身没有id属性/方法。
2. 局部视图中直接对集合对象@products调用.id方法，同时没有对@products做遍历操作，视图中使用的product局部变量也没有定义来源，双重问题触发报错。

1. 调整index页面视图结构，遍历产品集合 #

在index视图中遍历@products集合，将遍历得到的单个Product实例传入局部视图：

<!-- app/views/products/index.html.erb -->
<div class="card-product-container">
  <div class="cards">
    <% @products.each do |product| %>
      <%= render "product_card", product: product %>
    <% end %>
  </div>
</div>

2. 修正局部视图的错误调用 #

将局部视图中错误的@products.id替换为单个产品实例的product.id，修正后的局部视图代码：

<!-- app/views/products/_product_card.html.erb -->
<div class="card-product">
  <%= link_to product_path(product.id) do %>
    <img src="https://source.unsplash.com/random/?<%= product.name %>" />
    <div class="card-product-footer">
      <div>
        <h2><%= product.name %></h2>
        <p><%= product.category %></p>
      </div>
      <h2><%= product.price %></h2>
    </div>
  <% end %>
</div>

优化提示：如果局部视图命名遵循Rails约定（命名为_product.html.erb），可以直接使用<%= render @products %>简写语法，框架会自动完成集合遍历、单条实例传参的操作，无需手动编写each循环。

内容的提问来源于stack exchange，提问作者Adrienne Ackley