Q&A
id=13のBookは存在してますか？
一度データベースを確認してみてください
投稿を削除する際に以下のエラーが発生しました。
ActiveRecord::RecordNotFound in BooksController#destroy Couldn't find Book with 'id'=13 Extracted source (around line #36): 34 35 def destroy 36 @book = Book.find(params[:id]) 37 @book.destroy 38 redirect_to book_url(book.id) 39 end
調べたところroutesに問題がある場合が多かったため確認しましたが、解決できませんでした。
routesはresourcesで設定しています。
以下コードになります。
routes.rb Rails.application.routes.draw do devise_for :users, controllers: { sessions: 'users/sessions', registrations: 'users/registrations' } get root 'top#top' get '/home/about', to: 'top#home' resources :users resources :books # For details on the DSL available within this file, see http://guides.rubyonrails.org/routing.html end
$ rake routes Prefix Verb URI Pattern Controller#Action new_user_session GET /users/sign_in(.:format) users/sessions#new user_session POST /users/sign_in(.:format) users/sessions#create destroy_user_session DELETE /users/sign_out(.:format) users/sessions#destroy new_user_password GET /users/password/new(.:format) devise/passwords#new edit_user_password GET /users/password/edit(.:format) devise/passwords#edit user_password PATCH /users/password(.:format) devise/passwords#update PUT /users/password(.:format) devise/passwords#update POST /users/password(.:format) devise/passwords#create cancel_user_registration GET /users/cancel(.:format) users/registrations#cancel new_user_registration GET /users/sign_up(.:format) users/registrations#new edit_user_registration GET /users/edit(.:format) users/registrations#edit user_registration PATCH /users(.:format) users/registrations#update PUT /users(.:format) users/registrations#update DELETE /users(.:format) users/registrations#destroy POST /users(.:format) users/registrations#create root GET / top#top home_about GET /home/about(.:format) top#home users GET /users(.:format) users#index POST /users(.:format) users#create new_user GET /users/new(.:format) users#new edit_user GET /users/:id/edit(.:format) users#edit user GET /users/:id(.:format) users#show PATCH /users/:id(.:format) users#update PUT /users/:id(.:format) users#update DELETE /users/:id(.:format) users#destroy books GET /books(.:format) books#index POST /books(.:format) books#create new_book GET /books/new(.:format) books#new edit_book GET /books/:id/edit(.:format) books#edit book GET /books/:id(.:format) books#show PATCH /books/:id(.:format) books#update PUT /books/:id(.:format) books#update DELETE /books/:id(.:format) books#destroy refile_app /attachments #<Refile::App app_file="/home/ec2-user/.rvm/gems/ruby-2.6.3/bundler/gems/refile-46b4178654e6/lib/refile/app.rb">
books_controller.rb class BooksController < ApplicationController before_action :authenticate_user!, only: [:show] def create book = Book.new(book_params) book.user_id = current_user.id if book.save! redirect_to user_url(current_user.id) else render action: :show end end def index @book = Book.page(params[:book]).reverse_order end def show @book = Book.find(params[:id]) @user = @book.user end def edit @book = Book.find(params[:id]) end def update book = Book.find(params[:id]) book.update(book_params) redirect_to user_url(current_user.id) end def destroy @book = Book.find(params[:id]) @book.destroy redirect_to book_url(book.id) end private def book_params params.require(:book).permit(:title, :body) end end
books/show.html.erb <%= render 'shared/header' %> <div class="top"> <div class="container"> <div class="row"> <div class="col-lg-3"> <p>User info</p> <%= attachment_image_tag @user, :profile_image, :fill, 150, 150, format: 'jpeg', fallback: "no_image.jpg", size:'150x150' %> <table class="table"> <tr> <th>name</th> <th><%= current_user.name %></th> </tr> <tr> <th>introduction</th> <th><%= current_user.introduction %></th> </tr> </table> <%= link_to edit_user_path(@book.user), class: "btn btn-default" do %> <i class="fa fa-wrench"></i> <% end %> <p>New book</p> <%= form_for @book do |f| %> <p>Title</p> <%= f.text_field :title %> <p>Opinion</p> <%= f.text_area :body %> <%= f.submit 'Create Book' %> <% end %> </div> <div class="col-lg-9"> <p>Book detail</p> <table class="table table-hover"> <tbody> <tr> <td><%= attachment_image_tag @user, :profile_image, :fill, 50, 50, format: 'jpeg', fallback: "no_image.jpg", size:'50x50' %><br><%= current_user.name %></td> <td><%= @book.title %></td> <td><%= @book.body %></td> <td><%= link_to "Edit", edit_book_path(@book), class: "btn btn-success" %></td> <td><%= link_to "Delete", book_path(@book), class: "btn btn-danger", method: :delete %></td> ←投稿削除ボタン </tr> </tbody> </table> </div> </div> </div> </div> <%= render 'shared/footer' %>
def destroy を呼ぶのはshow.html.erbのしたの方の
<td><%= link_to "Edit", edit_book_path(@book), class: "btn btn-success" %></td>
<td><%= link_to "Delete", book_path(@book), class: "btn btn-danger", method: :delete %></td> ←投稿削除ボタン
のDeleteからですか？
回答頂きありがとうございます。
確認した所、id=13はありませんでした。
前のページに戻って他の投稿を削除した所、NameErrorになりました。
NameError in BooksController#destroy
undefined local variable or method `book' for #<BooksController:0x00007f69f098e6b0> Did you mean? @book
Extracted source (around line #38):
36 @book = Book.find(params[:id])
37 @book.destroy
38 redirect_to book_url(book.id)←エラー箇所
39 end
40
おそらく削除はできていますが、画面の移動がうまくいっていないと考えていますが、どうでしょうか？
<td><%= link_to "Edit", edit_book_path(@book), class: "btn btn-success" %></td>
<td><%= link_to "Delete", book_path(@book), class: "btn btn-danger", method: :delete %></td> ←投稿削除ボタン
のdeleteからのエラーです。
タイトルのエラーとNameErrorは別問題ですね。
まずタイトルの「Couldn't find Book with 'id'=13」は、単純にid=13のデータが存在しなかったので、
存在しないものは削除できない、というエラーです。
NameErrorの方は変数「book」が見つからない、というエラーです。
（@bookとbookは別物です。destroyアクションの中で定義されているのは@bookであり、bookではないですよね）
あと、削除してからのページ移動の流れが気になるのですが、
Bookを削除した後、削除したBookのページに移動するって辻褄が合わないような…
本当ですね。
Bookを消した後はredirectで指定しているので間違いです。
申し訳ありませんでした。
解決できなければ別で質問してみます。
ありがとうございました。
回答1件
0
ベストアンサー
redirect_to book_url(book.id)
ここがエラーの原因です。
削除したあとどのページを表示したいのでしょうか？
ホーム画面にリダイレクトしたいのであれば
redirect_to "/"
book一覧にリダイレクトしたいのであれば
redirect_to books_url
などです(表記方法は色々ありますが)
投稿2019/12/20 12:18
総合スコア192
ご回答頂きありがとうございます。
削除はできるのですが、リダイレクト先で削除したデータを取得しようとしてエラーが出ているようです。
ActiveRecord::RecordNotFound in BooksController#show
Couldn't find Book with 'id'=15
Extracted source (around line #21):
19
20 def show
21 @book = Book.find(params[:id])
22 @user = @book.user
23 end
24
'id'=15は削除した投稿なのでこちらを読み込まないようにするのはどうすれば良いでしょうか？
今ってredirect_toはどのようにしていますか？
redirect_to book_url(book.id)
これが記述してある限りエラーになります。これを消した上で
redirect_to books_url
などにしてください。
現在は記述しなおしてredirect_to books_urlとなっております。
エラーはコメントしたCouldn't find Book with 'id'=15が出ております。
削除→リダイレクトの過程で発生していると思われます。
ActiveRecord::RecordNotFound in BooksController#show
これの意味は，deleteしたあとにshowアクションを起こそうとしています。しかし消したものを表示しようとしてもできるはずがないのでエラーが出ています。
なぜdeleteしたあとにshowが動いてしまっているかというと，redirect_to book_url(book.id)の意味が「GET /books/:id(.:format) 」だからです。
もう一度削除画面のところから別のidのbookを消してみても同じエラーが出てしまいますか…？
redirect_to action: 'index'
にしてみたら解決するかもしれません
確認した所、リダイレクト先を誤って設定しており、正しくはredirect_to books_urlでした。
するとリダイレクト先でエラーにはなりましたが移動と削除はできていたため解決できたと思われます。
お時間頂きありがとうございました。
あなたの回答
tips
プレビュー
