【Rails】チェックで選択した項目の個数と備考欄を付けて保存したい

Question

• 前提・実現したいこと

Railsで物件登録システムを作っています。
物件を登録する時に既に作られているモデルをチェックボックスから選択してそのチェックした個数と備考欄を子テーブルとして登録したいです。
このような要件の場合、どのようにしてチェックしたものを他情報を添えて登録することができるのでしょうか。
ご教示頂けますと幸いです。
なお質問要件の前はチェックしたものと物件を紐づけるのみでしたので、中間テーブルとして登録していました。
その時のモデルファイルが以下になります。

モデル

  has_many :gifts, :dependent => :destroy
  has_many :accessories, through: :gifts
  accepts_nested_attributes_for :gifts, allow_destroy: true

  belongs_to :property, optional: true
  belongs_to :accessory, optional: true

  has_many :gifts
  has_many :properties, through: :gifts

_form.html.erb
<%= form_with model:property, url:properties_path, local:true do |f| %>

  <div class="container">

    <div class="row create-property-forms">
        <div class="col-7">

          <div class="card">
            <div class="card-header bg-gray">物件情報</div>
            <div class="card-body">
            <!-- 基本情報 -->
            <%= render :partial => "standard", :locals => { f: f } %>
            <!-- テキストエリア -->
            <%= render :partial => "textareas", :locals => { f: f } %>
            <!-- 掲載期限 -->
            <%= render :partial => "pereod", :locals => { f: f } %>
          </div><!--左半分-->

        <div class="col-5">
          <!-- 業態 -->
          <%= render :partial => "business_lists", :locals => { f: f} %>
          <!-- エリア -->
          <%= render :partial => "area", :locals => { f: f} %>
          <!-- 譲渡品 -->
          <%= render :partial => "gifts", :locals => { f: f} %>
        </div><!--右半分-->

    </div><!--全体ラップ-->
  </div><!--container-->

  <div class="container">
        <!-- トップ画像 -->
        <%= render :partial => "top_image", :locals => { f: f} %>
        <!-- 複数画像 -->
        <%= render :partial => "many_image", :locals => { f: f} %>
  </div>

    <!-- 追従 -->
    <div class="fixed-box">
      <div class="container">
        <div class="d-flex  justify-content-between">
          <div class="">
            <div class="secret-check">
              <label for="secret_check" name="会員のみ公開" class="text-white">
              <%= f.check_box :secret, cheked:true, id:"secret_check" %>
              会員のみ公開</label>
            </div>
            <div class="recommend-check">
              <label for="recommend_check" name="おすすめに表示する" class="text-white">
              <%= f.check_box :recommend, cheked:true, id:"recommend_check" %>
              おすすめに表示する</label>
            </div>
          </div>
            <div class="my-2"><%= f.submit "登録", class:"btn btn-primary px-5 fixed-submit" %></div>
        </div>
      </div>
    </div>

<% end %><!-- formend -->

<style media="screen">
  .last-margin {
    margin-bottom: 10rem!important;
  }
  .station_select {
    width: 200px;
  }
</style>



<script type="text/javascript">

  $(document).ready(function(){
    $('#property_pref').change(function(){
     var pref_id = $('#property_pref') .val();
      $.get("/properties/city",
      {pref_id: pref_id},
       function(data){}
        );
    })

    $('#property_pref').change(function(){
    var pref_id = $('#property_pref') .val();
     $.get("/properties/station",
      {pref_id: pref_id},
      function(data){}
      );
    })
 });

</script>

_gifts.html.erb
    <div class="card my-2">
      <span class="card-header">譲渡品</span>

      <div class="rounded card-body p-3 m-3 acc-table-innner" id="acc_chk">
              <%= f.collection_check_boxes(:accessory_ids, Accessory.all, :id, :name, include_hidden: false) do |b| %>
                <%= b.label { b.check_box + b.text } %>
              <div>
              <%= f.number_field :stock, name: "property[gift][#{b.value}][stock]" %>
              <%= f.text_field :remark, placeholder: "備考を入力" ,name: "property[gift][#{b.value}][remark] " %>
              </div>

          <% end %>

      </div>
    </div>

  def create
    @property = Property.new(property_params)

    respond_to do |format|
      if @property.save!

　　　　# ギフト作成
        params[:property][:accessory_ids].each do | accessory_id |
        @property.gifts.create(
          accessory_id: accessory_id,
          stock: params[:property][:gift][accessory_id][:stock],
          remark: params[:property][:gift][accessory_id][:remark]
          )
        end

         params[:property_images][:image].each do |image|
          @property.property_images.create!(image: image, property_id: @property.id)
         end

         flash.now[:alert] = "#{@property.title}を作成しました。"
　　　　  redirect_to property_new_path
      else
        flash.now[:alert] = "物件作成に失敗しました"
        format.html { render :new }
        format.json { render json: @property.errors, status: :unprocessable_entity }
      end
     end
    end

    private
    def property_params
      params.require(:property).permit(:title, :preperty_no,:control_num,:price,:deposit,:admin_const,
        :othoer_price,:facility,:street_address,:traffic,:age,:area,
        :floor,:parcel,:construction,:parking,:parking_exp,:contract_method,:contract_period,:time,:secret,
        :recommend,:current_status,:list_comment,:detail_comment,:recommend_comment, :pref ,:city, :station,
        :inu_or_suke, :end_on, :image,
        area_tag_ids: [],category_ids: [], 
        preperty_images_attributes: [:image, :id, :_destroy]).merge(user_id: current.id)
    end

イメージ図

Comments

[winterboum]

わかりにくいところがいろいろあります。
Property に　has_many :gifts, :dependent => :destroy　とあるのに
Accsesory にも has_many :gifts, :dependent => :destroy とあります。
どちらが間違い？
Giftに　belongs_to :item,　とあります。　Itemってなにもの？

formを呼び出す controller　と　formが送り込まれるcontroller（今のを）載せてください

formをrenderしているviewも載せてください

[ganchan]

追記いたしました。すみませんItemは丸ごとだと問題があるかなと勝手に思い込んで変更して掲載したものです。修正しました。

Answer 1

このままでgiftにデータ入っていませんか？
accepts_nested_attributes_for :gifts, allow_destroy: true があり
<%=f.fields_for :gifts do |ff| %> <%= ff.select :stock, @num, <%= ff.text_field :remark %>
があり
def property_params params.require(:property).permit( gifts_attributes: [:property_id, :accessory_id, :stock, :remark],
がある。
気になるのは　
<%= form_with model:property, url:properties_path, local:true do
と properties_pathなのに create で受けているところですが、これで他のデータがsaveされているのならそこは合っているということですね。
stock、remark は入っていませんか？

追記：非スマート
_gifts.html.erb　の
<%= gift.collection_check_boxes .... do |b| %> <% end %> の所の中身
`<%= b.label { b.check_box + b.text } %>

<div> <%= gift.select :stock, @num, {include_blank: '個数を入力'},name: "property[gift][#{b.value}][stock]" %> <%= gift.text_field :remark, placeholder: "備考を入力" ,name: "property[gift][#{b.value}][remark] " %> </div> ` controller#create　の　if @property.save!　の後に ` gifts = params[:property][:gift][:accessory_id].each do | accessory_id | gift = params[:property][:gift][accessory_id] @property.gifts.create(accessory_id: accessory_id, stock: gift[:stock], remark: gift[:remark] ) end ` 辺りかな。 うまく行かなかったら、params見せてください

Comments

[ganchan]

ありがとうございます。
今記載しているコードでのlogを見てみると
giftの部分は
"gift"=>{"accessory_id"=>["10", "11"], "stock"=>"", "remark"=>""},
となっており、フォームにstockとremarkを入力しているのにわたっていない状態です。
今はgiftは作られていません。
モデルの関連記述とfields_forの使い方が間違っているということでしょうか。
検証でのinputのnameは
property[gift][accessory_id][]　<チェックボックス>
property[gift][stock]　＜個数＞
property[gift][remark] 　＜備考＞
となっています。
また "form_width"のurl先についてですが、createに向いているのでそこは問題なさそうです。
このようなケースを言語化して調べるのもワードが上手く見つからず頓挫しかけている状態です。
分からないままfields_forを使っているのでもう少し自分でも調べてまいります。

[winterboum]

そかそか、入力エリアができているんで詳しく見てませんでした。
<%=f.fields_for :gifts do |ff| %>　の中に有る
<%= ff.select :stock, @num, {include_blank: "個数を入力"} %>
<%= ff.text_field :remark %>
の id、nameが皆同じですね。
paramsが　params["gift_attributes"][32]["stock"] の様になっていないとcreateで自動に、はならないです。
accesory_id,stock,remarksのセットがアクセサリーの数だけ。
この３２の所はgiftのid、まだ作られて居ない時はありえないid。

で、自動でなくても良い、としてもどのアクセサリーのstockなのかがわかる構造にする必要があります。

＃今日はごめん疲れてるので明日。
# にしてもこのview疲れますね。フォーカスしてもスクロールするとどこだかわからなくなる。
# 機能毎にpartialにして構造がわかりやすいようにしたほうがよいかも

[ganchan]

読みにくいコードになっていてすみません。パーシャル化しました。
全てのinput=nameがproperty[gifts_attributes][0][カラム名]になっており原因を調べています。

[winterboum]

「原因を調べています。」いや原因はわかってます。。
ただ、このデータ構造のケースは私は初めてなので、どうしたらスマートな書き方が出来るか悩んでます。
スマートで無くてもよい？

[winterboum]

パーシャル化したのに張り替えてよ、、、　　;_;

[ganchan]

本当に申し訳ございません。ギフトのrender先を追記してformも修正しました。
スマートでない場合でもいいので知りたいです。

[ganchan]

おはようございます。先日掲示頂きましたようにコード改変いたしましたところ
Mysql2::Error: Field 'accessory_id' doesn't have a default value: INSERT INTO `gifts` (`property_id`, `created_at`, `updated_at`) VALUES (318, '2020-05-11 10:35:19', '2020-05-11 10:35:19')
となりINSERTまではいきますが、ROLLBACKしてしまうようです。

paramsは
"gifts_attributes"=>{"0"=>{"accessory_id"=>["10", "11"]}},
"gift"=>{"10"=>{"stock"=>"2", "remark"=>{" "=>"冷蔵庫"}}, "11"=>{"stock"=>"14", "remark"=>{" "=>"エアコン"}}, "13"=>{"stock"=>"", "remark"=>{" "=>""}}, "15"=>{"stock"=>"", "remark"=>{" "=>""}}},
となっております。

[winterboum]

想定内の動きです。
まずroolbackしたのは　必須　の　accessory_id　が渡っていないからです。
gift"=>{"　をみると　checkしていなかったaccessoryの情報も来てますね。
"0"=>{"accessory_id"=>["10", "11"]}},　でCHECKしたaccessoryがわかるので、loopｗ回してgift"=>{"　の数量などを得てGiftをつくる
ということになりますが、やってみてください。
だめだったらhelpします

[ganchan]

giftsを登録してチェックされたものに入っているremarkとstockも保存することができました！
…ですが
remark: "{\" \"=>\"エアコン\"}",という形で入ってしまっているようです。
paramsの
"gift"=>{"10"=>{"stock"=>"2", "remark"=>{" "=>"冷蔵庫"}}とremarkの所のparamsが思っていた形になっていません。collection do の中に二つあってstockと同じ条件のはずなのですが…。

[ganchan]

すみませんハッシュでremarkが入っている問題は自己解決しました。name=""の部分に余計な空欄があってそのせいでハッシュとしてつくられたようです。
何度もご指摘頂き登録まで無事にこぎ着けることができました。
ご迷惑おかけして申し訳ありませんでした。助かりました。