Ruby on rails 予期せぬ値が取得されて処理が失敗する

Question

実現したいこと

現在Ruby on railsで宿泊予約アプリを作成しているものです。
実装したい内容としては
・施設詳細画面にて「予約する」ボタンを押下
・予約確認画面に遷移し「予約確定」ボタンを押下した際に予約が完了する
というものです。

発生している問題・分からないこと

現在施設詳細画面は引数として存在する施設の値が指定されるように実装したため表示できている(http://localhost:3000/rooms/1)のですが、
予約確認画面に遷移すると存在しない施設の値が引数に指定され「予約を確定」ボタンを押下しても存在しない施設IDが値として指定されるためエラーとなっている状態です。(http://localhost:3000/reservations/29)
reservation＿contrrollerに記載のconfirmメソッドで値を取得している認識なのですが、下記のcreateメソッドとなにか競合する部分があるのか、そもそも上記の存在しない組織IDはどこから渡されているのかなど調べているのですが原因がわからない状態です。

エラーメッセージ

NoMethodError in ReservationsController#confirm
undefined method `update' for nil:NilClass
Extracted source (around line #43):
41
42
43
44
45
46
              
  # 予約確定処理
  def confirm
    if @reservation.update(confirmed: true)
      redirect_to reservations_path, notice: "予約が確定しました。"
    else
      flash[:alert] = "予約確定に失敗しました。"

Rails.root: /Users/sunao/reservation_manager

Application Trace | Framework Trace | Full Trace
app/controllers/reservations_controller.rb:43:in `confirm'

該当のソースコード

class ReservationsController < ApplicationController
    before_action :set_room, only: [:new, :create]
    before_action :set_reservation, only: [:show, :destroy]
    before_action :authenticate_user!

    # 予約一覧
    def index
      @reservations = current_user.reservations.includes(:room)
    end
  
    # 予約確認ページ
    def show
    end
  
    # 新規作成フォーム
    def new
      @reservation = Reservation.new
      @room = Room.find(params[:room_id])
    end
  
    # 作成処理
    def create
      @room = Room.find(params[:room_id])
      @reservation = current_user.reservations.new(reservation_params)
      @reservation.room = @room
  
      if @reservation.save
        redirect_to @reservation, notice: "予約が確定しました。"
      else
        flash.now[:alert] = "予約に失敗しました。入力内容をご確認ください。"
        render :new
      end
    end

      # 予約削除
      def destroy
        @reservation.destroy
        redirect_to reservations_path, notice: "予約を削除しました。"
      end

  # 予約確定処理
  def confirm
    if @reservation.update(confirmed: true)
      redirect_to reservations_path, notice: "予約が確定しました。"
    else
      flash[:alert] = "予約確定に失敗しました。"
      render :confirmation, status: :unprocessable_entity
    end
  end
  


    private

    def set_room
      @room = Room.find(params[:room_id])
    end

    def set_reservation
        @reservation = current_user.reservations.find_by(id: params[:id])
      unless @reservation
        redirect_to reservations_path, alert: "予約が見つかりませんでした。"
      end
    end
  
    def reservation_params
      params.require(:reservation).permit(:check_in_date, :check_out_date, :number_of_guests)
    end
  end
  

   # 作成処理
    def create
      @room = Room.find(params[:room_id])
      @reservation = current_user.reservations.new(reservation_params)
      @reservation.room = @room
  
      if @reservation.save
        redirect_to @reservation, notice: "予約が確定しました。"
      else
        flash.now[:alert] = "予約に失敗しました。入力内容をご確認ください。"
        render :new
      end
    end

試したこと・調べたこと

• teratailやGoogle等で検索した
• ソースコードを自分なりに変更した
• 知人に聞いた
• その他

上記の詳細・結果

ChatGPTにエラーメッセージを入力して対応を調べたのですが存在しない施設のIDが引数に指定されているということまでしかわからず原因がわからない状態です。
何卒お力添えをいただけますと幸いです。

補足

特になし

Comments

[maisumakun]

def confirmにいきなりある、@reservationはどのように取得していますでしょうか。

before_actionや何かしらのgemを利用しているのであれば、そのあたりもご提示ください。

[nao_sss]

maisumakun様
ご回答いただきありがとうございます。
また記載漏れがあったようで大変失礼いたしました、before actionについては以下を定義しております。

before_action :set_room, only: [:new, :create]
before_action :set_reservation, only: [:show, :destroy]
before_action :authenticate_user!

[nao_sss]

また重複となり恐縮ですがconfirmメソッドとして以下を定義しております。

# 予約確定処理
def confirm
if @reservation.update(confirmed: true)
redirect_to reservations_path, notice: "予約が確定しました。"
else
flash[:alert] = "予約確定に失敗しました。"
render :confirmation, status: :unprocessable_entity
end
end

またreservation_controllerにつきまして全文をあらためて掲載させていただきました。
よろしければ引き続きご教授のほど何卒よろしくお願いいたします。

[maisumakun]

> before actionについては以下を定義しております。

confirmではどれも動かないです。

[nao_sss]

ご回答いただきありがとうございます。
こちら私の方で before_action :set_reservationでconfirmを許可などしたところ問題なく予約が完了いたしました。
お忙しいところお手数をおかけし恐縮です、またよろしければご助力を頂けますと幸いです。

Answer 1

undefined method `update' for nil:NilClassと出ていますが、このエラーは「nilに対してメソッドを呼び出した」という意味であり、つまりレシーバーの@reservationがnilということです。

Railsでは、コントローラーはリクエストごとにインスタンスが生成されるので、前のリクエストで@reservationに代入した値が自動で引き継がれることはありません。特に代入しなければ、インスタンス変数はnilとなります。

Comments

[maisumakun]

> reservation＿contrrollerに記載のconfirmメソッドで値を取得している認識なのですが

掲載された範囲でそのようなコードは見当たりません。なんの処理もせずに@reservationを参照すれば、値はnilです。

[winterboum]

confirmメソッドで値を取得 しても、createメソッドが呼ばれるときには消えています。
maisumakunさんが書かれているように、ブラウザーからのリクエスト毎に controllerインスタンスは作り直されますから

[nao_sss]

maisumakun様
ご回答いただきありがとうございます。
また記載漏れがあったようで大変失礼いたしました、
「@reservationに代入した値が引き継がれない」という認識が抜けておりご指摘いただき大変幸甚に存じます。

confirmメソッドには以下の処理を記述しております。
よろしければ何卒ご確認を頂けますと幸いです。

# 予約確定処理
def confirm
if @reservation.update(confirmed: true)
redirect_to reservations_path, notice: "予約が確定しました。"
else
flash[:alert] = "予約確定に失敗しました。"
render :confirmation, status: :unprocessable_entity
end
end

[nao_sss]

winterboum様
ご回答いただきありがとうございます。
お手数をおかけし大変恐縮です、またご指摘いただきありがとうございます。
私の方でインスタンスに関する認識が抜けており大変勉強になります。
よろしければまたご教授を頂けますと幸いです。

[maisumakun]

> confirmメソッドには以下の処理を記述しております。

もとの回答のとおり、@reservationに対する代入は行われていないですし、追記されたbefore_actionにも存在しません。

先に@reservation = （Reservationを検索するコード）のようなものを実行しないといけないですが、それが見当たらないです。