バリデーションチェックにより、投稿・編集が失敗した場合にエラーメッセージを表示させたい。

前提・実現したいこと

本の投稿に関するモデル名は、Bookとしてください。
Bookモデルのカラム名は、以下のように設定してください。
title：本のタイトル
body：感想
読んだ本のタイトルと、その本についての感想を投稿・編集・削除ができるようにしてください。
画面は下記の4つを作成してください。
アプリケーショントップ画面
投稿一覧画面：投稿された本の感想を一覧表示する画面
投稿詳細画面：投稿された本の感想を個別に表示する画面
投稿編集画面：投稿された本の感想を編集できる画面

バリデーションチェックにより、投稿・編集が失敗した場合にエラーメッセージを表示させたい。
バリデーションの実装後、エラーメッセージを表示するためにindex.html.erbのフォーム内に記入したのですが、render :indexで一覧画面に返すとエラーが起きます。（投稿と一覧の機能を同じページに置いてます）

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

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

undefined method `each' for nil:NilClass
Extracted source (around line #11):
9
10
11
12
13
14
              
  </thead>
  <tbody>
    <% @books.each do |book| %>
    <tr>
      <td><%= book.title %></td>
      <td><%= book.body %></td>

Rails.root: /home/ec2-user/environment/Bookers

Application Trace | Framework Trace | Full Trace
app/views/books/index.html.erb:11:in `_app_views_books_index_html_erb__1234319390587803345_70279925734160'
app/controllers/books_controller.rb:19:in `create'
Request
Parameters:

{"utf8"=>"✓", "authenticity_token"=>"u9x0dKd33/CWrHwK4EytBk9Km3mL+8gUDSih+Y+yNadziU2ZYxynMeZakFpU0K+34LdEfPFKfZ70uVTXMEe8cw==", "book"=>{"title"=>"", "body"=>""}, "commit"=>"Create Book"}

該当のソースコード

class BooksController < ApplicationController
  def top

  end

  def index
    @books = Book.all.order(created_at: :desc)
    @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) #セーブできた時
    else
      render :index
    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 :edit
    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

<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) %><%= link_to "Edit", edit_book_path(book) %><%= link_to "Destroy", book_path(book), method: :delete, "data-confirm" => "Are you sure?" %></td>
    </tr>
    <% end %>
  </tbody>
</table>

<h2>New book</h2>

<%= form_with model:@book, local:true do |f| %>

<% if @book.errors.any? %>
  <%= @book.errors.count %> prohibited this book from being saved: %>
    <ul>
      <% @book.errors.full_messages.each do |message| %>
        <li><%= message %></li>
      <% end %>
    </ul>
  </div>
<% end %>

<h3>title</h3>
<%= f.text_field :title %>
<h3>body</h3>
<%= f.text_area :body %>
<%= f.submit "Create Book" %>
<% end %>

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

試したこと

render先を、仮にnewにすると、new.html.erb上では反映されていました。
今回の実装は、投稿画面と一覧画面を同じにしたので、render先はindexにしています。
書いてある事は色々試したのですが、どれも上手くいかなかったので原因を教えて欲しいです。

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

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

行動規範の内容に同意します

回答2件

ベストアンサー

indexをrenderするときのコントローラーはshowアクションですよね？

showアクション内で@booksを定義しておかないと、当然ですがindexをrenderしたときに@booksを呼び出せません

それと、showアクション内でbook_pathの引数にbookを渡していますが、@bookじゃないとエラーになると思われます

投稿2020/10/09 15:13

kokitail

総合スコア135

wrx_sti

2020/10/11 10:20

indexをrenderするときのコントローラーはcreateアクションです。index.htmlで新規投稿した内容を、createアクションで保存できなかった時にindex.htmlに戻したいのです。ご指摘ありがとうございます！おそらくcreateアクション内のbook_pathの引数ですよね？@をつけ忘れていました！