【rails】一対多対一のリレーションを使うフォームで、バリデーションに失敗する(仮)(最終更新:210826)

Question

おことわり

「unknown attribute 'XXX' for XXX. で必要ないはず/存在するはずのカラムが要求される」の続きになります。
そのため、「詰まっている箇所」まで同じ文面が続きます。

前提

クリック/キータッチで効果音を鳴らすことができるサンプラーアプリを作っています。

ここで作っているアプリはこれに加えて好きな音源をフォームで読み込んで音量やループ可否なども設定できます。
さらに、ログイン中は変更した設定を保存して、使いたい時に保存した設定を読み込むことができます。

イメージとしては「効果音ラボ」の「効果音ポン出し」に保存機能がついたようなものです。もっと具体的に言うと「KLANG」に当たると思われます。

効果音ラボ 効果音ポン出し
https://soundeffect-lab.info/pon/app.html

KLANG
http://m-tank.info/common/klang

実現したいこと

マウスクリック/キータッチで効果音を鳴らせるボタンが12あり、
12のボタンごとに「使用する効果音の音源(とその名前)」と「ボタンごとの設定(どのキーに配置するか、音量、ループ可能かどうか、ボタンの色)」を読み込みます。
それら12ボタンごとの設定と読み込む音源を、「サンプラー」の一つの投稿として保存(作成)する仕組みです。

##使用するテーブル
使用するテーブルは以下の通りです。
ユーザー(User)もありますが今回は省略いたします。
ちなみに簡単に言うとログインはdeviseで、メールアドレスではなくユーザーネーム(半角英数字)とパスワードでログインする仕組みにしています。

サンプラー(Sampler)
|保存するもの|カラム名|データ型|必須?|
|:--|:--|:--|
|id|id|-|
|ユーザーid|user_id|bigint?|
|サンプラー名|sampler_name|string|

ボタンごとの設定(Seboard)
|保存するもの|カラム名|データ型|必須?|
|:--|:--|:--|
|id|id|-|
|サンプラーid|sampler_id|bigint|true|
|効果音ライブラリid|sefile_id|bigint?|
|キー配置|position|integer|
|ボタン色|btncoler|integer|
|音量|volume|integer|
|ループ可否|loopable|loopable|

効果音ライブラリ(Sefile)
|保存するもの|カラム名|データ型|必須?|
|:--|:--|:--|
|id|id|-|
|ユーザーid|user_id|bigint|
|ボタンごとの設定のid|seboard_id|bigint|
|効果音(をDBに保存する際)の名前|sename|string|
|効果音の音源データ|sedata|(binary)string|

「効果音の音源」を「ボタンごとの設定」と分けているのは、「音源」は「サンプラー」の一つの投稿として投稿する以外にも直接「音源」を投稿できる仕組みにしたいためです。
証拠に、user_idのカラムがあり、idを取得してリレーションできるようにしています。

投稿(保存)のしくみ

一度の投稿で更新されるsampler一つにつき12のseboardがあり、それぞれに一つずつ(空白可)sefileが割り当てられます。

新規作成/更新のしくみ

ボタンは一つに統一し、「新規作成/更新」機能は一つに統一します。
具体的に言うと以下の通りです。

「『サンプラーに付ける名前』に前回保存したサンプラーの名前と同じ投稿がある場合、その投稿に今回保存するサンプラーの設定を更新。
それ以外は新規作成とする。」

以上、基本的な保存の仕組みに加えて、もう一つ

「『ボタンごとの設定』で読み込む音源に、
『効果音ライブラリ』の「効果音の名前」「効果音の音源データ」両方の値が同じデータがある場合、そのデータのidを『ボタンごとの設定』の『効果音ライブラリid』に割り当てる。
ない場合は新規データとして『効果音ライブラリ』のデータとして保存。そのidを割り当てる」

が加わります。

詰まっている箇所(更新:210810)

エラー画面はこのような感じです。

ただし、どこで詰まっているかはターミナルを見た方が詳しい気がするのでそちらを説明します。

現在の状況(ターミナル)

コードを貼ると字数を超えてしまうため、画像2つで表示しています。
申し訳ありません。

このコードを見る限り、リレーションの最上位であるsamplerテーブルの値は満足に送れている気がします。
それが、その下層であるseboardテーブルの値から何らかのエラーが起きて送信が停止しているように見えます。

現在のコード

/app/controllers/sampler_controller.rb

class SamplerController < ApplicationController

  before_action :authenticate_user!

  # 投稿を一覧形式にする(予定)
  def seindex
    @samplers = current_user.samplers.all
  end

  def save
    @sampler = current_user.samplers.find_or_initialize_by(sampler_name: params[:sampler_name])
    @sampler.seboards.build
    @sampler.sefile.find_or_initialize_by(sename: params[:sename], sedata: params[:sedata])
  end

  def submit
    @sampler = current_user.sampler.find_or_initialize_by(sampler_name: params[:sampler_name])
    unless @sampler.persisted?
      @sampler = current_user.sampler.new(sampler_params)

      unless @sampler.save!
        flash[:danger] = "保存に失敗しました"
      else
        flash[:success] = "保存しました"
      end

    else
      @sampler = current_user.sampler.update(update_sampler_params)

      unless @sampler.save!
        flash[:danger] = "更新に失敗しました"
      else
        flash[:success] = "更新しました"
      end

    end

    redirect_to root_path

  end


  def show
    @sampler = Sampler.find(params[:id])
  end

  private

  def sampler_params
    params.require(:sampler).permit(:sampler_name, seboards_attributes: [:position, :btncolor, :volume, :loopable, sefile_attributes: [:sename, :sedata]])
  end

  def update_sampler_params
    params.require(:sampler).permit(:sampler_name, seboards_attributes: [:position, :btncolor, :volume, :loopable, sefile_attribute: [:sename, :sedata]])
  end
end

/app/models/seboard.rb

class Seboard < ApplicationRecord
  belongs_to :sampler,inverse_of: :seboards#, class_name: "Sampler", foreign_key: 'sampler_id'

  has_one :sefile,inverse_of: :seboard#, class_name: "Sefile", foreign_key: 'sefile_id'
  accepts_nested_attributes_for :sefile


end

/app/models/sefile.rb

class Sefile < ApplicationRecord
  belongs_to :user, class_name: "User", foreign_key: 'user_id'

  belongs_to :seboard, class_name: "Seboard", foreign_key: 'seboard_id' ,inverse_of: :sefile, optional: true

  mount_uploader :sedata, SeUploader

end

/app/views/sampler/_save.html.erb

  <%= form_with model: @sampler do |sp| %>
    <%= sp.fields_for :seboards do |sb| %>
      <%= sb.fields_for :sefile do |sf| %>

  <!-- 上段 -->
  <div id="bg-q" class="case-btn-q case-btn" onclick="play(soundQ, loopQ, loopValue_q); btnColorInit(btnColorQ, bg_Q)">
    <p>Q</p>
    <%= sb.hidden_field :position,:name=>"sampler[seboards_attributes][1][position]", :value => "1" %>
    <%= sb.range_field :volume, :name=>"sampler[seboards_attributes][1][volume]", :onchange=>"volumeInit(soundQ, volumeQ, volumeValue_q)", :id=>"vol-q", :value=>50, :min=>1, :max=>99 %>
    <div class="radio-field_q">
      <%= sb.radio_button :btncoler, "1", :name=>"sampler[seboards_attributes][1][btncolor]", checked: true, :class=>"red" %>
      <%= sb.radio_button :btncoler, "2", :name=>"sampler[seboards_attributes][1][btncolor]", :class=>"blue" %>
      <%= sb.radio_button :btncoler, "3", :name=>"sampler[seboards_attributes][1][btncolor]", :class=>"white" %>
    </div><!--  .radio-field_q -->
    <%= sb.check_box "loopable", :name=>"sampler[seboards_attributes][1][loopable]", :onclick=>"loopInit(soundQ, loopQ, loopValue_q)", :id=>"loopable_q", :checked_value =>true, :unchecked_value=>false %>
      <%= sf.file_field :sedata, :name=>"sampler[seboards_attributes][1][sefile_attributes][sedata]", :id=>"selectFile_q", :onclick=>"readFile(soundQ, form_fileQ)" %>
      <%= sf.text_field :sename, :name=>"sampler[seboards_attributes][1][sefile_attributes][sename]" %>

  </div><!--  .case-btn-d -->
  <div id="bg-w" class="case-btn-w case-btn" onclick="play(soundW, loopW, loopValue_w); btnColorInit(btnColorW, bg_W)">
    <p>W</p>
    <%= sb.hidden_field :position,:name=>"sampler[seboards_attributes][2][position]", :value => "2" %>
    <%= sb.range_field :volume, :name=>"sampler[seboards_attributes][2][volume]", :onchange=>"volumeInit(soundW, volumeW, volumeValue_w)", :id=>"vol-w", :value=>50, :min=>1, :max=>99 %>
    <div class="radio-field_w">
      <%= sb.radio_button :btncoler, "1", :name=>"sampler[seboards_attributes][2][btncolor]", checked: true, :class=>"red" %>
      <%= sb.radio_button :btncoler, "2", :name=>"sampler[seboards_attributes][2][btncolor]", :class=>"blue" %>
      <%= sb.radio_button :btncoler, "3", :name=>"sampler[seboards_attributes][2][btncolor]", :class=>"white" %>
    </div><!--  .radio-field_q -->
    <%= sb.check_box "loopable", :name=>"sampler[seboards_attributes][2][loopable]", :onclick=>"loopInit(soundW, loopW, loopValue_w)", :id=>"loopable_w", :checked_value =>true, :unchecked_value=>false %>
      <%= sf.file_field :sedata, :name=>"sampler[seboards_attributes][2][sefile_attributes][sedata]", :id=>"selectFile_w", :onclick=>"readFile(soundW, form_fileW)" %>
      <%= sf.text_field :sename, :name=>"sampler[seboards_attributes][2][sefile_attributes][sename]" %>
  </div><!--  .case-btn-d -->

  <% end %>
  <% end %>

  <%= sp.text_field :sampler_name %>
  <%= sp.submit "保存する" %>
  <% end %>

/db/schema.rb

リレーション、特に外部キーの設定で何度もつまづいているため、念のためschema.rbから外部キーの設定も貼っておきます。

(略)
  add_foreign_key "samplers", "users"
  add_foreign_key "seboards", "samplers"
  add_foreign_key "seboards", "sefiles"
  add_foreign_key "sefiles", "seboards"
  add_foreign_key "sefiles", "users"
end

試したこと(更新:210814)

sefileにuser_idがない、とのご指摘をいただいたので解法を調べていたところ、

1. deep_merge
2. to_h.deep_merge(配列に変換した後deep_merge)
3. reverse_merge

の3つがありました。

deep_merge

コードは以下の通りに修正しました。

/app/controllers/sampler_controller.rb

  def sampler_params
    params.require(:sampler).permit(:sampler_name, seboards_attributes: [:position, :btncolor, :volume, :loopable, sefile_attributes: [:sename, :sedata]]).deep_merge(seboards_attributes:[sefile_attributes:[user_id: current_user.id]])
  end

  def update_sampler_params
    params.require(:sampler).permit(:sampler_name, seboards_attributes: [:position, :btncolor, :volume, :loopable, sefile_attributes: [:sename, :sedata]]).deep_merge(seboards_attributes:[sefile_attribute:[user_id: current_user.id]])
  end

その結果以下のようなエラーが出ました。

to_h.deep_merge

コードは以下の通りに修正しました。

/app/controllers/sampler_controller.rb

  def sampler_params
    params.require(:sampler).permit(:sampler_name, seboards_attributes: [:position, :btncolor, :volume, :loopable, sefile_attributes: [:sename, :sedata]]).to_h.deep_merge(seboards_attributes:[sefile_attributes:[user_id: current_user.id]])
  end

  def update_sampler_params
    params.require(:sampler).permit(:sampler_name, seboards_attributes: [:position, :btncolor, :volume, :loopable, sefile_attributes: [:sename, :sedata]]).to_h.deep_merge(seboards_attributes:[sefile_attribute:[user_id: current_user.id]])
  end

その結果以下のようなエラーが出ました。

reverse_merge

最後はreverse_mergeですが、結果は変わりませんでした。

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

ブラウザ:chrome(最新のバージョン)
テキストエディタ:atom(ver:1.32.2)
rails: Rails 6.1.4

Comments

[mather]

バリデーションエラーなので、 Seboards のモデル定義が影響していると思います。

[kaoru-drosera]

ご指摘のありましたseboardのモデルのコードを追記いたしました。

Answer 1

sefile のパラメーターに user_id が無いからです。

Answer 2

大変遅くなりました。
解決いたしましたので簡単に解決方法を示してから質問を閉めさせていただきます。

ただし、まだこのアプリに関した質問はまだもう一つ行うことになってしまいます。申し訳ありません。

#解決方法

結論から言うと、winterboum様が回答した通り「sefileのパラメータにuser_idが渡ってきていない」ことが原因です。

sefile側の「belongs_to」に「optional:true」を加え、空白可にすることで正常に投稿ができるようになりました。

/app/models/sefile.rb

  belongs_to :user, class_name: "User", foreign_key: 'user_id', optional: true

  has_many :samplers, through: :seboards, source: :sampler, foreign_key: 'sampler_id'
  accepts_nested_attributes_for :samplers


  belongs_to :seboard, class_name: "Seboard", foreign_key: 'seboard_id' ,inverse_of: :sefile, optional: true


  mount_uploader :sedata, SeUploader

  # バリデーションはこれ以降に

end

本当ならサンプラー経由で投稿してもuser_idが入ることでユーザーが自由に管理できるようにしたかったのですがここは妥協です。

最後に

別解としては、seboard側にもuser_idのカラムを外部キー付きで入れることも挙げられると考えています。ただし、他に実装すべき最低限の機能を優先的に実装させたいので試すのはだいぶ後になると思います。

長い間、そして今一度先のお目汚し申し訳ありません。
乱文乱筆、ご拝読重ねてありがとうございます。
自己解決で締めさせていただきますが、回答を下さったwinterboum様をベストアンサーといたします。