注文した商品の一覧を表示させたいです。rails

Question

前提・実現したいこと

前提
現在、Ruby on Railsを学習しており、レストランでの注文アプリを作成したいと考えています。
仕様としては
1.レストランに入って、席に置いてあるQRコードを読み取るとメニュー画面が表示される
2.メニュー一覧の商品をクリックすると、商品詳細画面に飛ぶ
3.商品詳細画面で、数量を入力して注文ボタンを押すと、カートに情報がとぶ
4.カートはsessionを用いてidを保存して、メニュー画面に戻っても、同じカートが表示される
5.カート内の商品一覧画面で注文を確定するボタンを押すとカートidが注文確定テーブルに保存され、sessionのカートidを削除する
6.席ごとに注文を判別できるように確定した注文と席の情報を結びつける
と考えております。

テーブルとしては、
1.商品を登録・表示させるためもの（Menuテーブル）　
2.注文したいものを入れるカートの役割をするためもの(Orderテーブル)
3.どの商品が何個入っているか判別するための１と２の中間テーブル(MenuOrderテーブル)
4.注文確定テーブル(OrderConfirmテーブル)
5.席番号のテーブル(Tableテーブル)
を作っています

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

注文確定された商品を出力するためにテーブルコントローラーで下記を定義しましたが
def show
@table = Table.find(params[:id])
@menu_orders = MenuOrder.where(order_id: @table.order_confirms.order_id)
end
をで定義しましたが、下記のエラーがでます。
NoMethodError in TablesController#show
undefined method `order_id' for #OrderConfirm::ActiveRecord_Associations_CollectionProxy:0x00007fa345f5e3e0 Did you mean? order

どこか記述が間違っているのでしょうか。また、エラーを解消してもこの記述では出力できない等はありますでしょうか。

該当のソースコード

ルーティング

Rails.application.routes.draw do
  root 'menus#index'
  resources :tables, only: [:index, :new, :create, :show] do
    resources :menus 
    resources :orders, only: [:new, :create]
    get '/my_order' => 'orders#my_order'
    post '/add_menu' => 'orders#add_menu'
    post '/update_menu' => 'orders#update_menu'
    delete '/delete_menu' => 'orders#delete_menu'
    resources :order_confirms, only: [:create, :destroy]
  end
end

モデル周り

class Table < ApplicationRecord
  has_many :order_confirms
end

class Order < ApplicationRecord
  has_many :menu_orders, dependent: :destroy
  has_many :menus, through: :menu_orders
  has_one :order_confirm
end

class OrderConfirm < ApplicationRecord
  belongs_to :order
  belongs_to :table
end

class Menu < ApplicationRecord
  has_one_attached :image
  has_many :menu_orders, dependent: :destroy
  validates :price, numericality: { only_integer: true }
  validates :name, presence: true
  has_many :order, through: :menu_orders
end

class MenuOrder < ApplicationRecord
  belongs_to :menu
  belongs_to :order

  def sum_of_price
    menu.price * quantity
  end
end

コントローラー周り

class TablesController < ApplicationController
  def index
    @tables = Table.includes(:order_confirms)

  end
  def new
    @table = Table.new
  end
  def create
    @table = Table.new(table_params)
    if @table.save
      redirect_to tables_path
    else
      render :new
    end
  end
  def show
    @table = Table.find(params[:id])
    @menu_orders = MenuOrder.where(order_id: @table.order_confirms.order_id)
  end

  private

  def table_params
    params.require(:table).permit(:table_num)
  end

end

class OrdersController < ApplicationController
  before_action :setup_order_menu!, only: [:add_menu, :update_menu, :delete_menu]

  def my_order
    @menu_orders = current_order.menu_orders.includes([:menu])
    @total = @menu_orders.inject(0){ |sum, item| sum + item.sum_of_price }
    @order_confirm = OrderConfirm.new
  end

  # 商品一覧画面から、「商品購入」を押した時のアクション
  def add_menu
    @menu_order ||= MenuOrder.new(order_id: current_order.id, menu_id: params[:menu_id])
    @menu_order.quantity += params[:quantity].to_i
    if @menu_order.save
      flash[:notice] = '注文を追加しました'
      redirect_to table_my_order_path(params[:table_id])
    else
      flash[:alert] = '注文の追加に失敗しました'
      redirect_to table_menu_url(table_id: params[:table_id], id: params[:menu_id])
    end
  end

  # カート詳細画面から、「更新」を押した時のアクション
  def update_menu
    @menu_order.update(quantity: params[:quantity].to_i)
    redirect_to table_my_order_path(params[:table_id])
  end

  # カート詳細画面から、「削除」を押した時のアクション
  def delete_menu
    @menu_order.destroy
    redirect_to table_my_order_path(params[:table_id])
  end

  private

  def setup_order_menu!
    @menu_order = current_order.menu_orders.find_by(menu_id: params[:menu_id])
  end
end

class OrderConfirmsController < ApplicationController
  def create
    @order_confirm = OrderConfirm.new(order_confirm_params)
    @order = Order.find(@order_confirm.order_id)
    if @order_confirm.save
      session.delete(:order_id)
    else
      render "orders/my_order"
    end
  end
  def destroy
    
  end

  private

  def order_confirm_params
    params.require(:order_confirm).permit(:order_id).merge(table_id: params[:table_id])
  end
end

class MenusController < ApplicationController
  def index
    @menus = Menu.all
  end
  def new
    @menu = Menu.new
  end
  def create
    @menu = Menu.new(menu_params)
    if @menu.save
      redirect_to root_path
    else
      render :new
    end
  end
  def edit
    @menu = Menu.find(params[:id])
  end
  def update
    @menu = Menu.find(params[:id])
    if @menu.update(menu_params)
      redirect_to root_path
    else
      render :edit
    end
  end
  def destroy
    menu = Menu.find(params[:id])
    menu.destroy
  end
  def show
    @menu = Menu.find(params[:id])
    @menu_order = MenuOrder.new
  end

  private
  def menu_params
    params.require(:menu).permit(:name, :price, :image)
  end
end

マイグレーション

class CreateMenus < ActiveRecord::Migration[6.0]
  def change
    create_table :menus do |t|
      t.string  :name,  null: false
      t.integer :price, null: false
      t.timestamps
    end
  end
end

# This migration comes from active_storage (originally 20170806125915)
class CreateActiveStorageTables < ActiveRecord::Migration[5.2]
  def change
    create_table :active_storage_blobs do |t|
      t.string   :key,        null: false
      t.string   :filename,   null: false
      t.string   :content_type
      t.text     :metadata
      t.bigint   :byte_size,  null: false
      t.string   :checksum,   null: false
      t.datetime :created_at, null: false

      t.index [ :key ], unique: true
    end

    create_table :active_storage_attachments do |t|
      t.string     :name,     null: false
      t.references :record,   null: false, polymorphic: true, index: false
      t.references :blob,     null: false

      t.datetime :created_at, null: false

      t.index [ :record_type, :record_id, :name, :blob_id ], name: "index_active_storage_attachments_uniqueness", unique: true
      t.foreign_key :active_storage_blobs, column: :blob_id
    end
  end
end

class CreateOrders < ActiveRecord::Migration[6.0]
  def change
    create_table :orders do |t|
      t.timestamps
    end
  end
end

class CreateMenuOrders < ActiveRecord::Migration[6.0]
  def change
    create_table :menu_orders do |t|
      t.integer :quantity, default: 0
      t.references :menu, null: false, foreign_key: true
      t.references :order, null: false, foreign_key: true
      t.timestamps
    end
  end
end

class CreateTables < ActiveRecord::Migration[6.0]
  def change
    create_table :tables do |t|
      t.integer :table_num, null: false
      t.timestamps
    end
  end
end

class CreateOrderConfirms < ActiveRecord::Migration[6.0]
  def change
    create_table :order_confirms do |t|
      t.references :order,  null: false, foreign_key: true
      t.references :table,  null: false, foreign_key: true
      t.timestamps
    end
  end
end

テーブルのshowビュー

<main class="main">
    <div class="card__wrapper">
      <%= render partial: "table", locals: {table: @table} %>
      <% @menu_orders.each do |order| %>
        <%= order.menu.name %>
      <% end %>
    </div>
    <div>
      <%= link_to "メニュー画面へ", table_menus_path(@table.id) %>
    </div>
</main>

試したこと

order_confirmテーブル上にorder_idカラムがあり、データが保存されていたため、order_idがないわけではない。そのためアソシエーションに問題があるのではと考え、tableとorder_confirmのアソシエーションを見てみたが、問題は発見できなかった。

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

ここにより詳細な情報を記載してください。

Answer 1

model の関連にちょっと疑問はありますが、とりあえずそのままとして、
class Table < ApplicationRecord has_many :order_confirms ですと @table.order_confirms.order_id はおかしいですね。order_confirmsは配列もしくはcollentionですから order_idは持ちません。やるなら @table.order_confirms.map(&:order_id)' とか pluck(:order_id)ですね。 で、class MenuOrder はbelongs_to :order` ですから order_idは持ってるので、このエラーおかしいな。

関連にちょっと疑問

table 1:多 OrderConfirm 1:1 Order 1:多 Memue
とありますが、
OrderConfirmを挟む必要は無いのでは
挟んだとしても 1:1 なのでは

Comments

[toshiki.0707]

ご回答いただきありがとうございます！！
ご指摘の通りで、pluckを使ったことで無事にエラーを解消することができました！

### 関連について

関連についてですが、Orderの段階（カート機能に入れた段階）では、まだ商品の数量変更や取り消しをユーザーができるように設定しているため、レストランでの使用を考えた際に、厨房側で商品を作っていたら、数量が減った・無くなったということが起こらないように一度OrderConfirmに移して注文を確定させるという処理が必要かと考えました。
order.idをsessionに保存していますが、注文確定する段階(OrderConfirmを挟む際)でorder.idを消して、もう一度注文する際には新しいOrderを作成する形で考えています。そのため、注文の度にOrderが新しく作られるため、Tableとの関係は１対多と定義しました。長文でわかりにくくてすみません。考え方が間違っている・もっとシンプルにできる等アドバイスがありましたらいただけますと幸いです。

[winterboum]

注文の確定はOrderにstatusをつければ済むかと思うのです

[toshiki.0707]

Orderだけで可能なんですね！すみません勉強不足で、statusをつければ可能ということが理解できないので、カート機能についてもっと色々な記事を見てみます！もし、この記事はわかりやすくて参考になる等ありましたら、教えていただけますと幸いです。