railsのバリデーション

Question

前提・実現したいこと

ここに質問の内容を詳しく書いてください。
（例）ruby on railsで本の感想を追加できるようなシステムを作っています。
バリデーションの機能を実装中に以下のエラーメッセージが発生しました。

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

該当のソースコード

Rails.application.routes.draw do
  # For details on the DSL available within this file, see http://guides.rubyonrails.org/routing.html
  get '' => 'homes#top'
  get 'books' => 'books#index'
  post 'books' => 'books#create'
  get 'books/:id' => 'books#show', as: 'book'
  get 'books/:id/edit' => 'books#edit', as: 'edit_book'
  
  patch 'books/:id' => 'books#update', as: 'update_book'
  get 'books/:id/update' => 'books#update'
  delete 'books/:id' => 'books#destroy', as: 'destroy_book'

end

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

class BooksController < ApplicationController

  def new

  end

  def index
    @books = Book.all
    @book = Book.new
  end

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

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

  

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



  def create
    
    
    
    book = Book.new(book_params)
    if book.save
      flash[:success] = "Book was successfully created."
      redirect_to "/books/#{book.id}"
    else
      # flash.now[:danger] = "Failed to save."
      #render "/books"
      render :index
    end
  end
  
  
  
  

  def update
    book = Book.find(params[:id])
    
    
    if book.update(book_params)
      flash[:success] = "Book was successfully updated."
      redirect_to "/books/#{book.id}"
    else
      flash.now[:danger] = "Failed to update."
      render "/books"
    end
  end



  private

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

end

    <div class="notifications">
        <% flash.each do |message_type, message| %>
            <%= message %>
        <% end %>
    </div>
   

    <h1>Books</h1>

    <!-- 一覧表示 -->
    <table style="table-layout: fixed;">

        <tr>
            <th>Title</th>
            <th>Body</th>
            <th colspan="3"></th>
        </tr>


        <% @books.each do |book| %>
            <tr>
                <td><%= book.title %></td>
                <td><%= book.body %></td>
                <!--<td><a class="show" href="/books/">Show</a></td>-->
                <td><%= link_to "Show", "/books/#{book.id}" %></td>
                <td><%= link_to "Edit", "/books/#{book.id}/edit" %></td>
                <td><%= link_to "Destroy", destroy_book_path(book.id), method: :delete, "data-confirm" => "Are you sure?" %></td>
            </tr>
        <% end %>
    </table>


    <% if @book.errors.any? %>
        <%= @book.errors.count %>件のエラーが発生しました
        <% @book.errors.full_messages.each do |message| %>
            <%= message %>
        <% end %>
    <% end %>
    
    <!-- 新規投稿 -->
    <%= form_with model:@book, url:'/books', local:true do |f| %>
        <div class="new-book">
            <h2>New book</h2>
            <p>Title</p>
            <%= f.text_field :title %>
            <p>Body</p>
            <%= f.text_area :body %>
            <div>
                <%= f.submit 'Create Book' %>
            </div>
        </div>
    <% end %>

試したこと

ﾌﾟﾛｸﾞﾗﾑを目で追ってみましたが、わかりません。
@booksがnilになっていてエラーをはいてると思うのですが、参考にさせていただいた他の方は出ていないように思います。

参考資料

https://qiita.com/clbcl226/items/af336ff06bfba4dacfd9

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

ＡＷＳ
windows
rails -v 5.2.5

Comments

[maisumakun]

エラーメッセージ中に「23行目」と書かれていますが、どこが23行目かはわかりますか？

[ttt452]

説明なく、申し訳ありません。
indexの23行目
<% @books.each do |book| %>
です！

Answer 1

render :indexする以上は、@booksが必要です。

参考にさせていただいた他の方は出ていないように思います。

通常はindexに新規作成の画面を一緒につけることはない（し、ましてやindexつきのビューでバリデーション失敗時の再描画を行うこともない）ので問題となりません。

Comments

[ttt452]

回答ありがとうございます。
createアクションで、
@books = Book.all
を記述すればいいのでしょうか？

試してみたところ今度は、index.htmlで、
38行目：<% if @book.errors.any? %>
エラー：undefined method `errors' for nil:NilClass
というようなエラーが出てしまいました。

が、解決しました。一個目のエラーと同じですね。
@bookと@booksをアクションで定義してあげたらいけました！
routingから呼び出されるindexアクションと
createのrenderから呼び出されるindexアクションは生成のされかたが違うから、ちゃんと定義してあげないといけないってことですよね！？
(エラーである空データも扱うため)

[maisumakun]

> routingから呼び出されるindexアクションと
createのrenderから呼び出されるindexアクションは生成のされかたが違うから、ちゃんと定義してあげないといけないってことですよね！？

まあ、そうではあるのですが、再入力の際に一覧リストは余計でしかないので、専用の画面を作ったほうが適切かとは思います（編集画面とフォームを共通化すればそう手間にはなりません）。

[ttt452]

なるほど。。。次回からはそこのところ意識してやってみます！
今回は課題としてnewアクション封印されているので。。。

丁寧な説明、感謝いたします！

[maisumakun]

> 今回は課題としてnewアクション封印されているので。。。

課題の設定そのものが良くない感じはしますね。