【Rails5】１つのフォームで複数のモデルにデータを保存

Question

###動作環境
ruby 2.4.4
Rails 5.2.0
gem 2.6.14.1
carrierwave 1.2.3

現在行っていること・問題

・Rails5を用いて、複数のモデルにデータを保存するフォームを作成中です。
・SirabasuというモデルとImageモデルがあり、１対多の関係を持っています。
・Sirabasuにレコードを１つ保存するフォーム内において、画像アップロードのfieldを記述し、その画像はImageに保存されるようにしたいです。（現状、画像は１枚としていますが、いずれは複数登録したい）
・Sirabasuにはデータが保存されているのですが、Imageには保存されませんでした。
調べた限りでの対策は行ったつもりなのですが、どうして良いかわかりません。どなたか解決策をご教授いただけないでしょうか。

参考にしたサイト

【Rails】formで複数モデル(テーブル)にレコードを登録 - teratail
一つのform_forで複数のテーブルに保存！！ - Qiita

###schema(一部)

create_table "images", force: :cascade do |t|
    t.string "image_path"
    t.integer "sirabasu_id"
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
  end

create_table "sirabasus", force: :cascade do |t|
    t.string "name"
    t.text "content"
    t.integer "number"
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
    t.integer "userid"
    t.integer "cid"
  end

###Model(Sirabasu)

class Sirabasu < ApplicationRecord
    has_many :images, dependent: :destroy
    accepts_nested_attributes_for :images
end

###Model(Image)

class Image < ApplicationRecord
    belongs_to :sirabasu, optional: true
end

  def new
      @sirabasu = Sirabasu.new
      @sirabasu.images.build
  end

  def create
    @sirabasu = Sirabasu.new(sirabasu_params)
    if @sirabasu.save
        redirect_to('/sirabasus')
    else
      render 'new'
    end
  end

  def sirabasu_params
    params.require(:sirabasu).permit(:number, :name, :content, :userid, :cid, sirabasus_attributes: [:image_path])
  end

###View(sirabasu#new)一部

<%= form_for(@sirabasu) do |form| %>
   <%= form.number_field :number %>
   <%= form.text_field :name %>
   <%= form.text_area :content %>
   <%= form.fields_for :images do |i| %>
      <%= i.file_field :image_path %>
   <% end %>
   <%= form.hidden_field :cid, :value => "" %>
   <%= form.hidden_field :userid, :value => "" %>
   <%= form.submit "登録", :class =>"link-button"%>
<% end %>

Answer 1

formオブジェクトというデザインパターンを使うのがいいかなと思います。フォームにActiveRecordのモデルを渡すのではなく保存に関する要件をまとめた新しいクラスを渡します。

SirabasuFormという保存のためのクラスを新たに用意します。SirabasuFormの中でSirabasuとImageのそれぞれのモデルを保存するようにします。
Controller

def new
  @sirabasu_form = SirabasuForm.new　#formオブジェクトパターンのためのクラスをviewでformに渡します
end

def create
  @sirabasu_form = SirabasuForm.new(sirabasu_params)
  if @sirabasu_form.save
    redirect_to('/sirabasus')
  else
    render 'new'
  end
end

def sirabasu_params
  params.require(:sirabasu_form).permit(:number, :name, :content, :userid, :cid, :image_path)
end

Model

class Sirabasu < ApplicationRecord
  has_many :images, dependent: :destroy
end

class Image < ApplicationRecord
  belongs_to :sirabasu, optional: true
end

class SirabasuForm
  include ActiveModel::Model #テーブルは持ちませんがApplicationRecordのsaveメソッドなどを提供します

  attr_accessor :number, :name, :content, :userid, :cid, :image_path

  validates :name, :content, presence: true

  def save
    return false if invalid? #バリデーションを追加する場合はここに追加します
 
    sirabasu = Sirabusu.new(name: name, content: content, number: number, userid: userid, cid: cid)
    sirabusu.images.new(image_path: image_path) #この処理をimageの分だけループすれば複数保存も出ます
    sirabusu.save
  end
end

View

<%= form_with(model: @sirabasu_form, url: sirabasu_path) do |form| %>
  <%= form.number_field :number %>
  <%= form.text_field :name %>
  <%= form.text_area :content %>
  <%= form.file_field :image_path %>
  <%= form.hidden_field :cid, :value => "" %> 
  <%= form.hidden_field :userid, :value => "" %> 
  <%= form.submit "登録", :class =>"link-button"%>
<% end %>

hidden_fieldで入力を求めない値はコントローラで受け渡してもいいかもしれません

def create
  @sirabasu_form = SirabasuForm.new(sirabasu_params.merge(cid: ***, userid: ***))

Comments

[退会済みユーザー]

回答ありがとうございます。アドバイスをもとに試してみたところ、画像のアップロードに成功しました。
formオブジェクトやform_withは全く知らなかったので、参考になりました。複数の画像をアップロードする機能にはまだ挑戦していませんが、質問の件については解決したので、ひとまず安心です。ありがとうございました。