Ruby on Rails でバリデーション機能を実装する

Question

前提・実現したいこと

投稿サイトを作っています。
現在バリデーション機能を実装し、新規投稿されたものが空白だった場合にエラー分を
表示させる機能を作っています

発生している問題・エラーメッセージ

<% @books.each do |book| %>にエラーが発生しています

NoMethodError in Books#create
Showing /home/ec2-user/environment/bookers/app/views/books/index.html.erb where line #18 raised:

undefined method `each' for nil:NilClass
Extracted source (around line #18):
    </thead>
<tbody>
<% @books.each do |book| %>

    <tr>
    <td><%=book.title%></td>


   
   

該当のソースコード

class BooksController < ApplicationController
  def index
    @books = Book.all
    @book = Book.new
  end

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


  def create
    book = Book.new(book_params)
    if book.save
    redirect_to book_path(book.id)
    else
    render action: :index
    end
    flash[:notice] = "Book was successfully created."
  end

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

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

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


  private

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

end

<%if flash[:notice]%>
<p id = "notice">
    <%=flash[:notice]%>
</p>
<%end%>

<h1>Books</h1>

<table>
    <thead>
    <tr>
        <th>Title</th>
        <th>Body</th>
        <th></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)%>
        <%=link_to"Edit",edit_book_path(book.id)%>
        <%=link_to"Destroy","/books/#{book.id}",method: :delete,"data-confirm" => "Are you sure?" %>

    </td>
    </tr>
    <%end%>
    <tbody>
<table>

<h2>New Book</h2>
<%= form_with model:@book,local:true do |f| %>

<div class ="field">
<label for = "book_titile">Title</label>
<%= f.text_field:title %>
</div>

<div class ="field">
<label for = "book_body">Body</label>
<%= f.text_area:body %>
</div>

<div class = "actions">
<%= f.submit 'Create Book' %>
</div>
<%end%>

試したこと

色々調べ考えたのですが分からず質問している次第です

補足情報（FW/ツールのバー）

Rails 5.2.6

Answer 1

create で render action: :index する前に @books を設定してないので
@books が nil になり、undefined method `each' for nil:NilClass

また index.html.erb では @book も参照しているのでインスタンス変数にします。

def create
  @book = Book.new(book_params)
  if book.save
    flash[:notice] = "Book was successfully created."
    redirect_to book_path(@book.id)
  else
    @books = Book.all
    render action: :index
  end
end

Comments

[uruman]

回答有難うございます。@books = Book.allを追加したところ無事動きました！
質問なにですが、@books = Book.allがなかったため実装したいエラ分を受け取ることが
出来なかったという認識でよろしのでしょうか？

[neko_daisuki]

バリデーションエラーの内容は @book に入っています。
@books は取得しているだけなのでバリデーションは関係ありません。

@book.save して、バリデーションにひっかかると、@book.errors にエラーの内容が追加されます。
view ではそれを表示する必要があります。

[uruman]

なるほど。ご丁寧に有難うございました。助かりました！