renderを使いerror_messageを表示したい

Question

renderを使いerror_messageを表示

保存に失敗した際にrenderを使いerror_messageを表示したいのですが、以下のエラーメッセージが発生しました。いくつか方法を試してみましたがうまくいかず、ご教授願いたいです。

NoMethodError in Books#create

ActionView::Template::Error (undefined method `each' for nil:NilClass):
     9:   	  </tr>
    10:   	</thead>
    11:   	<tbody>
    12:   	  <% @books.each do |book| %>
    13:   	    <tr>
    14:   	      <td><%= book.title %></td>
    15:   	      <td><%= book.body %></td>

該当のソースコード

class BooksController < ApplicationController
  def top
  end

  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 book_path(book.id)
  	else
       render :index
       # renderだとうまくいかない？indexの繰り返し処理の@booksがnilになっている？
       # redirect_toにするとindexに遷移するようになったが、error_messageが表示されない
  	end
  end

  def show
  	@book = Book.find(params[:id])
  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 book_path(book.id)
  	else
  		render :show
      # ここもredirect_toで指定して遷移させればいいのか？
  	end
  end

  def destroy
  	book = Book.find(params[:id])
  	book.destroy
  	flash[:notice] = "Book was successfully destroyed."
  	redirect_to books_path
  end

  private
  def book_params
  	params.require(:book).permit(:title, :body)
  end
end

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

<%= form_for(@book) do |f| %>
    <% if @book.errors.any? %>
      <div id="error_explanation">
        <h2><%= @book.errors.count %> errors prohibited this book from being saved:</h2>
        <ul>
          <% @book.errors.full_messages.each do |message| %>
            <li><%= message %></li>
          <% end %>
        </ul>
      </div>
    <% end %>
    <div class=field>
  	  <label for="book_title">Title</label>
  	  <%= f.text_field :title %>
    </div>
    <div class="field">
      <label for="book_body">Body</label>
      <%= f.text_area :body %>
    </div>
    <%= f.submit "Create Book" %>
<% end %>

 

試したこと

createアクションにて保存に失敗した際の処理としてredirect_toを使うと、index.html.erbへ遷移することは確認できましたが、それだと空のモデルが渡されるためにフォームに入力前の状態に戻る？
each文の@booksがnilのためrenderの前に定義が必要なのかと思い、@books = Book.newを入れてみたがうまくいかない（←これが間違っているのか？）

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Answer 1

create と update にて
else @books = Book.all render :index end
としてください

Comments

[komai]

ご回答ありがとうございます。
助言いただいた通りに記述し、併せてcreateアクションとupdateアクションで定義しているインスタンス変数に@を追加したところうまくいきました。恐らくviewで表示しなければならないので@が必要なんですね。
初歩的なところなのでしょうが、理解できました。ありがとうございました！

[winterboum]

今回の本質は viewで使っている @booksが定義されていなかったことです。
@books=Book.newでも定義されているとは言えますが、、、
viewで必要としているのは Bookの登録済のものの集まりなので、Book.new（ひとつだけ）では eachでエラーになります

[komai]

elseの下で再度@books=book.allでデータを取得しなければ、index.html.erbのeach文のところが表示できない。なぜならeach文では全てのデータを必要としているから、Book.new（ひとつだけ）ではダメということですね。

[winterboum]

はい
正確には
一つだからだめ　なのではなく
一つしか入れられない入れ物だから　だめ
です。
Book.allした結果一つも無くても、複数入れられる入れ物なので OK　です

[komai]

なるほど、参考になります！
正確な部分まで教えていただきありがとうございました！