フォームに入力した情報がエラーでないが保存できない

Question

前提・実現したいこと

購入機能を実現したいのですが、
フォームに入力した内容が反映されず、全部フォームを埋めてもバリデーションで弾かれてしまいます
一つのビューで複数のテーブルを更新ということをtrade_address.rbでやろうとしてます

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

エラーメッセージは無く、投稿した内容が反映されて無い状態です

該当のソースコード

routes.rb

  resources :drinks, only: [:index,:new,:show,:create,:destroy] do
    resources :trades,only: [:index,:new,:create]
  end
  post '/drinks/:id/trades/new', to: 'trades#create'

drinks/show.html.erb

 <%= link_to "商品を購入する", drink_trades_path(@drink) %>

このパスの指定はあってると思います

trades/index.erb

<%= form_with model: @order,url: drink_trades_path,method: :post, local: true do |f|%>
  <%= render 'shared/error_messages',object: f.object%>
  <p>苗字</p>
  <%= f.text_field :fam_name %>
  <p>名前</p>
  <%= f.text_field :first_name %>
  <p>苗字カナ</p>
  <%= f.text_field :fam_name_kana %>
  <p>名前カナ</p>
  <%= f.text_field :first_name_kana %>
  <p>誕生日</p>
        <%= raw sprintf(
                  f.date_select(
                    :birthday,
                    class:'select-birth',
                    id:"birth-date",
                    use_month_numbers: true,
                    prompt:'--',
                    start_year: 1930,
                    end_year: (Time.now.year - 5),
                    date_separator: '%s'),
                  "<p> 年 </p>", "<p> 月 </p>") + "<p> 日 </p>" %>
  <p>郵便番号</p>
  <%= f.text_field :postal_code %>
  <p>都道府県</p>
  <%= f.collection_select(:prefecture_id, Prefecture.all, :id, :name, {}, {class:"select-box", id:"prefecture"}) %>
  <p>市</p>
  <%= f.text_field :city %>
  <p>番地</p>
  <%= f.text_field :house_num %>
  <p>建物名</p>
  <%= f.text_field :building_name%>
  <p>電話番号</p>
  <%= f.text_field :phone_num%>
  <p>登録</p>
  <%= f.submit "確定"%>
<% end%>

trades_controller.rb

class TradesController < ApplicationController
  include SessionsHelper
  def index
    # 商品を表示

    @item = Drink.find(params[:drink_id])
    @order = TradeAddress.new
    #redirect_to root_path if @drink.trade
  end

  def new

    @item = Drink.find(params[:drink_id])
    @order = TradeAddress.new
    # 商品購入ページ
  end

  def create
    
    @item = Drink.find(params[:drink_id])
    @order = TradeAddress.new(trade_params)
    binding.pry
    if @order.valid?
      @order.save
      redirect_to root_path
    else
      render 'index'
    end
  end

  private
    def trade_params
      params.permit(:item_id,:user_id,:fam_name,:first_name,:fam_name_kana,:first_name_kana,:birthday, :postal_code, :prefecture_id, :city, :house_num, :building_name, :phone_num, :trade_id).merge(user_id: current_user.id,price: @item.price)
    end
end

trade_address.rb

class TradeAddress
  include ActiveModel::Model
  # form_withやrender等の色々な機能が使える
  attr_accessor :fam_name,:first_name,:fam_name_kana,:first_name_kana,:birthday,:postal_code, :city, :house_num, :building_name, :phone_num, :drink_id, :prefecture_id,:trade_id, :user_id,  :price

  with_options presence: true do

    with_options format: { with: /\A[ぁ-んァ-ン一-龥]/, message: 'は全角で入力してください。' } do
      validates :fam_name
      validates :first_name
    end

    with_options format: { with: /\A[ァ-ヶー－]+\z/, message: 'は全角カタカナで入力して下さい。' } do
    validates :fam_name_kana
    validates :first_name_kana
    end

    validates :birthday
    validates :postal_code,format: {with: /\A\d{3}[-]\d{4}\z/}
    validates :prefecture_id, numericality: { other_than: 1 } 
    validates :city
    validates :house_num
    validates :phone_num
    validates :trade_id
  end


  def save
    trade = Trade.create(user_id: user_id, drink_id: drikn_id)
    Address.create(fam_name: fam_name,first_name: first_name,fam_name_kana: fam_name_kana,postal_code: postal_code, prefecture_id: prefecture_id, city: city, house_num: house_num, building_name: building_name, phone_num: phone_num, trade_id: trade.id)
  end
end

各モデルのアソシエーション

user.rb

  has_many :drinks, dependent: :delete_all
  has_many :trades

drink.rb

  belongs_to :user
  has_one :trade

trade.rb

  belongs_to :user
  belongs_to :drink

address.rb

  belongs_to :trade

マイグレーションファイル

class CreateAddresses < ActiveRecord::Migration[6.0]
  def change
    create_table :addresses do |t|
      t.string     :fam_name        ,null: false
      t.string     :first_name      ,null: false
      t.string     :fam_name_kana   ,null: false
      t.string     :first_name_kana ,null: false
      t.date       :birthday        ,null: false
      t.string     :postal_code     ,null: false
      t.integer    :prefecture_id   ,null: false
      t.string     :city            ,null: false
      t.string     :house_num       ,null: false
      t.string     :building_name   ,null: false
      t.string     :phone_num       ,null: false
      t.references :trade           ,null: false,foreign_key: true
      t.timestamps
    end
  end
end

create_trade.rb

class CreateTrades < ActiveRecord::Migration[6.0]
  def change
    create_table :trades do |t|
      t.references :user, foreign_key: true
      t.references :drink,foreign_key: true
      t.timestamps
    end
  end
end

試したこと

From: /Users/soichirohara/coffee_passport/app/controllers/trades_controller.rb:22 TradesController#create:

    18: def create
    19:   
    20:   @item = Drink.find(params[:drink_id])
    21:   @order = TradeAddress.new(trade_params)
 => 22:   binding.pry
    23:   if @order.valid?
    24:     @order.save
    25:     redirect_to root_path
    26:   else
    27:     render 'index'
    28:   end
    29: end

[1] pry(#<TradesController>)> @order
=> #<TradeAddress:0x00007fb81f1652b8 @price=3000, @user_id=6>
[2] pry(#<TradesController>)> @item
=> #<Drink:0x00007fb81f1578e8
 id: 1,
 name: "aaaa",
 price: 3000,
 explain: "aaaa",
 user_id: 6,
 created_at: Mon, 30 Nov 2020 08:29:00 UTC +00:00,
 updated_at: Mon, 30 Nov 2020 08:29:00 UTC +00:00>

このように入力した情報が反映されて無い状態が確認できました

[1] pry(#<TradesController>)> params
=> <ActionController::Parameters {"authenticity_token"=>"dD3/gLc7zNCF6f5s2Rj1VrjpSO27R5QReDwAstYFUc5h833ALZNUxw1a8N8d2kfTBeJ4JyhdXY/QedgPgXyqcg==", "trade_address"=>{"fam_name"=>"原", "first_name"=>"ソウ", "fam_name_kana"=>"", "first_name_kana"=>"", "birthday(1i)"=>"", "birthday(2i)"=>"", "birthday(3i)"=>"", "postal_code"=>"", "prefecture_id"=>"1", "city"=>"", "house_num"=>"", "building_name"=>"", "phone_num"=>""}, "commit"=>"確定", "controller"=>"trades", "action"=>"create", "drink_id"=>"1"} permitted: false>
[2] pry(#<TradesController>)> 

```試しに,fam_name,first_nameに当たる部分を入力して送信したら、パラメーターの中にはあるようです、、。
パラメーターの情報を上手く取得できないです

ストロングパラメーターで格納したい情報が書き切れて無いのでしょうか、、、。
もしくは格納したい情報に不備があるのか、、、。

ネットで調べた事によると,
belongs_toの後にoptional: trueをつけて、外部キーの空のカラムを許容するとのことですが、これは購入機能の場合大丈夫なんでしょうか、、、
回答をいただければ幸いです、、、


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

ruby 2.6系
rails 6.0.0
vscode

Answer 1

恐らくストロングパラメータの記述に問題があると思います。
trade_address

    def trade_params
      params.requier(:trade_address).permit(省略).merge(user_id: current_user.id,price: @item.price)
    end

permit内は長かったので、省略させてもらいました。
一度これでお試しいただいてもよろしいでしょうか？

Comments

[divclass123]

requireがあっても上手くいったんですかね、、、。
協力していただいてありがとうございました、、、、！

[tomtomtomtom]

高評価ありがとうございました。
お役に立てず申し訳ありません。
パラメーターを見て、てっきりrequierと思ってしまいました^^;
また、機会がありましたら宜しくお願い致します。

Answer 2

  def save
    trade = Trade.create(user_id: user_id, drink_id: drikn_id)
    Address.create(fam_name: fam_name,first_name: first_name,fam_name_kana: fam_name_kana,postal_code: postal_code, prefecture_id: prefecture_id, city: city, house_num: house_num, building_name: building_name, phone_num: phone_num, trade_id: trade.id)
  end

trade_address.rbで

address.createの中にfirst_name_kanaを入れてなかった