NoMethodErrorの解決法について

Question

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

現在本の投稿アプリを作成中なのですが以下のようなエラーが発生し、解決することができません。
どなたかご教授いただけませんでしょうか。

NoMethodError in Books#show
Showing /home/vagrant/work/bookers_new/app/views/books/show.html.erb where line #24 raised:
undefined method `name' for nil:NilClass

 <tr>
 					<td>name</td>
 					<td><%= @book.user.name %></td>/この部分が赤くなっています
 				 </tr>

該当のソースコード

app/views/books/show.html.erb

<div class="flash">
<% flash.each do |key, value| %>
  <%= content_tag(:div, value, class: "#{key}") %>
<% end %>
</div>


<div class="container">
 <div class="row">
 	<div class="col-lg-3" style="background-color: white">

 		<h2>User info</h2>

 		<%= attachment_image_tag @user, :profile_image, :fill, 100, 100, fallback: "/assets/sample_image.gif" ,format: '/assets/sample_image.gif' %>

 		<table class="table table-hover">

 			<tbody>

 				 <tr>
 					<td>name</td>
 					<td><%= @book.user.name %></td>
 				 </tr>

 				 <tr>
 					<td>Introduction</td>
 					<td><%= @book.user.introduction %></td>
 				 </tr>

 			</tbody>

 		</table>

<%= link_to(edit_user_path(@book.user.id), class:  "btn btn-default") do %>
		<span class="glyphicon glyphicon-wrench" aria-hidden="true"></span>
<% end -%>

<h1>New book</h1>

<%= form_for(@book) do |f| %>

  <p>Title</p>
  <%= f.text_field :title , class: "edit-title"%>

  <p>Body</p>
  <%= f.text_area :body, class: "edit-body"%>
  <br>
  <%= f.submit 'Create Book', class: "btn btn-primary" %>

<% end %>

 	</div>

<div class="col-lg-9">
 <h1>Book detail</h1>

	<table class="table table-striped">
		<thead>
		</thead>
		<tbody>
		  <div class="flash">
              <% flash.each do |key, value| %>
              <%= content_tag(:div, value, class: "#{key}") %>
              <% end %>
          </div>


               <table class="table table-hover">
	           <thead>
	           </thead>
	 <tbody>
		  <tr>
			  <td><span><%= attachment_image_tag @book.user, :profile_image, :fill, 50, 50, format: 'jpeg' %></span></td>
			  <td><%= @book.title %></td>
			  <td><%= @book.body %></td>

			   <!-- もしも、ログインしているユーザーと、投稿したユーザーが合致したら編集と削除をするボタンをだします --> 
			 <% if current_user.id == @book.user_id %>
				<td><h4><span class="label label-success"><%= link_to "edit", edit_book_path(@book.id) %></span></h4></td>
				<td><h4><span class="label label-danger"><%= link_to "destroy", book_path(@book), method: :delete, "data-confirm" =>"Are you sure？" %></span></h4></td>
			 <% else %>
				<td></td>
				<td></td>

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

class BooksController < ApplicationController
    before_action :authenticate_user!

  

  def create
  	    @book = Book.new(book_params)
        @book.user_id = current_user.id
  	   [e-mail]
  	    redirect_to book_path(@book.id)
       else
        @books = Book.all
        render :show
       end
   end

  def index
    @books = Book.all
  	@books = Book.page(params[:page]).reverse_order
    @book = Book.new

  end

  def show
  	@book = Book.find(params[:id])
    @books = Book.all
    @book = Book.new

  end

  def usershow
    @book = Book.find(params[:id])
    @books = Book.all
    @book = Book.new
  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 update"
  end

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

  def userinfo
    @book = Book.new
  end

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

book.rb

class Book < ApplicationRecord
    attachment :profile_image
	belongs_to :user

	validates :title, presence: true
    validates :body , presence: true
    validates :body , length: { maximum: 200 }
end

Answer 1

@teasy さん、こんにちは。
エラーの原因については @naokit-dev さんとおなじ意見です。

Railsでは、scaffold_controllerで生成されるコードが理想とされています。
Rails ジェネレータとテンプレート入門 - Railsガイド

rails g scaffold_controller books で作成されるControllerクラスが参考になるとおもいます！

手前味噌ですが、scaffoldに近い形に添削をしてみましたので、ご参考まで。

class BooksController < ApplicationController
  before_action :authenticate_user!
+ before_action :set_book %i[show edit update destroy]
  
  def create
-   @book = Book.new(book_params)
-   @book.user_id = current_user.id
+   @book = current_user.books.build(book_params)
    [e-mail]
      redirect_to book_path(@book.id)
    else
-     @books = Book.all
-     render :show
+     render :new
    end
  end

  def index
-   @books = Book.all
    @books = Book.page(params[:page]).reverse_order
-   @book = Book.new
  end

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

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

  def update
-   book = Book.find(params[:id])
-   book.update(book_params)
+   if @book.update(book_params)
      redirect_to book_path(book.id)
      flash[:notice] = "Book was successfully update"
+   else
+     render :edit
+   end
  end

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

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

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

Answer 2

追記(最終)

-    @book = Book.new #ココ

---

@bookがNillなためにnillに対してnameというメソッドはないよというエラーだと思います

以下が問題だと思います

  def show
      @book = Book.find(params[:id])
    @books = Book.all
    @book = Book.new #ココ

  end

すいません考えたのですがよくわかりません、

敢えて継続して回答いたしますが

book = 1
book = 0

の結果bookに何が入るかわかりますか

加えて

rails c

railsコンソールにて

book = Book.new
book.user

とすると何が表示されますか

最初の回答をより丁寧にすると
@book.userがnilだからnillに対してnameというメソッドはないよというエラー
といえます

申し訳無いのですが、これ以上わかりやすく説明できる自信はないです

Comments

[teasy]

まだrails初心者でこのような質問大変申し訳ないのですが、この後どのようにすればよいのでしょうか。

[naokit-dev]

showアクションが呼ばれた結果、
@bookに何が入るか考えれば解決すると思います

[teasy]

すいません考えたのですがよくわかりません、