rails6を使ってテーブルに複数のレコードを一括保存したい

Question

前提・実現したいこと

rails6を使ってテーブルにレコードを一括保存したいです。

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

エラーメッセージは出ませんがbinding.pryをかけるとuser_idがnilになります。

該当のソースコード

class MenusController < ApplicationController
  def index
    @menus = Menu.all.order(created_at: :desc).includes(:user)
  end

  def new
    @form = Form::MenuCollection.new
  end

  def create
    @form = Form::MenuCollection.new(menu_collection_params)
    if @form.save
      redirect_to root_path
    else
      render :new
    end
  end

  private

  def menu_collection_params
    params.require(:form_menu_collection).permit(menus_attributes: [:training_date, :part, :training, :weight, :rep,:memo]).merge(user_id: current_user.id)
  end


end

class Form::Base
  include ActiveModel::Model
  include ActiveModel::Callbacks
  include ActiveModel::Validations
  include ActiveModel::Validations::Callbacks
end

class Form::MenuCollection < Form::Base
  FORM_COUNT = 3
  attr_accessor :menus,:user_id

  def initialize(attributes = {})
    super attributes
    self.menus = FORM_COUNT.times.map { Menu.new() } unless self.menus.present?
  end
  

  def menus_attributes=(attributes)
    self.menus = attributes.map{ |_, v| Menu.new(v) }
  end



  def save
 
    Menu.transaction do
      self.menus.map(&:save!)
    end
      return true
    rescue => e
      return false
  end
end

class Menu < ApplicationRecord
  belongs_to :user
  validates :training_date, presence: true
  validates :part, presence: true
  validates :training, presence: true
  validates :rep, presence: true
end

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 :menus
  accepts_nested_attributes_for :menus

  validates :nickname, presence: true

end

<div class="new-training">

<%= form_with model: @form,url: menus_path, method: :post,local: true do |f|%>

<%= f.fields_for :menus do |i|%>
<div class="menu-forms">
  
  <div class="taraining-date">
    <%= i.label :training_date,"日付",id:"date" %><br />
    <%= i.datetime_field :training_date %>
  </div>

  <div class="menu-form">
    <%= i.label :training, "種目" %><br />
    <%= i.text_field :training,id:"menu_training" %>
  </div>

  <div class="menu-form">
    <%= i.label :weight, "重量" %><br />
    <%= i.number_field :weight,id:"menu_weight" %>kg
  </div>

  <div class="menu-form">
    <%= i.label :rep, "rep" %><br />
    <%= i.number_field :rep,id:"menu_rep" %>回
  </div>

  <div class="hidden-content">
    <%= i.hidden_field :part ,:value => "胸", id:"menu_part" %>
  </div>

  </div>
  <% end %>

  <div class="menu-low">
  <div class="memo">
    <%= f.label :memo, "MEMO" %><br />
    <%= f.text_area :memo,id:"menu_memo" %>
  </div>

  <div class="menu-submit">
    <button type="submit" class="btn">記録する</button>
  </div>
  </div>
  <% end %>
</div>

試したこと

外部サイトを参考にし、実装しています。
attr_accesorにuser_idを記述して、user.rbにaccepts_nested_attributes_for :menusの記述をしました。
また、binding.pryをcreateアクションで行うと
[1] pry(#<MenusController>)> @form
=> #<Form::MenuCollection:0x00007f872026ccb8
@menus=
[<Menu:0x00007f871ca98198
id: nil,
training_date: Wed, 12 May 2021,
part: "胸",
training: "a",
weight: 1,
rep: 1,
memo: nil,
user_id: nil,
created_at: nil,
updated_at: nil>,
<Menu:0x00007f87278ac298
id: nil,
training_date: Wed, 12 May 2021,
part: "胸",
training: "b",
weight: 2,
rep: 2,
memo: nil,
user_id: nil,
created_at: nil,
updated_at: nil>,
<Menu:0x00007f87278b7800
id: nil,
training_date: Wed, 12 May 2021,
part: "胸",
training: "c",
weight: 3,
rep: 3,
memo: nil,
user_id: nil,
created_at: nil,
updated_at: nil>],
@user_id=1>
と表示され、user_idのみ"nil"となってしまいます。

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

rails 6.0.0

Answer 1

MenusController の menu_collection_params がおかしいと思います。
順を追って確認していきましょう。
まず、controller が受け取る生の params は以下のような構造になっています。

# params
{
  'form_menu_collection' => {
    'menus_attributes' => {
      '0' => {
        'training_date' => '...', 'part' => '...', (中略), 'memo' => '...'
      },
      '1' => {
        'training_date' => '...', 'part' => '...', (中略), 'memo' => '...'
      }
    }
  }
}

これを require と permit で処理すると以下のようなハッシュになります。

# params.require(...).permit(...)
{
  'menus_attributes' => {
    '0' => {
      'training_date' => '...', 'part' => '...', (中略), 'memo' => '...'
    },
    '1' => {
      'training_date' => '...', 'part' => '...', (中略), 'memo' => '...'
    }
  }
}

このハッシュに対して .merge(user_id: ...) を行っているので、最終的なハッシュは

# params.require(...).permit(...).merge(...)
{
  'menus_attributes' => {
    '0' => {
      'training_date' => '...', 'part' => '...', (中略), 'memo' => '...'
    },
    '1' => {
      'training_date' => '...', 'part' => '...', (中略), 'memo' => '...'
    }
  },
  'user_id' => 123  # ← ここにマージされちゃう
}

となります。

でも、本来 user_id は以下のように追加する必要があります。

# 本当は↓みたいな形にしないといけない
{
  'menus_attributes' => {
    '0' => {
      'training_date' => '...', 'part' => '...', (中略), 'memo' => '...', 'user_id' => 123
    },
    '1' => {
      'training_date' => '...', 'part' => '...', (中略), 'memo' => '...', 'user_id' => 123
    }
  }
}

なので、上記のようなハッシュになるよう、 MenusController#menu_collection_params を修正してあげれば良いと思います。

Comments

[hieee]

mergeにこだわりすぎていました、、
無事解決できました！！！！！ありがとうございます！！！！！！