【Ruby on Rails】NoMethodErrorへの対処方法

Question

バリデーションを作成している最中なのですが、NoMethodErrorの原因がわかりません。

shutaコントローラです↓

class ShutaController < ApplicationController
  def new
    @book = Book.new
  end
  
  
  def index
    @books = Book.all
  end
  
  def show
    @book = Book.find(params[:id])
  end
  
  def create
    @book = Book.new(book_params)
    if @book.save
      redirect_to shutax_path(@book.id)
      flash[:notice] = "Book was successfully created"
    else
      render :new
    end
  end
  
  def edit
    @book = Book.find(params[:id])
  end
  
  def update
    book = Book.find(params[:id])
    book.update(book_params)
    redirect_to shutax_path(book.id)
    flash[:notice] = "Book was successfully updated"
  end
  
  def destroy
    book = Book.find(params[:id])
    book.destroy
    redirect_to index_path
  end
  
  
  
  private
  
  def book_params
    params.permit(:title, :body)
  end
  
end

index.html.erbのコードです↓

<h1>Books</h1>

<table class="table">
  <thead>
    <tr>
      <th>Title</th>
      <th>Body</th>
      <th colspan="3"></th>
    </tr>
  </thead>
  <tbody>
    <% @books.each do |book| %>
     <tr>
       <td>
        <%= book.title %>
       </td>
       <td>
        <%= book.body %>
       </td>
       <td>
         <%= link_to "Show", shutax_path(book.id) %>
         <%= link_to "Edit", edit_shutax_path(book.id) %>
         <%= link_to "Destroy", destroy_shutax_path(book.id), method: :delete %>
       </td>
     </tr>
    <% end %>
  </tbody>
</table>

<h1>New book</h1>

<% 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| %>
  <h4>Title</h4>
  <%= f.text_field :title %>
  <h4>Body</h4>
  <%= f.text_area :body %>
  <%= f.submit 'Create Book' %>
<% end %>

エラーの内容です↓

shutaコントローラのindexアクションに問題があるのは分かるのですが、@booksや@bookに変更してみてもエラーが消えません。
なにが原因なのか、ご教授お願いします。

Answer 1

新規作成するために、@bookを定義する必要があります。

@book = Book.new

Comments

[maisumakun]

もっとも、入力にエラーがあった場合の再表示はcreateが行う以上、indexサイドのエラー表示が有効となる場面はなさそうですので、エラー表示処理ごと外してもいいかもしれません。

[shuta-a]

ありがとうございます。ご指摘通りindexアクションで@book変数の定義して色々変えたら、エラーは消えました。

Answer 2

今回は変数名のミスかなと。

コントローラでは、@booksにレコードを代入しています。

def index
  @books = Book.all
end

エラーとなったビューの行では、

<% if @book.errors.any? %>

と書いています。
@booksは存在するが、@bookは存在しないのでnilとなり、nilにはerrorsというメソッドはないのでエラーとなります。
なので、@bookではなく、@booksに修正したら解決するのではないでしょうか・

Comments

[shuta-a]

ご指摘いただいた通り、@booksに直したのですが、同じくerrorsのところでエラーが出てしまいます。
indexアクションに@bookを定義するのも考えましたが、どのようなメソッドを付けて定義すればよいのか悩んでおります。