RailsでECサイトを作成中、商品をカートに追加するとき、NoMethodError in CartsController#add_itemが発生

Question

前提・実現したいこと

Railsでカート機能を備えたECサイトを作成しています。

ユーザーがログイン済みの状態でproducts（商品一覧）ページの「カートに入れる」ボタンを押すと、ユーザーidと紐づいたカートが生成され、cart_itemテーブルに商品の個数と、選んだ商品のidを保存できるようにしたいです。
現在、「カートに入れる」ボタンを押すと、下記のエラーが発生します。
cartscontrollerのsetup_cart_item!のメソッドの書き方に問題があるのか、と疑ってみたものの、
解決の糸口が見つからず、困っています。

発生している問題・エラーメッセージ

該当のソースコード

#routes.rb
Rails.application.routes.draw do
  devise_for :accounts
  
  resources :products
 
  resources :carts, only: [:show]
  
  root to: 'top#index'

  post '/show' ,to: 'carts#show'
  post '/buy' ,to: 'carts#buy'

  get '/my_cart' => 'carts#my_cart'
  post '/add_item' => 'carts#add_item'
  post '/update_item' => 'carts#update_item'
  delete '/delete_item' => 'carts#delete_item'

  get 'products/index'
  get 'products', to: 'products#index'

  get 'products/add'
  post 'products/add'

  get 'products/:id', to:'products#show'
  
  get 'products/edit/:id', to:'products#edit'
  patch 'products/:id/edit', to:'products#edit'
  
  get 'products/delete/:id', to:'products#delete'

  # For details on the DSL available within this file, see https://guides.rubyonrails.org/routing.html
  get 'top/index'
  get 'top', to: 'top#index'
  post 'top/index'
  post 'top', to: 'top#index'
  
  get 'top/products'
  post 'top/products'

end

#CartsController.rb
class CartsController < ApplicationController
  layout 'carts'
  before_action :setup_cart_item!, only: [:add_item, :update_item, :delete_item]
  #before_action :setup_cart_item!, only: %i[add_item update_item delete_item]

    def show
      @cart_items = current_cart.cart_items
      #@cart_items = CartItem.all
    end
    
    # カート内アイテムの表示
    def my_cart
      @cart_items = current_cart.cart_items.find_by(product_id: params[:product_id])
      @total = @cart_items.inject(0) { |sum, item| sum + item.sum_of_price }
    end  
 
    # 商品一覧画面から、「商品購入」を押した時のアクション
    def add_item
      @cart_item ||= current_cart.cart_items.build(product_id: params[:product_id])
      @cart_item.quantity += params[:quantity].to_i
      if  @cart_item.save
        flash[:notice] = '商品が追加されました。'
        redirect_to my_cart_path
      else
        flash[:alert] = '商品の追加に失敗しました。'
        redirect_to product_url(params[:product_id])
      end
    end

    # カート詳細画面から、「更新」を押した時のアクション
    def update_item
      @cart_item.update(quantity: params[:quantity].to_i)
      redirect_to current_cart
    end
  
    # カート詳細画面から、「削除」を押した時のアクション
    def delete_item
      @cart_item.destroy
      redirect_to current_cart
    end

    # カート詳細画面から、「購入手続きへ」を押した時のアクション
     def buy
      @msg = '購入手続き'
    end 

    private
  
    def setup_cart_item!
      @cart_item = current_cart.cart_items.find_by(product_id: params[:product_id])
      Rails.logger.debug("デバッグcurrent_cart2" + @current_cart.inspect)
    end

  end

#carts_helper.rb
module CartsHelper
    def current_cart
        if current_account
          # ユーザーとカートの紐付け
          if Cart.where(account_id: current_account.id)
            @current_cart = Cart.where(account_id: current_account.id)
          else
            @current_cart = Cart.new
            @current_cart.account_id = current_account.id
            @current_cart.save
          end
  
          logger.debug("デバッグcurrent_account" + @current_account.inspect)
          logger.debug("デバッグcurrent_cart" + @current_cart.inspect)
        else
          # セッションとカートの紐付け
          current_cart = Cart.find_by(id: session[:cart_id]) || Cart.create
          session[:cart_id] ||= current_cart.id
        end
        current_cart
    end
  
  end

#ApplicationController.rb
class ApplicationController < ActionController::Base
    protect_from_forgery with: :exception
    include CartsHelper  

end

#app/views/top/products.html.erb
<table class="table">
  <tr>
    <%# <th>ID</th> %>
    <th>商品名</th><th>価格</th><th>購入数</th><th colspan="2"></th>
  </tr>
  <% @products.each do |obj| %>
  <tr>
    <%# <td><%= obj.id %></td>
    <td><%= obj.brand %></a></td>
    <td><%= obj.price %></td>
    <td><%= number_field_tag("quantity","1",min:1,max:1000,class:"form-control") %></td> 
    <td><%= button_to 'カートに入れる', controller: :carts, action: :add_item, 
        params: {cart_id: :cart_id, product_id: :product_id, quantity: :quantity } , class: 'fbtn' %></td>
  </tr>
  <% end %>
</table>

#account.rb
class Account < ApplicationRecord
  # Include default devise modules. Others available are:
  # :confirmable, :lockable, :timeoutable, :trackable and :omniauthable
  devise :database_authenticatable, :registerable,
         :recoverable, :rememberable, :validatable
  has_one :cart
end

#cart_item.rb
class CartItem < ApplicationRecord
  belongs_to :product
  belongs_to :cart

  # カート内の商品合計に利用
  def sum_of_price
    product.price * quantity
  end

end

#cart.rb
class Cart < ApplicationRecord
    has_many :cart_items
    belongs_to :accounts
end

試したこと

current_cartがnilになっていることを疑い、デバッグしたところ、
Carts_helperのcurrent_cartでは、current_cartの値が取得できていた。
CartsContllolerのsetup_cart_item!のメソッドで、エラーが発生してしまう。

補足情報（FW/ツールのバージョンなど）

ruby 2.7.4p191
Rails 6.1.4.1

参考にしたサイト

Rails5でカート機能を作るためのロジックを作ってみた
RailsでECサイトのカート機能を実装する（ユーザーログインあり・なし両方対応）

Answer 1

#carts_helper.rb
module CartsHelper
    def current_cart
        if current_account
          # ユーザーとカートの紐付け
          if Cart.where(account_id: current_account.id)
            current_cart = current_account.cart
           else
            current_cart = Cart.new
            current_cart.account_id = current_account.id
            current_cart.save
          end
        else
          # セッションとカートの紐付け
          current_cart = Cart.find_by(id: session[:cart_id]) || Cart.create
          session[:cart_id] ||= current_cart.id
        end
        current_cart
    end
    

  end

Answer 2

carts_helper.rb
で最後にcurrent_cartを返しているが
上のifでは@current_cartとcurrent_cartを使っている
ので@current_cartにしか入れていないときは当然nilが返るので
全部@current_cartに統一しないと×

#carts_helper.rb
module CartsHelper
    def current_cart
        if current_account
          # ユーザーとカートの紐付け
          if Cart.where(account_id: current_account.id)
            @current_cart = Cart.where(account_id: current_account.id)
          else
            @current_cart = Cart.new
            @current_cart.account_id = current_account.id
            @current_cart.save
          end

          logger.debug("デバッグcurrent_account" + @current_account.inspect)
          logger.debug("デバッグcurrent_cart" + @current_cart.inspect)
        else
          # セッションとカートの紐付け
          @current_cart = Cart.find_by(id: session[:cart_id]) || Cart.create
          session[:cart_id] ||= @current_cart.id
        end
        @current_cart
    end

  end

Comments

[motaka]

回答ありがとうございます。
修正してみたのですが、エラー文言に変化はありませんでした。
指摘いただいた箇所以外にも、間違っているところがあったようです。

Cartshelperを見直しました。

```修正前
#carts_helper.rb
if current_account
# ユーザーとカートの紐付け
if Cart.where(account_id: current_account.id)
@current_cart = Cart.where(account_id: current_account.id)
else
@current_cart = Cart.new
@current_cart.account_id = current_account.id
@current_cart.save
end
```

↓

```修正後
#carts_helper.rb
if current_account
# ユーザーとカートの紐付け
if Cart.where(account_id: current_account.id)
current_cart = current_account.cart
else
current_cart = Cart.new
current_cart.account_id = current_account.id
current_cart.save
end
```