購入情報をDBに保存したいです。

Question

購入機能の実装のところで、購入情報をDBに保存したいのですが、そのままリダイレクトせずトップページに遷移せず、そのページに残ってしまいます。User must exist, Item must existtoiuというエラーメッセージがブラウザ上に表示されるだけの状態になってしまいます。
どなたかご教授願えますでしょうか？よろしくお願いします。

app/controllers/purchase_history_controller.rb

class PurchaseHistoriesController < ApplicationController
  def index
    @user_item = UserItem.new
    @item = Item.find(params[:item_id])
  end

  def create
    @item = Item.find(params[:item_id])
    @user_item = UserItem.new(purchase_history_params)
    @user_item = SendingDestination.create
    @user_item = PurchaseHistory.create
    if @user_item.valid?
      @user_item.save
      redirect_to root_path
    else
      render action: :index
    end
  end

  private

  def purchase_history_params
    params.permit(:nickname, :email, :encrypted_password, :first_name, :family_name, :first_name_kana, :family_name_kana, :birth_day, :image, :name, :introduction, :category_id, :item_condition_id, :postage_id, :prefecture_id, :prepare_id, :price, :post_code, :prefecture_id, :city, :house_number, :building_name, :phone_number)
  end
end

app/models/user_item.rb

class UserItem
  include ActiveModel::Model
  attr_accessor :nickname, :email, :encrypted_password, :first_name, :family_name, :first_name_kana, :family_name_kana, :birth_day, :image, :name, :introduction, :category_id, :item_condition_id, :postage_id, :prefecture_id, :prepare_id, :price, :post_code, :prefecture_id, :city, :house_number, :building_name, :phone_number

  with_options presence: true do
    validates :nickname
    validates :email
    validates :password, format: { with: /\A[a-zA-Z0-9]+\z/, message: "Include both letters and numbers."}
    validates :first_name, format: { with: /\A[ぁ-んァ-ン一-龥]/, message: "Full-width characters."}
    validates :first_name_kana, format: { with: /\A[ァ-ヶー－]+\z/, message: "Full-width katakana characters."}
    validates :family_name, format: { with: /\A[ぁ-んァ-ン一-龥]/, message: "Full-width characters."}
    validates :family_name_kana, format: { with: /\A[ァ-ヶー－]+\z/, message: "Full-width katakana characters."}
    validates :birth_day
    validates :image
    validates :name
    validates :introduction
    validates :price
    validates :post_code, format: {with: /\A[0-9]{3}-[0-9]{4}\z/, message: "is invalid. Include hyphen(-)"}
    validates :prefecture_id, numericality: { other_than: 0, message: "Select" }
    validates :city
    validates :house_number
    validates :phone_number, numericality: { only_integer: true, message: "Input only number" }
    validates :user
    validates :item
  end
  validates :price, numericality: { greater_than_or_equal_to: 300, less_than_or_equal_to: 9999999, message: "Out of setting range"}
  validates :price, numericality: { only_integer: true, message: "Half-width number." }

  
  with_options numericality: { other_than: 0, message: "Select" } do
    validates :item_condition_id
    validates :postage_id
    validates :prefecture_id
    validates :prepare_id
    validates :category_id
  end

  def save
    user =  User.create(nickname: nickname, email: email, encrypted_password: encrypted_password, first_name: first_name, family_name: family_name, first_name_kana: first_name_kana, family_name_kana: family_name_kana, birth_day: birth_day)
    Item.create(image: image, name: name, introduction: introduction, category_id: category_id, item_condition_id: item_condition_id, postage_id: postage_id, prefecture_id: prefecture_id, prepare_id: prepare_id, price: price, user_id: user.id)
    SendingDestination.create(post_code: post_code, prefecture_id: prefecture_id, city: city, house_number: house_number, building_name: building_name, phone_number: phone_number, purchase_history_id: purchase_history.id)
    PurchaseHistory.create(user_id: user.id, item_id: item.id)

  end

end

config/routes.rb

Rails.application.routes.draw do
  devise_for :users
  root "items#index"
  resources :items do
    resources :purchase_histories, only: [:index, :create]
  end
end

以下他のモデルです。

app/models/sendingdestination.rb

class SendingDestination < ApplicationRecord
  belongs_to :purchase_history, optional: true
  with_options presence: true do
    validates :post_code, format: {with: /\A[0-9]{3}-[0-9]{4}\z/, message: "is invalid. Include hyphen(-)"}
    validates :prefecture_id, numericality: { other_than: 0, message: "Select" }
    validates :city
    validates :house_number
    validates :phone_number, numericality: { only_integer: true, message: "Input only number" }
  end
end

app/models/purchase_history.rb

class PurchaseHistory < ApplicationRecord
  belongs_to :user, optional: true
  belongs_to :item, optional: true
  has_one :sending_destination
  
end

試したこと

デバッック(binding.pry)を利用して保存する情報がパラメーターに存在しているのかどうか見てみましたが、ちゃんと記録されているみたいでした。
条件分技のifの処理がtrueにならず、うまく行われていないのが原因みたいです。
optinal:trueを購入履歴のモデルファイルにつけくわてみました。

Answer 1

ありがとうございます。userとitemのエンティティ自体がいらなかったようです。formオブジェクトの記述時の仕方も間違っていたようで、saveメソッドの処理の中で購入履歴に関する情報を先に定義しその後に住所情報を定義するみたいでした。住所情報（sendingdestination）の定義の中に購入履歴（purchase_history）のidを入れるために、初めのpurchase_historyのインスタンス変数を作る必要があったみたいです。
あとストロングパラメーターの中に商品情報に関するidも含んでいなかったためエラーが絶えなかったようでした。
おかげさまでなんとかDB保存は成功したようです。お騒がせしてすいませんでした。ありがとうございました！

Answer 2

def create にデバッグの跡のようなコードがありますが、実際のコードは以下の状態だとすると、

  def create
    @item = Item.find(params[:item_id])
    @user_item = UserItem.new(purchase_history_params)
    if @user_item.valid?
      @user_item.save
      redirect_to root_path
    else
      render action: :index
    end
  end

@user_item.valid? が false になっているんだと思います。
この valid? というメソッドは、バリデーションを通るなら true 通らないなら false を返します。

バリデーションは class UserItem にずらっと書いてありますが、
User must exist, Item must exist に該当する部分が見当たりません。
なにか心当たりはありませんか？

Comments

[murohi-08]

ご回答ありがとうございます。
userとitemのバリデーションを組んでみましたが、変化ありません。他にどこか記述がおかしい箇所はありますでしょうか？

[neko_daisuki]

追加されたバリデーションの記述は元からあったのでしょうか？
元からあって User must exist, Item must exist と出るのなら分かるんですが、
なかったのに出るならなんかおかしいですね・・・

あとviewにnicknameやemailなどユーザーに関連する情報がないのに、
それを受け取って保存しようとしているのも不思議です。

[murohi-08]

現在Formオブジェクトパターンを使って、複数のテーブルにデータを保存するという実装をやっており、nicknameやemailなどのカラムは新規登録のviewにおいてあります。
今他のモデルにoptional :trueというUser must exist, Item must exist のエラーメッセージを消すオプションを使って消したのですが、今度は上記のエラーが発生します。
編集したのでご確認の程よろしくお願いします。

[neko_daisuki]

このエラーは、optional: true にしたことでユーザーが空でもPurchaseHistoryのバリデーションは通ったけど、
MySQL の外部キー制約かNOT NULL制約に引っかかってます。

新規登録と同じフォームモデルを使いまわしては上手くいきません。
PurchaseHistoryを作成したいだけでも、新規登録に必要なのと同じパラメーターも必要になってしまいます。

[murohi-08]

つまりコントローラーのパラメーターの部分の記述がおかしいということでしょうか？
どの箇所を直した方がよろしいでしょうか？ご教授願います。

[neko_daisuki]

PurchaseHistoryを作成するのに UserItem を使うのをやめたら良いんじゃないかと思います。
そうすればコントローラーの記述も変わり以下のようになるでしょう。

@p = PurchaseHistory.new(purchase_history_params)
@s = SendingDestination.new(sending_destination_params)
if @p.valid? && @s.valid?

def purchase_history_params
params.permit(purchase_historyのカラムだけ書く)
end

def sending_destination_params
params.permit(sending_destination のカラム)
end