Rails未登录用户购物车无法访问及遍历问题求助

核心错误原因 #

已登录用户的@cart_products是CartProduct实例集合，调用cart_item.product可获取关联商品；但未登录用户的@cart_products是Product实例集合，直接调用cart_item.product会报错（Product类无product方法）。同时未登录用户购物车为空时，@cart_products为nil，调用empty?也会抛出异常。

修复步骤 #

1. 统一控制器返回的数据结构（推荐方案） #

修改carts_controller.rb的show方法，让未登录用户的购物车数据模拟CartProduct结构，视图无需修改即可兼容两种场景：

class CartsController < ApplicationController
  def show
    if user_signed_in?
      @cart_products = current_user.cart.cart_products
    elsif session[:product_id]
      # 用OpenStruct包装Product，模拟CartProduct的product属性
      @cart_products = Product.where(id: session[:product_id]).map do |product|
        OpenStruct.new(product: product)
      end
    else
      # 空购物车返回空数组，避免nil报错
      @cart_products = []
    end
  end

  # 其他方法保持不变，后续优化见额外修复部分
end

2. 直接修改视图兼容两种类型（备选方案） #

若不想修改控制器，可在视图中判断item类型，分别处理：
修改carts/show.html.erb的循环部分：

<% @cart_products.each do |cart_item| %>
  <%# 判断是Product还是CartProduct，获取对应商品实例 %>
  <% cart_prod = cart_item.is_a?(Product) ? cart_item : cart_item.product %>
  <div class="max-w-sm bg-stone-900 rounded-lg dark:bg-stone-800 dark:bg-stone-800 ">
    <!-- 后续内容保持不变 -->
  </div>
<% end %>

同时修改空购物车判断，处理nil情况：

<% if @cart_products.nil? || @cart_products.empty? %>
  <div class="m-8">
    <h1 class="text-white text-center"> Your cart is empty. Go fill it :) </h1>
  </div>
<% end %>

额外潜在问题修复 #

避免未登录用户重复添加商品 #

修改add_product方法，防止同一商品重复加入session：

def add_product
  @product = Product.find(params[:id])
  if user_signed_in?
    @cart_product = current_user.cart.cart_products.create(cart_id: current_user.cart, product_id: @product.id)
  else
    session[:product_id] ||= []
    # 仅当商品ID未存在时才添加
    session[:product_id] << params[:id].to_i unless session[:product_id].include?(params[:id].to_i)
  end
end

修复未登录用户删除商品的类型不匹配问题 #

修改destroy_product方法，将参数转为整数再删除，避免session中整数ID与字符串参数不匹配：

def destroy_product
  @product = Product.find(params[:id])
  if user_signed_in?
    @cart_product = current_user.cart.cart_products.find_by(cart_id: current_user.cart, product_id: @product.id)
    @cart_product.destroy
  else
    session[:product_id].delete(params[:id].to_i)
    # 购物车为空时清空session，避免后续判断异常
    session.delete(:product_id) if session[:product_id].empty?
  end
  redirect_to show_cart_path
end

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