ActiveRecord::RecordNotFound in NotesController#showの意味

Question

ActiveRecord::RecordNotFound in NotesController#showと表示されました。
Couldn't find Note with 'id'=showとさらに表示されました。
notes_controller.rbのprivateメソット以下のset_noteメソットがおかしいようで
def set_note
@note = Note.find(params[:id])←ここの部分がおかしい
end
と表示されました。
idがshowメソットに記されていないという認識で間違いないですよね？
SequelProを見ても、notesテーブルにきちんと投稿したカラムにidが書かれています。
何かコマンドを打ち忘れ、データが反映されていないのでしょうか？
回答お願いします。

routes.rbは
Rails.application.routes.draw do

devise_for :users
resources :users, only: [:index, :show, :edit, :update]
resources :notes, only: [:show, :create, :edit, :update, :destroy]

get'/top' => 'home#top'

get'/connection' => 'home#connection'
get'/notes/new' => 'notes#new'
post'/notes' =>'notes#create'
get'/notes' =>'notes#index'
get'/show' =>"notes#show"
get'/edit' =>'notes#edit'
patch'/notes/:id' =>"notes#update",as:'update_note'
delete'/notes/:id' =>"notes#destroy",as:'destroy_note'

post '/like/:note_id' => 'likes#like', as: 'like'
delete '/unlike/:note_id' => 'likes#unlike', as: 'unlike'

root 'home#top'
get'/about' => 'home#about'
end
のようになっています。
idはshowである、のように記述していないのでは？と思って困っています。
お願いいたします。

Comments

[rifuch]

問題のページを呼び出している部分を見せてください。というか、そこをチェックすれば自ずとわかる気がしますが･･･

Answer 1

画面がないので詳細にはわかりませんがroutingがおかしいです

route.rbの中で
notes/:id, to: 'notes#show'みたいに書いてあるとおもうですが、おそらくいま表示しているページは
/notes/showではないでしょうか

なので
/notes/showにアクセス
routing的には
/notes/:idなので
/notes/showはid=showを探しに行く
Note.find('show') => idがshowのやつを探しに行く。あれ、ないじゃん、
idがshowっていうデータないよーーーーで
Couldn't find Note with 'id'=show
がでているというわけです

なので実際にアクセスするべきページは
/notes/1とか/notes/23とかの数字が入るはずですこうすると
Noteテーブルからidが1とかのレコードを探してきてくれます！

Comments

[退会済みユーザー]

なるほど！質問にコードを追加したのですが、idはshowである、のように記述していないのでは？と思って困っています。 教えてください、お願いします。

[MasakazuFukami]

端的に答えを言うと
> idはshowである、のように記述していない
これは、
route.rbの
resources :notes, only: [:show, :create, :edit, :update, :destroy]
ここに含まれています。
railsが使えるところでrake routesする or /rails/info/routes
（http://localhost:3000/rails/info/routes）してみると
GET /notes/:id(.:format) notes#show
がどこかにあると思います！

また、routingの決まりを一応確認なのですが、
同じroutingがある場合、先に書いたroutingを参照します。

結局railsでurlから何かを取得する場合は
routingで
/xxxxxxx/:id（実際にアクセスするurl）, to: 'xxxxxxx/show'（ xxxxxx_controller,rbのshowメソッド）
ってやります
そうすると
上記urlにアクセスした瞬間にviewで必要なデータ等を準備するために
xxxxxxx_controller.rbのdef showに行きます
で@note = Note.find params[:id]で@noteを準備
viewで@noteを使ってレンダリング
みたいになるわけです。

ちなみに質問者様が書いている
get'/show' =>"notes#show"
get'/edit' =>'notes#edit'
これですと、
/showやeditにアクセスした時に本当は「画面に表示したり、編集したりしたい」のにcontrollerからすればurlに「どのid」のデータを取ってくるのかが入っていないため、何もすることができません。

なので正しいroutingは
get '/notes/:id' => 'notes#show'
get '/notes/:id/edit' => 'notes#edit'
みたいになるわけです！

とごちゃごちゃ書きましたが、
get '/notes/:id' => 'notes#show'
get '/notes/:id/edit' => 'notes#edit'
このようなroutingを全てやってくれるのが
resources :notes, only: [:show, :create, :edit, :update, :destroy]
これです。

長々書いてしまいましたすいません。
再度になりますが、質問者様の疑問の
> idはshowである、のように記述していない
これは、
route.rbの
resources :notes, only: [:show, :create, :edit, :update, :destroy]

が全てやってくれています！

[退会済みユーザー]

丁寧なコメントありがとうございます！！助かりました！！