Formオブジェクトでupdateがうまくいくとnewのビューが開かなくなる

Question

前提・実現したいこと

item_categoryというFormオブジェクトで、1つのフォームからitemsテーブル、categoriesテーブルに保存をします。
下記コードで実装すると、編集ページ(items/editのビュー)からupdateはできますが、新規登録するために新規登録ページ(items/newのビュー)にとぼうとすると、下記エラーが出てしまいます。
初心者で恐縮ですが、どの箇所がおかしなことになっているのかご教示願います。

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

エラーメッセージ
NoMethodError in ItemsController#new
undefined method `category_name_id' for nil:NilClass

ターミナル
Started GET "/shops/2/items/new" for ::1 at 2021-04-06 00:11:33 +0900
Processing by ItemsController#new as HTML
  Parameters: {"shop_id"=>"2"}
  Owner Load (0.6ms)  SELECT `owners`.* FROM `owners` WHERE `owners`.`id` = 3 ORDER BY `owners`.`id` ASC LIMIT 1
  Shop Load (0.7ms)  SELECT `shops`.* FROM `shops` WHERE `shops`.`id` = 2 LIMIT 1
  ↳ app/controllers/items_controller.rb:55:in `set_shop'
  Owner Load (0.7ms)  SELECT `owners`.* FROM `owners` WHERE `owners`.`id` = 3 LIMIT 1
  ↳ app/controllers/items_controller.rb:64:in `move_to_index'
Completed 500 Internal Server Error in 23ms (ActiveRecord: 1.9ms | Allocations: 4182)
  
NoMethodError (undefined method `category_name_id' for nil:NilClass):
  
app/models/item_category.rb:55:in `default_attributes'
app/models/item_category.rb:26:in `initialize'
app/controllers/items_controller.rb:12:in `new'
app/controllers/items_controller.rb:12:in `new'

該当のソースコード

app/controllers/items_controller.rb

class ItemsController < ApplicationController
  before_action :authenticate_owner!, only: [:new, :create, :edit, :update, :destroy]
  before_action :set_shop, only: [:index, :new, :create]
  before_action :set_shop_item, only: [:show, :edit, :update, :destroy]
  before_action :move_to_index, only: [:new, :create, :edit, :update, :destroy]

  def index
    @items = @shop.items.order('created_at DESC')
  end

  def new
    @item_category = ItemCategory.new
  end

  def create
    @item_category = ItemCategory.new(item_params)
    if @item_category.valid?
      @item_category.save
      redirect_to action: :index
    else
      render :new
    end
  end

  def show
  end

  def edit
    @item_category = ItemCategory.new(item: @item)
  end

  def update
    @item_category = ItemCategory.new(item_params, item: @item)
    if @item_category.valid?
      @item_category.save
      redirect_to action: :show
    else
      render :edit
    end
  end

  def destroy
    redirect_to action: :index if @item.destroy
  end

  private

  def item_params
    params.require(:item).permit(:image, :name, :info, :color_id, :price, :category_name_id).merge(
      owner_id: current_owner.id, shop_id: params[:shop_id]
    )
  end

  def set_shop
    @shop = Shop.find(params[:shop_id])
  end

  def set_shop_item
    @shop = Shop.find(params[:shop_id])
    @item = Item.find(params[:id])
  end

  def move_to_index
    redirect_to root_path unless current_owner == @shop.owner
  end
end

app/models/item_category.rb

class ItemCategory
  include ActiveModel::Model
  attr_accessor :image, :name, :info, :category_name_id, :color_id, :price, :owner_id, :shop_id

  with_options presence: true do
    validates :image
    validates :name, length: { maximum: 40 }
    validates :info, length: { maximum: 1500 }
    validates :category_name_id, numericality: { message: 'select' }
    validates :color_id, numericality: { message: 'select' }
    validates :price, numericality: { only_integer: true,
                                      with: /\A[0-9]+\z/,
                                      message: 'half-width number' }
    validates :owner_id
    validates :shop_id
  end
  validates :price, numericality: { greater_than_or_equal_to: 100,
                                    less_than_or_equal_to: 9_999_999,
                                    message: 'out of setting range' }

  delegate :persisted?, to: :item

  def initialize(attributes = nil, category: Category.new, item: Item.new)
    @category = category
    @item = item
    attributes ||= default_attributes
    super(attributes)
  end

  def save
    ActiveRecord::Base.transaction do
      @category.update(category_name_id: category_name_id)
      @item.update(image: image, name: name, info: info, color_id: color_id, price: price, owner_id: owner_id, shop_id: shop_id,
                   category_id: @category.id)
    end
  end

  def to_model
    item
  end

  private

  attr_reader :item, :category

  def default_attributes
    {
      image: item.image,
      name: item.name,
      info: item.info,
      color_id: item.color_id,
      price: item.price,
      owner_id: item.owner_id,
      shop_id: item.shop_id,
      category_name_id: item.category.category_name_id
    }
  end
end

app/models/item.rb

class Item < ApplicationRecord
  has_one_attached :image
  belongs_to :owner
  belongs_to :shop
  belongs_to :category, dependent: :destroy
  has_one :order

  extend ActiveHash::Associations::ActiveRecordExtensions
  belongs_to :color
end

app/models/category.rb

class Category < ApplicationRecord
  has_many :items

  extend ActiveHash::Associations::ActiveRecordExtensions
  belongs_to :category_name
end

app/models/category_name.rb

class CategoryName < ActiveHash::Base
  self.data = [
    { id: 1, name: 'ベッド&マットレス' },
    { id: 2, name: 'クッション、寝具' },
    { id: 3, name: 'ソファ' },
    { id: 4, name: 'カーテン' },
    { id: 5, name: 'テーブル、机' },
    { id: 6, name: 'イス、チェア' },
    { id: 7, name: 'テレビ台、リビング収納' },
    { id: 8, name: '本棚、シェルフ、収納家具' },
    { id: 9, name: 'ラグ、カーペット' },
    { id: 10, name: '照明' },
    { id: 11, name: 'キッチン収納' },
    { id: 12, name: '調理家具、食器' },
    { id: 13, name: '洗濯用品、掃除グッズ' },
    { id: 14, name: 'トイレ用品、バス用品' },
    { id: 15, name: 'インテリア雑貨、日用品' },
    { id: 16, name: '家電、電化製品' },
    { id: 17, name: '子供家具、ベビー用品' },
    { id: 18, name: 'ペット用品' },
    { id: 19, name: 'ガーデニング用品、ガーデンファニチャー' },
    { id: 20, name: 'その他' }
  ]

  include ActiveHash::Associations
  has_many :categories
end

app/views/items/new.html.erb

<h2>商品の出品</h2>

<%= form_with model: @item_category, url: shop_items_path, local: true do |form| %>
  <%= render "shared/error_messages", model: form.object %>

  <div>
    <%= form.label :image, '商品画像' %>
    <%= form.file_field :image %>
  </div>
  <div>
    <%= form.label :name, '商品名' %>
    <%= form.text_field :name %>
  </div>
  <div>
    <%= form.label :info, '商品情報' %>
    <%= form.text_area :info %>
  </div>
  <div>
    <%= form.label :category_name_id, 'カテゴリー' %>
    <%= form.collection_select(:category_name_id, CategoryName.all, :id, :name, {include_blank: "--"}, {}) %>
  </div>
  <div>
    <%= form.label :color_id, '色' %>
    <%= form.collection_select(:color_id, Color.all, :id, :name, {include_blank: "--"}, {}) %>
  </div>
  <div>
    <%= form.label :price, '価格' %>
    <%= form.text_field :price %>
  </div>
  <div>
    <%= form.submit '出品する' %>
  </div>
<% end %>

app/views/items/edit.html.erb

div>
<%= @shop.name %>
</div>
<div>
<%= @item.name  %>
</div>

<%= form_with model: @item_category, url: shop_item_path(@shop.id, @item.id), local: true do |form| %>
  <%= render "shared/error_messages", model: form.object %>
  
  <div>
    <%= form.label :image, '商品画像' %>
    <%= form.file_field :image %>
  </div>
  <div>
    <%= form.label :name, '商品名' %>
    <%= form.text_field :name %>
  </div>
  <div>
    <%= form.label :info, '商品情報' %>
    <%= form.text_area :info %>
  </div>
  <div>
    <%= form.label :category_name_id, 'カテゴリー' %>
    <%= form.collection_select(:category_name_id, CategoryName.all, :id, :name, {include_blank: "--"}, {}) %>
  </div>
  <div>
    <%= form.label :color_id, '色' %>
    <%= form.collection_select(:color_id, Color.all, :id, :name, {include_blank: "--"}, {}) %>
  </div>
  <div>
    <%= form.label :price, '価格' %>
    <%= form.text_field :price %>
  </div>
  <div>
    <%= form.submit '変更する' %>
  </div>
  <div>
    <%= link_to 'もどる', shop_item_path(@shop.id, @item.id) %>
  </div>
<% end %>

試したこと

おそらく、newアクションでFormオブジェクトの

def initialize

end

の箇所が動かないのは、

 def default_attributes
    {
      image: item.image,
      name: item.name,
      info: item.info,
      color_id: item.color_id,
      price: item.price,
      owner_id: item.owner_id,
      shop_id: item.shop_id,
      category_name_id: item.category.category_name_id
    }
  end

のせいと考えて、ここを

 def default_attributes
    {
      image: item.image,
      name: item.name,
      info: item.info,
      color_id: item.color_id,
      price: item.price,
      owner_id: item.owner_id,
      shop_id: item.shop_id
    }
  end

として、category_name_idの記述を削除すれば、newもeditもビューを開けて、create、updateどちらも可能でした。
ただし、 category_name_id: item.category.category_name_idを消去すると、editのビューページ（編集フォーム）を開いた時category_name_id内の既存の文言はなくなり「ーー」となってしまいます（他のnameからinfoまでの内容は残ったままです。ActiveStorageでつけた写真は消えます）。

category_name_idを残したまま実装することは難しいでしょうか。

参考にしたページ

https://laptrinhx.com/formobujekutode-fu-shuteburuhe-zhino-bao-cun-539823778/

Answer 1

item.category が nil の状態のときに category_name_id を呼んでいるのでエラーになっています。

以下のようにすると item.category が nil のときは category_name_id が呼ばれません。

category_name_id: item.category && item.category.category_name_id

item.category が nil のときは category_name_id は nil になります。
item.category が存在する場合は item.category.category_name_id の値になります。

また、以下のようにちょっと短く書くこともできます。

category_name_id: item.category&.category_name_id

Comments

[maikoala]

ありがとうございます！
item.categoryがnilなのに、category_name_idは呼べないよって言われていたということなのですね…！
問題なくnewもeditも動きました。
大変助かりました。
ご丁寧にありがとうございました。