undefined method `new_record?' for nil:NilClassを解決したいです。

Question

undefined method `new_record?' for nil:NilClassを解決したいです。

新規投稿画面と編集画面を部分テンプレートで作成し
submitボタンの表示する文字をファイルによって変えたいです。新規投稿時はCreateBook、編集の時はUpdateBookが表示されるようにしたいです。

その際あるサイト(https://teratail.com/questions/312875)を参考に

_form.html.erb

<% if f.object.new_record? %>
      <%= f.submit "Create Book", class: "btn btn-success" %>
    <% else %>
      <%= f.submit "Update Book", class: "btn btn-success" %>
    <% end %>

を記述した所、userのshow.html.erb(userの詳細ページ)でエラーが発生しました

(サイトにあるようにnew_record?メソッドを自分で定義する必要は無いと思います)
(新規投稿のページや投稿を編集するページではエラーは発生していません)

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

NoMethodError in Users#show
Showing /home/ec2-user/environment/bookers2-debug/app/views/books/_form.html.erb where line #21 raised:

undefined method `new_record?' for nil:NilClass
Extracted source (around line #21):
19
20
21
22
23
24

<% if f.object.new_record? %>
   <%= f.submit "Create Book", class: "btn btn-success" %>
 <% else %>
   <%= f.submit "Update Book", class: "btn btn-success" %>

Trace of template inclusion: app/views/users/show.html.erb

Rails.root: /home/ec2-user/environment/bookers2-debug

Application Trace | Framework Trace | Full Trace
app/views/books/_form.html.erb:21:in block in _app_views_books__form_html_erb__3555135995817864763_70203137971340' app/views/books/_form.html.erb:1:in _app_views_books__form_html_erb__3555135995817864763_70203137971340'
app/views/users/show.html.erb:8:in `_app_views_users_show_html_erb___123336745112956188_70203137756200'
Request
Parameters:

{"id"=>"1"}
Toggle session dump
Toggle env dump
Response
Headers:

None

該当のソースコード

books/_form.html.erb

<%= form_with model:book,local:true do |f| %>
  <div class="form-group">
    <%= f.label :title %>
    <%= f.text_field :title, class: 'form-control book_title' %>
  </div>
  <div class="form-group">
    <%= f.label :opinion %>
    <%= f.text_area :body, class: 'form-control book_body' %>
  </div>
  <div class="form-group">
 <% if f.object.new_record? %>
      <%= f.submit "Create Book", class: "btn btn-success" %>
    <% else %>
      <%= f.submit "Update Book", class: "btn btn-success" %>
    <% end %>
  </div>
<% end %>

user/show.html.erb

<div class='container px-5 px-sm-0'>
  <div class='row'>
    <div class='col-md-3'>
      <h2>User info</h2>
      <%= render 'info', user: @user %>
      <h2 class="mt-3">New book</h2>
      <%= render 'books/form', book: @book_form %>
    </div>
    <div class='col-md-8 offset-md-1'>
      <h2>Books</h2>
      <%= render 'books/index',books: @books %>
    </div>
  </div>
</div>

books_controller.rb

class BooksController < ApplicationController
before_action :correct_user, only: [:edit, :update]
def show
    @book_form = Book.new
    @book = Book.find(params[:id])
    @user = @book.user
  end

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

  end

  def create
    @book = Book.new(book_params)
    @book.user_id = current_user.id
    if @book.save
      redirect_to book_path(@book), notice: "You have created book successfully."
    else
      @books = Book.all
      render 'index'
    end
  end

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

  end



  def update
    @book = Book.find(params[:id])
    if @book.update(book_params)
      redirect_to book_path(@book), notice: "You have updated book successfully."
    else
      render "edit"
    end
  end

  def destroy
    @book = Book.find(params[:id])
    @book.destroy
    redirect_to books_path
  end

  private

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

  def correct_user
    @book = Book.find(params[:id])
    @user = @book.user
    redirect_to(books_path) unless @user == current_user
  end

end

users_controller.rb

class UsersController < ApplicationController

 
  before_action :ensure_correct_user, only: [:update, :edit]



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

  def index
    @users = User.all
    @book = Book.new
    
  end

  def edit
    @user = User.find(params[:id])
  end

  def update
    if @user.update(user_params)
      redirect_to user_path(@user), notice: "You have updated user successfully."
    else
       @user = User.find(params[:id])
       @books = @user.books
       @book = Book.new
      render "show"
    end
  end

  private
    def user_params
      params.require(:user).permit(:name, :introduction, :profile_image)
    end

   
    def ensure_correct_user
      @user = User.find(params[:id])
      unless @user == current_user
        redirect_to user_path(current_user)
      end
    end

  end

models/book.rb

class Book < ApplicationRecord
	belongs_to :user

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

models/user.rb

class User < ApplicationRecord
  # Include default devise modules. Others available are:
  # :confirmable, :lockable, :timeoutable, :trackable and :omniauthable
  devise :database_authenticatable, :registerable,
         :recoverable, :rememberable, :validatable

  has_many :books
  attachment :profile_image, destroy: false

  validates :name, length: {maximum: 20, minimum: 2}, uniqueness: true
  validates :introduction, length: {maximum: 50}

end

試したこと

自分で調べた所、Rubyのnew_record?メソッドを使うことはわかったが
使い方まではわからなかった

エラー文よりobjectが空だと思い、
objectの部分をbookにしたが
解決には至らなかった

gemをインストールするのかと思い、gem new_record?で調べたが
解決には至らなかった

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

バージョン
Rails 5.2.5
ruby 2.6.3p62 (2019-04-16 revision 67580) [x86_64-linux]

Answer 1

[編集: 誤解があったので、大きく編集しました]

エラー文よりobjectが空だと思い、

おそらくそのとおりの様子です。
可能性は二つ思いつきます。
routes.rb がないので、100%の確信は持てませんが。

一つの可能性は、Controller中の@book_form = Book.newがERBにわたっていて、それがpartialの_form.html.erbにbookとして渡り、それを変数fでiteratorに渡しているという場合。
この場合、_form.html.erb の文
form_with model:book,local:true do |f|
中でf に入っているのはformオブジェクトであり、bookに入っているのが、モデルのインスタンスBook.newです。

method new_record? は、モデルに対して有効です。f.objectは、formオブジェクトのobjectメソッドの返り値です。少なくとも Rails 5+ では、それは、与えたモデルに相当するようです(APIマニュアル)
。だから、それがnilであるということは、book にそもそもモデルのインスタンスが入っていないことを意味します。
たとえば、new_record?の文を

if book.new_record?

と書き換えると、状況がより明快になるでしょう。同じことですが。
モデルがnilということは、@book_form がうまく渡っていないことを意味します。もう一つの回答に推測が書かれている(私とほぼ同時に投稿したものです)ので、そちらを参照するといいかも知れません。

もう一つの可能性は、そもそもroutes.rbが正しく記述されていなくて、意図通りのControllerが読めていないことです。ログ出力を見たり、debug情報を出力させたりして、意図通りに動いているか、チェックすると良いでしょう。

以上、推測に基づく回答でした。

Comments

[garyoman]

教えていただきありがとうございます！

Answer 2

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

ここの book が nil になっています。そのため f.object も nil。
原因は @book_form が nil だからです。

# user/show.html.erb
<%= render 'books/form', book: @book_form %>

  # users_controller.rb
  def show
    @user = User.find(params[:id])
    @books = @user.books
    @book = Book.new
    # 追加
    @book_form = Book.new
  end

これで f.object に book が入って、new_record? で判定できます。

<% if f.object.new_record? %>
  create
<% else %>
  update
<% end %>

蛇足ですが、以下のようにすると、book の状態によって Update Book と Create Book に勝手に切り替わります

<%= f.submit class: "btn btn-success" %>

Comments

[garyoman]

教えていただきありがとうございます！