ネストしたモデルのフォームに入力した値を計算して、DBに登録したい

Question

前提・実現したいこと

楽曲の情報を登録するアプリを作っています。任意の楽曲の演奏時間を登録する機能を実装しようとしています。曲は複数の楽章から構成されます。

その際に入力した分と秒を楽章別に計算して秒に変換し、DBに登録したいです。form_withとfields_forを使います。

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

曲の情報は親モデル(Post)、演奏そのものの情報は子モデル（Recording）、演奏時間の情報は孫モデル（Duration）の管轄です。

子モデルのフォームで分(min)と秒(sec)を登録し、それらを計算して秒(duration)に変換して、なおかつそれを孫モデルに格納する流れがわかりません。どうすればよいでしょうか。

Started POST "/posts/2/recordings" for ::1
Processing by RecordingsController#create as JS
  Parameters: {"authenticity_token"=>"N3NSB14zOEEgMKWfWwtVxnHN3PCyNpO6zwJHuSiadQvOfHTeP9RBofItGZeWDPc407Z/HyfSXpCDnt7q
KSWHQw==", 
"recording"=>{"composer"=>"Mozart", "title"=>"Symphony No.40", "conductor"=>"test", "orchestra"=>"test", "durations_attributes"=>{   
"0"=>{"mov"=>"1", "min"=>"1", "sec"=>"1"},    
"1"=>{"mov"=>"2", "min"=>"2", "sec"=>"2"},    
"2"=>{"mov"=>"3", "min"=>"3", "sec"=>"3"},    
"3"=>{"mov"=>"4", "min"=>"4", "sec"=>"4"}    
},
"year"=>"2000", "label"=>"test", "review"=>"test", "post_id"=>"2"}, 
"commit"=>"Create", "post_id"=>"2"}
Completed 500 Internal Server Error in 4ms (ActiveRecord: 0.0ms | Allocations: 3104)
NoMethodError (undefined method `set_duration' for nil:NilClass):

app/controllers/recordings_controller.rb:19:in `recording_params_with_duration'
app/controllers/recordings_controller.rb:12:in `create'

該当のソースコード

コントローラ(Recording)
class RecordingsController < ApplicationController

  def index
  end

  def new
      @recording = Post.find(params[:post_id]).recordings.new
      @recording.durations.build
  end

  def create
      @recording = Recording.new(recording_params_with_duration)
      @recording.save
      redirect_to('/posts/')
  end

  def recording_params_with_duration
    recording_params.reverse_merge(@durations.set_duration)
  end

  def recording_params
        params.require(:recording).permit(
　　　　　(中略)　　　　　  
　　　　　durations_attributes: [:mov, :min, :sec, :duration]
        )
  end

コントローラ(duration)
class DurationsController < ApplicationController

end



モデル

Post(親モデル)
class Post < ApplicationRecord
  has_many :recordings
  has_many :post_instruments
  has_many :instruments, :through => :post_instruments
  accepts_nested_attributes_for :recordings, {allow_destroy: true}
end

Recording(子モデル)
class Recording < ApplicationRecord
  belongs_to :post
  has_many :durations
  accepts_nested_attributes_for :durations
end

Duration(孫モデル)
class Duration < ApplicationRecord
  belongs_to :recording

  def duration
    min ＊ 60　+  sec 
  end

  def set_duration
    {:duration => duration }
  end
end


ビュー
<%= form_with model: @recording, url: post_recordings_path do |f| %>

  <% for i in [e-mail] do %>
    <%= f.fields_for :durations do |dur| %>
      <%= dur.number_field :mov, value: "#{i}" %><br>
      <%= dur.label :"mov#{i}min" %>
      <%= dur.number_field :min,  min: 0 %><br>
      <%= dur.label :"mov#{i}sec" %>
      <%= dur.number_field :sec, min: 0, max: 59 %><br>
    <% end %>
  <% end %>

データベース

CREATE TABLE IF NOT EXISTS "recordings" ("id" integer NOT NULL PRIMARY KEY, （中略）CONSTRAINT "fk_rails_47949a0700"
FOREIGN KEY ("post_id")
  REFERENCES "posts" ("id")
CREATE TABLE IF NOT EXISTS "durations" ("id" integer NOT NULL PRIMARY KEY,"min" integer DEFAULT NULL, "sec" integer DEFAULT NULL, "mov" integer DEFAULT NULL, "recording_id" integer DEFAULT NULL, "duration" integer, CONSTRAINT "fk_rails_4bba954102"
FOREIGN KEY ("recording_id")
  REFERENCES "recordings" ("id")

試したこと

マージを使ってパラメータをマージさせる方法を載せているウェブサイト、あるいは計算をする方法を載せているウェブサイトは複数あり、それを模倣してやってみました(例：https://qiita.com/miyzawa/items/18bb8d88ad004cbc439c)。しかしそれは同一モデル内での話で、親子モデル間でそれをすることができませんでした。

コントローラやモデルにおけるメソッドの間の受け渡しについての理解が足りないのではないかと思っております

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

Answer 1

結局、「そもそもDBに、２つの数値を計算した結果を格納する必要はない」ということに気づき、結局分と秒だけDBに登録して、ウェブページに表示させるときに秒に変換して、それを再び分：秒で表示することにしました。

durationモデルに以下の自作関数を作って(stringで入力したので、to_iでintegerに変換しないとエラーとなります)

  def total_seconds
    min.to_i * 60 + sec.to_i
  end

recordingコントローラに以下のことを書いたら望んでいたものが表示されるようになりました(別の問題が発生してはいますが、それは別の機会に書きます)。

      <% recording.durations.each do |duration| %>
      Mov<%= duration.mov %> <%=Time.at(duration.total_seconds).utc.strftime("%M:%S") %>/
      <% end %>

そもそも自分の実力では最初に考えていたやり方自体が無理だということでした。
もちろん、2つの入力項目を計算した上でDBに別のカラムで格納する方法がある場合はお教えください。

ヒントをくれたりエラー理由を指摘してくださったお二方には感謝いたします。

Answer 2

def create @recording = Recording.new(recording_params_with_duration) で呼んでいる　recording_params_with_duration にて
recording_params.reverse_merge(@durations.set_duration)
と @durations を使っていますが、この時点では @durations　は未定義なのでこのエラーとなります。
どういう情報を reverse_merge　したいのでしょう。

及び、質問の内容からして、孫modelのcodeも載せてください

Comments

[kawa0tatsu]

この時点では @durationsは未定義なのでこのエラーとなります。>
durationsは、recordingコントローラのnewメソッドですでに定義されてるものだと思い込んでました。

どういう情報を reverse_mergeしたいのでしょう。>
durationモデルのdurationメソッドでminとsecを計算したものを、set_durationメソッドでパラメータにし、それををmergeでくっつければ良いと思ってました。

恥ずかしながらmergeとreverse_mergeの違い自体よくわかっていなかったので、出来るのなら普通のmergeでいいです。

及び、質問の内容からして、孫modelのcodeも載せてください>
孫モデルはdurationだと思ってましたが、親モデルのpostのことでしょうか。一応追加しておきました。

Answer 3

小モデルというか、プログラム的な考え方をすると二次配列による集計方法ですよね。

https://ja.stackoverflow.com/questions/56121/ruby-2%E6%AC%A1%E5%85%83%E9%85%8D%E5%88%97%E5%86%85%E3%81%AE%E8%A8%88%E7%AE%97%E6%96%B9%E6%B3%95

Comments

[kawa0tatsu]

ヒントありがとうございました。もっと勉強して、いつかその方向でできるようにしたいです。