質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.50%
DELETE

ファイルシステムからファイル、データベースからレコードを削除することまたはメモリ内のオブジェクトの割り当てを取り消すことをさします。もしくは、HTTPプロトコルのDELETEを指すこともあります。

Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

Q&A

解決済

1回答

659閲覧

deleteでCouldn't find Book with 'id'=13

退会済みユーザー

退会済みユーザー

総合スコア0

DELETE

ファイルシステムからファイル、データベースからレコードを削除することまたはメモリ内のオブジェクトの割り当てを取り消すことをさします。もしくは、HTTPプロトコルのDELETEを指すこともあります。

Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

0グッド

0クリップ

投稿2019/12/20 07:57

投稿を削除する際に以下のエラーが発生しました。

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' %>

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

退会済みユーザー

退会済みユーザー

2019/12/20 08:00

id=13のBookは存在してますか? 一度データベースを確認してみてください
winterboum

2019/12/20 08:04

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からですか?
退会済みユーザー

退会済みユーザー

2019/12/20 08:07

回答頂きありがとうございます。 確認した所、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 おそらく削除はできていますが、画面の移動がうまくいっていないと考えていますが、どうでしょうか?
退会済みユーザー

退会済みユーザー

2019/12/20 08:09

<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からのエラーです。
退会済みユーザー

退会済みユーザー

2019/12/20 08:37 編集

タイトルのエラーとNameErrorは別問題ですね。 まずタイトルの「Couldn't find Book with 'id'=13」は、単純にid=13のデータが存在しなかったので、 存在しないものは削除できない、というエラーです。 NameErrorの方は変数「book」が見つからない、というエラーです。 (@bookとbookは別物です。destroyアクションの中で定義されているのは@bookであり、bookではないですよね)
退会済みユーザー

退会済みユーザー

2019/12/20 08:26

あと、削除してからのページ移動の流れが気になるのですが、 Bookを削除した後、削除したBookのページに移動するって辻褄が合わないような…
退会済みユーザー

退会済みユーザー

2019/12/20 12:16

本当ですね。 Bookを消した後はredirectで指定しているので間違いです。 申し訳ありませんでした。 解決できなければ別で質問してみます。 ありがとうございました。
guest

回答1

0

ベストアンサー

redirect_to book_url(book.id)

ここがエラーの原因です。
削除したあとどのページを表示したいのでしょうか?

ホーム画面にリダイレクトしたいのであれば

redirect_to "/"

book一覧にリダイレクトしたいのであれば

redirect_to books_url

などです(表記方法は色々ありますが)

投稿2019/12/20 12:18

k-810

総合スコア192

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

退会済みユーザー

退会済みユーザー

2019/12/20 13:23

ご回答頂きありがとうございます。 削除はできるのですが、リダイレクト先で削除したデータを取得しようとしてエラーが出ているようです。 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は削除した投稿なのでこちらを読み込まないようにするのはどうすれば良いでしょうか?
k-810

2019/12/20 14:19

今ってredirect_toはどのようにしていますか? redirect_to book_url(book.id) これが記述してある限りエラーになります。これを消した上で redirect_to books_url などにしてください。
退会済みユーザー

退会済みユーザー

2019/12/20 14:24

現在は記述しなおしてredirect_to books_urlとなっております。 エラーはコメントしたCouldn't find Book with 'id'=15が出ております。 削除→リダイレクトの過程で発生していると思われます。
k-810

2019/12/20 14:25

ActiveRecord::RecordNotFound in BooksController#show これの意味は,deleteしたあとにshowアクションを起こそうとしています。しかし消したものを表示しようとしてもできるはずがないのでエラーが出ています。 なぜdeleteしたあとにshowが動いてしまっているかというと,redirect_to book_url(book.id)の意味が「GET /books/:id(.:format) 」だからです。
k-810

2019/12/20 14:27

もう一度削除画面のところから別のidのbookを消してみても同じエラーが出てしまいますか…?
k-810

2019/12/20 14:29

redirect_to action: 'index' にしてみたら解決するかもしれません
退会済みユーザー

退会済みユーザー

2019/12/20 16:08

確認した所、リダイレクト先を誤って設定しており、正しくはredirect_to books_urlでした。 するとリダイレクト先でエラーにはなりましたが移動と削除はできていたため解決できたと思われます。 お時間頂きありがとうございました。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問