NoMethodErrorを解決したいです。（おそらく変数定義の仕方が悪いです） 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テーブル)
を作っています

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

お客様が注文確定したオーダーを店員が閲覧する用にMenuOrderのindexビューに全てのお客様の確定したオーダーを表示させたいと考えております。
そこでMenuOrderコントローラー内に確定したオーダーを取り出す変数@menu_ordersを定義したのですが、下記のエラーが出てしまいます。

エラーメッセージ
NoMethodError in MenuOrdersController#index
undefined method `order_confirms' 

@tables = Table.includes(:order_confirms)
@menu_orders = MenuOrder.where(order_id: @tables.order_confirms.map(&:order_id))

該当のソースコード

コントローラー周り

class MenuOrdersController < ApplicationController
  def index
    @tables = Table.includes(:order_confirms)
    @menu_orders = MenuOrder.where(order_id: @tables.order_confirms.map(&:order_id))
  end
end

class MenusController < ApplicationController
  before_action :set_menu, only: [:edit, :update, :show, :destroy]
  def index
    @menus = Menu.all.page(params[:page]).per(2)
  end
  def new
    @menu = Menu.new
  end
  def create
    @menu = Menu.new(menu_params)
    if @menu.save
      redirect_to tables_path
    else
      render :new
    end
  end
  def edit
  end
  def update
    if @menu.update(menu_params)
      redirect_to root_path
    else
      render :edit
    end
  end
  def destroy
    menu.destroy
  end
  def show
    @menu_order = MenuOrder.new
    @table = Table.find(params[:table_id])
  end

  private
  def menu_params
    params.require(:menu).permit(:name, :price, :image, {:tag_ids=> []})
  end

  def set_menu
    @menu = Menu.find(params[:id])
    @table = Table.find(params[:table_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 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
    @table = Table.find(params[:table_id])
  end

  # 商品一覧画面から、「商品購入」を押した時のアクション
  def add_menu
    @menu_order ||= MenuOrder.new(order_id: current_order.id, menu_id: params[:menu_id], table_num: params[:table_num])
    @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 TablesController < ApplicationController
  before_action :authenticate_user!, except: :show
  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.map(&:order_id))
  end
  def destroy
    @table = Table.find(params[:id])
    if @table.destroy
      redirect_to tables_path
    else
      render :show
    end
  end

  private

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

end

ルーティング

Rails.application.routes.draw do
  devise_for :users
  root 'tables#index'
  resources :tables, only: [:index, :new, :create, :show, :destroy] do
    resources :menus, only: [:index, :show]
    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
  resources :menus, only: [:new, :create, :edit, :update, :destroy]
  resources :menu_orders, only: :index
end

モデル周り

class MenuOrder < ApplicationRecord
  belongs_to :menu
  belongs_to :order

  def sum_of_price
    menu.price * quantity
  end

  validates :table_num, presence: true
end

class Menu < ApplicationRecord
  has_one_attached :image
  has_many :menu_orders, dependent: :destroy
  has_many :order, through: :menu_orders, dependent: :destroy
  has_many :menu_tag_relations, dependent: :destroy
  has_many :tags, through: :menu_tag_relations, dependent: :destroy

  validates :name, presence: true
  validates :price, numericality: { only_integer: true }
end

class OrderConfirm < ApplicationRecord
  belongs_to :order
  belongs_to :table
end

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

class Table < ApplicationRecord
  has_many :order_confirms, dependent: :destroy

end

class User < ApplicationRecord
  # Include default devise modules. Others available are:
  # :confirmable, :lockable, :timeoutable, :trackable and :omniauthable
  devise :database_authenticatable, :registerable,
         :recoverable, :rememberable, :validatable
        
  validates :nickname, presence: true
end

試したこと

似たような変数定義をした際に、tableを一つに指定した場合にはエラーが出ずに変数定義をすることが出来ました。そのため、tableのレコードが複数あることで何か問題が起きてしまっているのだと考えました。この仮説があっている場合、tableが複数の時にはどのように記述すれば、確定したオーダーを全て抽出することができるでしょうか。

また、エラーとは別件になりますが、現在オーダーを確定するのにオーダー確定テーブルを用いて確定をしています。
しかし、以前別のエラーで質問した際に回答いただいた方に、statusを用いればオーダーテーブルのみでオーダーを確定させることができるとのアドバイスをいただきました。ただ、自分の調べ方が悪いのかそのようなやり方を発見することができませんでした。わかる方がいらっしゃれば、オーダーのみでオーダーを確定させる方法をお教えいただくか、もしくは参考にできる記事等をお教えいただけますでしょうか。

以上、２点についてご教示いただけますでしょうか。

Answer 1

エラーメッセージのとおりです。コレクションである@tablesから直接order_confirmsを参照することはできません。

Comments

[maisumakun]

Tableモデルにhas_many :order, through: :order_confirmを定義して、そこからordersを取るのが楽かもしれません。

[maisumakun]

＞ 現在オーダーを確定するのにオーダー確定テーブルを用いて確定をしています。

orderがたかだか1つのテーブルにしか結びつかない以上、order側にtable_idをもたせれば、中間テーブルは不要です。