【Rails】モデル間で紐付けをするとDBにデータが格納されなくなる

Question

前提・実現したいこと

モデルの関係をもった状態で日次で食べた食品情報をDBに格納できるようにしたい

現状

日次で食べた食品情報を格納するFooddiaryモデルとユーザーの1日あたりに食べてよい食品情報を格納するIdealweightモデルを紐付けするとDBに食べた食品情報が格納されなくなってしまいます。

なおモデルの紐付けを外すと正常に格納されます。

紐付けがうまくいっていないのか。
なぜ格納されないのか調べても今ひとつわからなかったため質問させていただきました。

またfooddairyモデル（日次食品情報）とidealweightモデル（そのユーザーの1日あたりにとっていい食品情報）は多：１で考えています。

※該当コードは下記にまとめて記述しております。

###binding.pryすると

From: /home/ec2-user/environment/bodymake/app/controllers/fooddiaries_controller.rb:21 FooddiariesController#create:

    19: def create
    20:   @fooddiary = current_user.fooddiaries.create(fooddiary_params)
 => 21:   binding.pry
    22:   redirect_to action: 'index'
    23: end

[1] pry(#<FooddiariesController>)> @fooddiary.errors.messages
=> {:idealweight=>["must exist"], :idealweight_id=>["can't be blank"]}

補足

##fooddiaries_controller.rb

class FooddiariesController < ApplicationController
  
  def new
    @fooddiary = current_user.fooddiaries.new
  end
  
  def create
    @fooddiary = current_user.fooddiaries.create(fooddiary_params)
    redirect_to action: 'index'
  end
  
  private
  def fooddiary_params
    params.require(:fooddiary).permit(:gram, :kcal, :protein, :fat, :carbo, :timing, :foodname, :fooddate_id, :user_id, :idealweight_id)
  end

end

##fooddiaries/new.html.erb

<%= form_for @fooddiary do |f| %>
  <tbody>
    <tr>
      <th>foodname</th>
      <th>gram</th>
      <th>timing</th>
    </tr><br>
      <%= f.collection_select :fooddate_id, Fooddate.all, :id, :foodname %>
      <td><%= f.number_field :gram, placeholder: "gram" %></td>
      <td><%= f.select :timing, ['朝食', '昼食', '夕食', '間食'] %></td>
      <br><%= f.submit "送信", class: "btn" %>
  </tbody>
<% end %>

class Idealweight < ApplicationRecord
##idealweight.rb

  belongs_to :user
  has_many :fooddiaries
  
  before_save do
    self.basemetabolism = 13.397*(weight)+4.799*(height)-5.677*25+88.362
    self.kcalburned = (basemetabolism)*(exerciselevel)
    self.minusweight = (weight)-(targetweight)
    self.minuskcal = (minusweight)*7000
    self.kcalintake = (kcalburned)-(minuskcal)/(period)
    self.intakeproteingram = (weight)*2
    self.intakeproteinkcal = (weight)*2*4
    self.intakefatgram = (weight)*0.7
    self.intakefatkcal = (weight)*0.7*9
    self.intakecarbokcal = (kcalintake)-(intakeproteinkcal)-(intakefatkcal)
    self.intakecarbogram = (intakecarbokcal)/4
  end
end

##fooddiary.rb
class Fooddiary < ApplicationRecord

  belongs_to :fooddate
  
  belongs_to :user
  validates :user_id, presence: true
  
  belongs_to :idealweight
  validates :idealweight_id, presence: true
  
  def self.search(search) 
    if search
      where(['created_at LIKE ?', "%#{search}%"]) 
    end
  end

  before_save do
    self.kcal = (gram)*(fooddate.kcal)
    self.protein = (gram)*(fooddate.protein)
    self.fat = (gram)*(fooddate.fat)
    self.carbo = (gram)*(fooddate.carbo)
  end
  
end

Comments

[maisumakun]

Railsのバージョンはいくつですか？

[yastinbieber]

Rails 5.2.4になります！

Answer 1

binding.pry　するなら、@fooddiaries = current_user.fooddiaries.create(fooddiary_params)の後で@fooddiaries.errors.messages してみてください。
何が原因でsaveできないかがわかります。

なお、createで作られるのは一つですから、@fooddiary にしたほうが誤解／勘違いが減ります

Comments

[yastinbieber]

ありがとうございます。@fooddiaryに変更しました。
またご指摘いただいたようにbinding.pryを実施してみたところ下記のように出ました。

[1] pry(#<FooddiariesController>)> @fooddiary.errors.messages
=> {:idealweight=>["must exist"], :idealweight_id=>["can't be blank"]}

[winterboum]

つまり、これらにデータが入っていないからsaveできなくなっています。
これで解決できればおめでとう。

できなければ、入力するviewとcontrollerを載せてください

[yastinbieber]

解説ありがとうございます。

しかしながらidealweightモデルに情報も入れておりますのでなぜデータが入っていないと出るのかわからない状態です。
恐れ入りますが該当コードを追記しましたのでご確認いただけますと幸いです。

[winterboum]

モデルに項目があり、strongparameterに項目があるからといって、データがあるとは限りません。
viewで入力しているのは　:fooddate_id :gram, :timing, の３つ。
strong_paramater には沢山あるけど
:kcal, :protein, :fat, :carbo, :foodname, : :user_id, :idealweight_id
のデータは送られてこない

[yastinbieber]

そういうことなんですね。

ちなみにviewにて:fooddate_id :gram, :timingのみを記入としていますが、
fooddiary.rbにて
before_save do
self.kcal = (gram)*(fooddate.kcal)
self.protein = (gram)*(fooddate.protein)
self.fat = (gram)*(fooddate.fat)
self.carbo = (gram)*(fooddate.carbo)
end
のような形で記載をしており、Mysqlには:kcal, :protein, :fat, :carbo, :foodnameも登録されるようになっています。

上記の点から言うと:user_id, :idealweight_idの部分をviewに記載すれば正式にデータが送られるという認識であっておりますでしょうか？

[winterboum]

はい。

[yastinbieber]

ありがとうございます。

実際にnew.html.erbにて
<%= f.number_field :idealweight_id %>
を記載したところ無事にDBに情報が格納され反映されるようになりました。

しかしながらこのidealweightモデルですが、ユーザーの目標数値情報のため毎回入力するものではないのでフォームに入れなくともuser_idと同じ数値を格納したいです。（常にuser_idとidealweight_idは同じ数値である必要があるため。user_idが1ならidealweight_idも必ず1でなくてはならない）

そこでidealweight.rbにて
before_save do
　self.idealweight_id = user_id
end
のような風にも記載しましたがうまく格納されませんでした。（user_idは無事格納されるため）

そこで毎回入力しなくともuser_idと同じ数値を格納できるようにしたいのですがもしよろしければお教えいただけますと幸いです。
度々すみません・・・

[winterboum]

idealweight_id　は　Idealweight ではなくてFooddiaryのものですから、やるならそちらの before_save です。
user_idはどの時点で手に入るものですか？modelは知らずcontrollerが知っているものだと思うので、before_userで入れられるのが不思議。
Fooddiaryのをsaveしようとしているloginユーザのidですか？
でしたらもうひとつの手は、fooddiary_paramsの最後に、　
merge(idealweight_id: current_user_id)
をつける

[yastinbieber]

ご教示いただきありがとうございます。

controller内のcreate部分を編集したところ無事DBに格納されました。

■以前
def create
@fooddiary = current_user.fooddiaries.create(fooddiary_params)
redirect_to action: 'index'
end

■以後
def create
@fooddiary = Fooddiary.create(gram: fooddiary_params[:gram], kcal: fooddiary_params[:kcal], protein: fooddiary_params[:protein], fat: fooddiary_params[:fat], carbo: fooddiary_params[:carbo], timing: fooddiary_params[:timing], fooddate_id: fooddiary_params[:fooddate_id], user_id: current_user.id, idealweight_id: current_user.id )
redirect_to action: 'index'
end

大変勉強になりました。
ありがとうございました。