バリデーションを使って新規投稿時と編集時、空欄で投稿した時にエラーメッセージを表示させたい

Question

前提・実現したいこと

Rails勉強中のプログラミング初心者です。
新規投稿・閲覧・編集・削除などができる簡単なアプリケーションを作成して練習しています。
バリデーションを使って新規投稿時と編集時、空欄で投稿した時にエラーメッセージを表示させたいです。

新規投稿欄に空白で投稿すると下記のエラーが出てしまいます。
編集欄に空白で投稿するとボタンは押されるのですがエラーすら出ず、何も反応がありません。

エラーメッセージ

NoMethodError in Books#create
undefined method `each' for nil:NilClass

Request
Parameters:
{"utf8"=>"✓", "authenticity_token"=>"6UH9Tw6ev8ONJ5NbeXL3PzGOwEjQpU+ZVLm/P1AAkVogavNeOuSFSM2qkCz4neTcd1ejabAMRA/ebV5CrgMg0w==", "book"=>{"title"=>"", "body"=>""}, "commit"=>"Create Book"}

###index.html.erb

<% if flash[:notice] %>
 <div class="flash">
  <%= flash[:notice] %>
 </div>
<% end %>

<h1>Books</h1>
<table>
  <tr>
   <th>title</th>
   <th>body</th>
   <th></th>
  </tr>

 <% @books.each do |book| %>
  <tr>
   <td><span><%= book.title %></span></td>
   <td><span><%= book.body %></span></td>
   <td><%= link_to "Show", show_book_path(book.id) %></td>
   <td><%= link_to "Edit", edit_book_path(book.id) %></td>
   <td><%= link_to "Destroy", destroy_book_path(book.id), method: :delete, "data-confirm" => "Are you sure?" %></td>
  </tr>
 <% end %>
</table>

 <h3>New book</h3>
 <!-- form_with部分 -->
 <%= form_with model: @book, url: "/books/#{@book.id}", local:true do |f| %>
  <% if @book.errors.any? %>
     <h3><%= @book.errors.count %>errors prohibited this book from being saved:</h3>
     <ul>
      <% @book.errors.full_messages.each do |message| %>
      <li><%= message %></li>
      <% end %>
     </ul>
  <% end %>

  <h4>Title</h4>
  <%= f.text_field :title %>
  <h4>Body</h4>
  <%= f.text_area :body %>
  <%= f.submit 'Create Book' %>
 <% end %>

book.rb

class Book < ApplicationRecord
 validates :title,
   presence: true
 validates :body,
   presence: true
end

###books_controller.rb

class BooksController < ApplicationController
 def index
   @books = Book.all
   # 投稿一覧
   @book = Book.new
   # 新規投稿
 end

 def create
  @book = Book.new(book_params)
  if @book.save
   flash[:notice] = "Book was successfully created."
   redirect_to show_book_path(@book)
  else
   #書き込み失敗
   render :index
  end
 end

 def show
  @book = Book.find(params[:id])
 end

 def new
 end

 def edit
  @book = Book.find(params[:id])
 end

 def update
  book = Book.find(params[:id])
  if book.update(book_params)
   flash[:notice] = "Book was successfully updated."
  redirect_to show_book_path(book.id)
  end
 end

 def destroy
  book = Book.find(params[:id]) #データ(レコード)を1件取得
  if book.destroy #データ（レコード）を削除
   flash[:notice] = "Book was successfully destroyed."
   redirect_to books_path(book.id)
  end
 end

 private
  #ストロングパラメータ
 def book_params
  params.require(:book).permit(:title, :body)
 end
end

###edit.html.erb

<h1>Editing Book</h1>
<!-- form_with部分 -->
<%= form_with model: @book, url: "/books/#{@book.id}", local:true do |f| %>
 <h4>Title</h4>
 <%= f.text_field :title %>
 <h4>Body</h4>
 <%= f.text_area :body %>
 <%= f.submit 'Update Book' %>
 <%= link_to "Show", show_book_path(@book) %>
 <%= link_to "Back", index_book_path(@book) %>
<% end %>

###show.html.erb

<% if flash[:notice] %>
 <div class="flash">
  <%= flash[:notice] %>
 </div>
<% end %>

<h2>Title:</h2>
<p><%= @book.title %></p>
<h2>Body:</h2>
<p><%= @book.body %></p>
<%= link_to "Edit", edit_book_path(@book) %>
<%= link_to "Back", "/books" %>

new.html.erb

<h1>New Book</h1>
<!-- form_with部分 -->
<%= form_with model:@book, url: '/books', local:true do |f| %>
 <h4>Title</h4>
 <% f.text_field :title %>
 <h4>Body</h4>
 <%= f.text_area :body %>
 <%= f.submit 'Create Book'%>
<% end %>

routes.rb

Rails.application.routes.draw do
 # For details on the DSL available within this file, see http://guides.rubyonrails.org/routing.html
  root to: 'homes#top'
  get 'books' => 'books#index', as: 'index_book'
  post 'books' => 'books#create'

  get 'books/:id' => 'books#show', as: 'show_book'
  get 'books/edit' => 'books#edit'
  get 'books/:id/edit' => 'books#edit', as: 'edit_book'
  patch 'books/:id' => 'books#update', as: 'update_book'

  delete 'books/:id' => 'books#destroy', as: 'destroy_book'
end

---

undefined method `each' for nil:NilClassから、nil な何かには「each」メソッドがないから、実行できないと言われており、今回の場合は

<% @books.each do |book| %>

なので、eachの前の＠booksがnilということだと思うのですが、編集欄の**'Update Book'**ボタンが反応しないことも含め、ここからどう解決すればよいのか分かりません。

説明不足で申し訳ありませんが、よろしくお願いいたします。

Answer 1

newで失敗した場合 indexに飛んでます。　indexでは　@books を each していますが、それが未定義なのでエラーになってます。render の前に@books = Book.all をつけましょう。
ただ、多くのアプリではsave失敗した時は new に戻って入力させ直します。
また、newに戻ったにしても載っている newのviewにはエラー表示のcedeがないので、このままではエラー表示はありません。

「編集欄に空白で投稿するとボタンは押されるのですがエラーすら出ず、何も反応がありません。」
というのが　？？　です。　templateがないというエラーになると思われるのですが。
logを確認したいです

追記
「No template found for BooksController#update, rendering head :no_content
Completed 204 No Content in 25ms (ActiveRecord: 0.2ms)」
やっぱり出てますね。
else render :edit end
を追加しましょう

Comments

[veludo]

ご回答いただきありがとうございます。
説明不足で申し訳ございません。
一覧と新規投稿の画面は同じ画面に表示してあります。
Renderの前に@books = Book.allをつけたことで新規投稿欄にエラー表示されるようになりました。

編集欄に空白で投稿した際のLogはこのように表示されております。
development.log
Started PATCH "/books/40" for 60.41.22.240 at 2021-03-20 03:50:52 +0000
Cannot render console from 60.41.22.240! Allowed networks: 127.0.0.1, ::1, 127.0.0.0/127.255.255.255
Processing by BooksController#update as HTML
Parameters: {"utf8"=>"✓", "authenticity_token"=>"wfQIz1n/YI53SVnjVam6WlmILzUhuXHvp8zNbpwJDcB+KuTbWDFlN9b7LC/CCbB5NEMN6LV2WLkBHh3UR34NLQ==", "book"=>{"title"=>"", "body"=>""}, "commit"=>"Update Book", "id"=>"40"}
Book Load (0.1ms) SELECT "books".* FROM "books" WHERE "books"."id" = ? LIMIT ? [["id", 40], ["LIMIT", 1]]
↳ app/controllers/books_controller.rb:34
 (0.1ms) begin transaction
↳ app/controllers/books_controller.rb:35
 (0.0ms) rollback transaction
↳ app/controllers/books_controller.rb:35
No template found for BooksController#update, rendering head :no_content
Completed 204 No Content in 25ms (ActiveRecord: 0.2ms)

[veludo]

ありがとうございます。
else
     render :edit
   end
を追加したら、今度は　undefined method `id' for nil:NilClass　のエラーが出てしまいました。
抽出されたソースはedit.html.erbの3行目あたりと表示されています。
<h1>Editing Book</h1>
<!-- form_with部分 -->
<%= form_with model: @book, url: "/books/#{@book.id}", local:true do |f| %>
<h4>Title</h4>
<%= f.text_field :title %>

　url: "/books/#{@book.id}”のidに問題がありそうなのですが、解決方法が絞れません。
ご教示いただけますと幸いです。

[winterboum]

def update
@book = Book.find(params[:id])
ですね・
このくらいは自力してよ

[veludo]

ありがとうございます。解決しました。
お手数おかけして申し訳ありませんでした。