ActiveRecord::RecordNotFound in ObservationsController#edit Couldn't find Observation with 'id'=2

Question

編集画面に遷移出来ていたが、他の機能を実装した後に上記エラーが出る様になった。

エラー文

ActiveRecord::RecordNotFound in ObservationsController#edit
Couldn't find Observation with 'id'=2

def set_observation
    **@observation = Observation.find(params[:patient_id])**
  end
end

class ObservationsController < ApplicationController
  before_action :set_patient, only: [:new, :create, :edit, :update]
  before_action :set_observation, only: [:edit, :update]
  
  def new
    @observation = Observation.new
  end

  def create
    @observation = Observation.new(observation_params)
    if @observation.save  # バリデーションをクリアした時
      redirect_to patient_path(@patient)
    else
      render :new # バリデーションに弾かれた時
    end
  end

  def show
    observations = Observation.where(patient_id: params[:patient_id]).to_a
    @observations = observations.sort_by{|o| o.time.delete(":").to_i}
    @data = @observations.map  do |o|
      if o.temperature.present?
        [o.time, o.temperature]
      end
    end
    @data2 = @observations.map do |o|
      if o.pulse.present?
        [o.time, o.pulse]
      end
    end
    @data3 = @observations.map do |o|
      if o.respiration.present?
        [o.time, o.respiration]
      end
    end
    @data4 = @observations.map do |o|
      if o.high_blood_pressure.present?
        [o.time, o.high_blood_pressure]
      end
    end
    @data5 = @observations.map do |o|
      if o.low_blood_pressure.present?
        [o.time, o.low_blood_pressure]
      end
    end

  end

  def update
    if @observation.update(observation_params) # バリデーションをクリアした時
      redirect_to patient_observation_path(@observation)
    else
      render :edit  # バリデーションに弾かれた時
    end
  end

  private

  def observation_params
    params.require(:observation).permit(:temperature, :pulse, :respiration, :high_blood_pressure, :low_blood_pressure, :spo2, :food_intake,
    :water_intake, :excresion, :ex_amount, :atten_sound, :atten_part, :sputum, :cough, :sleep, :time, :hainyou).merge(user_id: current_user.id, patient_id: params[:patient_id], user_name: current_user.name)
  end

  def set_patient
    @patient = Patient.find(params[:patient_id])
  end

  def set_observation
    @observation = Observation.find(params[:patient_id])
  end
end

Rails.application.routes.draw do
  devise_for :users
  get 'patients/index'
  root to: "patients#index"
  resources :users, only: [:edit, :update]
  resources :patients, only: [:new, :create, :index, :show, :edit, :update] do
    resources :observations, only: [:new, :create, :edit, :show, :update]
  end
end

~views/observations/show.html.erb~
<div class="observation-show">
  <div class="observation-box">
  
    <div class="btn">
      <div class="ns-btn">
      <%= image_tag "くま看護師.png" ,class:"kuma-icon" ,width:"40",height:"40"%>
      <%= link_to "観察項目編集", edit_patient_observation_path %>
      </div>
      <div class="kensa-btn">
        <%= image_tag "レントゲン1.jpg" ,class:"kensa-icon" ,width:"40",height:"40"%>
        <%= link_to "検査結果参照", '#' %>
      </div>
      <div class="syoti-btn">
        <%= image_tag "点滴.jpeg" ,class:"treatment-icon" ,width:"40",height:"40"%>
        <%= link_to "処置の入力", '#' %>
      </div>
    </div>
  </div>
</div>

def set_observation
@observation = Observation.find(params[:patient_id])
end
のfind(params[:patient_id])の:patient_idが必要なのかどうかがいまいち分からないです。初心者です、よろしくお願い致します。

Answer 1

理由が分かりました。私のアプリのデータ形式が表テーブルに複数の@observation = Observation.find(params[:patient_id])のデータが有ったためだと思われます。
現在javascriptでjQueryやAjaxを用いての編集形式に変更しました。
回答して頂いたnecocoa様、考えて頂いた方々有り難うございました。

Answer 2

@observation = Observation.find(params[:id])こちらが正しいです。

他の機能を実装した後に上記エラーが出たわけではなく、新しくPatientを作りpatient_idが2になったためエラーになったのでしょう。

また、編集画面に遷移出来ていたとあるので、editが合ったと仮定しますが、
editアクションが無いですが誤って消したりしてませんか？

Comments

[Misa0]

ご回答ありがとうございます！
上記私の方でも試行していたのですが、変化無しです。
→新しくPatientを作りpatient_idが2になったためエラーになった
このご意見を頂き再度考察し、views/observations/show.html.erb　の
<%= link_to "観察項目編集", edit_patient_observation_path %>
パスの指定の仕方がおかしいのかなと思うのですが、色々試行するのですがうまくいきません。
仮説が間違っているのか、記述が違うのか、またご意見いただければ助かります。
よろしくお願い致します。

[necocoa]

edit_patient_observation_pathには
`:patient_id`と`:id`の2つのidが必要になります。

```
<%= link_to "観察項目編集", edit_patient_observation_path(@patient, @observation) %>
```

このような形で引数に渡すとできます。
あとは`@patient` `@observation`この2つを用意する必要があるので
showアクションにこの2つの変数を用意するコードを追加すれば解決ですね！

できなかった場合、observations/show.html.erbのコードを貼っていただけると助かります！

[Misa0]

有難うございます！
なのですが次はNo route matches {:action=>"edit", :controller=>"observations", :id=>nil, :patient_id=>nil}, missing required keys: [:id, :patient_id]と言うエラーが出ました。
〜controllers/observations_controller.rb〜
def edit
@patient = Patient.find(patient_id: params[:patient_id])
@observation = Observation.find(observation: params[:observation_id])
end

〜controllers/patients_controller.rb〜
def edit
@patient = Patient.find(patient_id: params[:patient_id])
end

を追記するのですが、変わらないですね。
因みにobservations/show.html.erbは既に貼っているのですが、何か不足な部分が有りますでしょうか？
ご回答宜しく御願いいたします。