Ruby on Railsでformで値を送信するが、保存されない

Question

Ruby on Railsでショッピングカート機能を実装しています。
商品一覧画面から、商品の個数を選択してcartテーブルに保存したい状況です。
※cartテーブルはsessionで仮に商品を保存するためのテーブルとなってます。

そして商品一覧indexのコントローラーで、cart.newを記述して、cartのコントローラーで
createメソッドに飛ばしています。

環境：
rails 5.2.3
gem devise機能を使用

---

index前のコントローラーになります。
class BoxlunchesController < ApplicationController
  def index
    @boxlunches = Boxlunch.all
    @cart = Cart.new
  end

  def show
  end
end

---

課題：
商品一覧画面から、商品の個数(boxlunch_quantity)を選択し、formで値を送信するが、boxlunch_quantityのみ保存されない
※boxlunch_idはcart_itemテーブルに保存されます。

  index.html.haml

 = render 'layouts/header'
= render "./template/menu_selections"
.menu-container
  %h2 Boxlunch Menu
  .menu-contents
    - @boxlunches.each do |boxlunch|
　　　　#cart_index_pathでcartのcreateに飛びます
      = form_for @cart, url: cart_index_path(boxlunch_id: boxlunch.id), method: :post do |f|
        .menu-package
          .menu-image
            = link_to "boxlunch/#{boxlunch.id}" do
              = image_tag boxlunch.image, width:"200px", height:"150px"
            %h4
              = link_to do
                = boxlunch.name
            .count-price-container
              .count-price
                .price
                  %p ¥#{boxlunch.price}(税込)
                .count
                  %label
                    =f.select :boxlunch_quantity, options_for_select(1..10), include_blank: '0'
              =f.submit "カートの中に入れる"
= render 'layouts/footer'

---

エラー内容
ブラウザーではエラーは表示されていないので
ターミナルでの記載になります。

ターミナル
Started POST "/cart?boxlunch_id=1" for ::1 at 2020-01-13 23:49:36 +0900
Processing by CartController#create as HTML
  Parameters: {"utf8"=>"✓", "authenticity_token"=>"O5g+me6pcK+uueZYWtdRBmkFdj+TzXqkyZRyw+q5KvkcqQ3wGLzM05qi79LfoOL9vd7aI54B78hSBNdXYOML2A==", "cart"=>{"boxlunch_quantity"=>"1"}, "commit"=>"カートの中に入れる", "boxlunch_id"=>"1"}
  Order Load (2.0ms)  SELECT  "orders".* FROM "orders" WHERE "orders"."id" IS NULL LIMIT $1  [["LIMIT", 1]]
  ↳ app/controllers/application_controller.rb:16
   (0.8ms)  BEGIN
  ↳ app/controllers/application_controller.rb:17
   (1.1ms)  ROLLBACK
  ↳ app/controllers/application_controller.rb:17
  Boxlunch Load (0.7ms)  SELECT  "boxlunches".* FROM "boxlunches" WHERE "boxlunches"."id" = $1 LIMIT $2  [["id", 1], ["LIMIT", 1]]
  ↳ app/controllers/cart_controller.rb:4
  Order Load (1.6ms)  SELECT  "orders".* FROM "orders" WHERE "orders"."id" IS NULL LIMIT $1  [["LIMIT", 1]]
  ↳ app/controllers/application_controller.rb:16
   (1.8ms)  BEGIN
  ↳ app/controllers/application_controller.rb:17
   (0.4ms)  ROLLBACK
  ↳ app/controllers/application_controller.rb:17
   (1.0ms)  BEGIN
  ↳ app/controllers/cart_controller.rb:6
  Cart Create (0.5ms)  INSERT INTO "carts" ("boxlunch_id", "created_at", "updated_at") VALUES ($1, $2, $3) RETURNING "id"  [["boxlunch_id", 1], ["created_at", "2020-01-13 14:49:36.819147"], ["updated_at", "2020-01-13 14:49:36.819147"]]
  ↳ app/controllers/cart_controller.rb:6
   (1.9ms)  COMMIT
  ↳ app/controllers/cart_controller.rb:6
  User Load (0.3ms)  SELECT  "users".* FROM "users" WHERE "users"."id" = $1 ORDER BY "users"."id" ASC LIMIT $2  [["id", 1], ["LIMIT", 1]]
  ↳ app/controllers/cart_controller.rb:7
Redirected to http://localhost:3000/cart/1
Completed 302 Found in 46ms (ActiveRecord: 12.1ms)


Started GET "/cart/1" for ::1 at 2020-01-13 23:49:36 +0900
Processing by CartController#show as HTML
  Parameters: {"id"=>"1"}
  Rendering cart/show.html.haml within layouts/application
  Rendered cart/show.html.haml within layouts/application (1.9ms)
Completed 200 OK in 63ms (Views: 60.4ms | ActiveRecord: 0.0ms)

---

アプリケーションコントローラーのコード
※current_cartを定義しています。

  class ApplicationController < ActionController::Base
  protect_from_forgery with: :exception
    before_action :authenticate_user!


  def after_sign_in_path_for(resource)
    root_path # ログイン後に遷移するpathを設定
  end

  def after_sign_out_path_for(resource)
    root_path # ログアウト後に遷移するpathを設定
  end
  
  private
  def current_cart
    @current_cart = Order.find_by(id: session[:order_id])
    @current_cart = Order.create unless @current_cart
    session[:order_id] = @current_cart.id 
    @current_cart
  end
end

---

![Model ER図改正]

カートのモデルコード

class Cart < ApplicationRecord
  belongs_to :order
  has_many :boxlunches


  def add_boxlunch(boxlunch_id)
    cart.find_or_initialize_by(boxlunch_id: boxlunch_id)
    end
  
    def add_single_menu(single_menu_id)
      cart.find_or_initialize_by(single_menu_id: single_menu_id)
    end
end

---

カートのコントローラーのコード

class CartController < ApplicationController
    def create
      cart = current_cart
      boxlunch = Boxlunch.find(params[:boxlunch_id])
      @cart = current_cart.add_boxlunch(boxlunch.id)
      if @cart.save
        redirect_to cart_path(current_user)
      else
        render :new
      end
    end
  
    private
    def cart_params
      params.require(:cart).permit(:boxlunch_quantity);
    end
  end

---

以上のような状況です。
お忙しい中ですが、教えていただけますと助かります。

宜しくお願いいたします。

Comments

[winterboum]

プログラム類は 逆シングルコーテーション３つの間にいれてインデントが判るようにしてください。
インデント命のhamlが、、、、
RUBYのcodeも読みにくい。
読みにくいと読んでもらえないので、回答がつきにくいです

[winterboum]

teisei

[nissyan]

winterboumさん
ご回答ありがとうございます。

①order_idが代入されるのは、17行目のsession[:order_id] = @current_cart.id です。
loginの時に定義しておりません。

②orderテーブルは、boxlunchとcart_itemの中間テーブルに位置するものです。
そして、logを撮った時とコードは同じです。

すみません！逆シングルコーテーション３つの間にいれてインデントが判るようにするということがわからないです。
""を``に変更するということですか。　　3つの間とはどうゆうことですか？
以上、ご確認よろしくお願いします。

[nissyan]

モデルの関係性をER図で書きましたので、一度ご確認していただけたら
嬉しいです。

cart_itemの方は、index.htmlに遷移する前のコントローラーで,
cart_item.newをしています。
order.newも必要でしょうか？
index.htmlの前のコントローラもコードを追記しておきます。

すみませんが、ご確認お願いいたします。

[winterboum]

Boxlunch　と Order をつなぐものが CartItemというのはいただけません。
CarItemという名前からは CartとItemをつなぐものと誤解されます、
これは直したほうがよい

[winterboum]

OrderとAddressの関連はない方が自然です。
Addressが沢山Orderをもつとか、Orderがaddressに属するというのは不自然。
Orderの送り先という考えかと思いますが、OrderしたのはUser,そのaddressへ送ればよいのです

Answer 1

application_controller.rb の 15,16,17行目ってこれですか？

@current_cart = Order.find_by(id: session[:order_id])
@current_cart = Order.create unless @current_cart
session[:order_id] = @current_cart.id 

この session[:order_id]　が最初に代入されるのはいつ、どのfileのどのmethodでしょう？
もしかして この１７行目？
すると最初に １５行目に来た時はまだ未定義ですから@carrent_cartはnilです。
もしかするとloginの時に定義してますか？
そこを確認してください

追記
不思議、、、、
(0.2ms) BEGIN
↳ app/controllers/cart_items_controller.rb:7
CartItem Create (0.9ms) INSERT INTO "cart_items" ("boxlunch_id", "created_at", "updated_at") VALUES ($1, $2, $3) RETURNING "id" [["boxlunch_id", 2], ["created_at", "2020-01-12 14:54:40.381901"], ["updated_at", "2020-01-12 14:54:40.381901"]]
↳ app/controllers/cart_items_controller.rb:7
(0.5ms) COMMIT
とあるのでCartItemは作られています。ただ見て判るように order がなになのかわからない状態で。
ところが、CartItemは　belongs_to :order　になっているので、order_idなしにsave出来ないはずです。
もしかしてこのlogを取った時は載せているプログラムとは違ってる？

追記２
＞①order_idが代入されるのは、17行目のsession[:order_id] = @current_cart.id です。
＞loginの時に定義しておりません。
とすると　current_cart　は永遠に nil です。

どこかで order なり cart なり確定する必要があります。

User, Order, Cart の関係を教えてください。

追記の３
モデル間の関連やモデル名の不自然なところはコメントで指摘しています。
さて
Orderが1:多だとすると、Order_idが決まるのはいつ、どこでですか？
その設計が無いようです。

n~~~結構厄介そう
Userと1:1なCartを作り、CartBoxLunchで注文を取り、注文確定したらOrderを作る
のほうが作りやすそう。

Comments

[nissyan]

＠winterboumさん
Userと1:1なCartを作り、CartBoxLunchで注文を取り、注文確定したらOrderを作る
のほうが作りやすそう。
・・・そうなんですね！もう少しこの状態で問題解決を頑張らせてもらってもいいですか？もし、今日明日無理なら方向変えます。あと、コメントで指摘していただけたとこを修正させていただきました。

すみませんが二つ質問させてください。
①Order_idを決める設計ということに対しての質問です。
Cartcontrollerのcreateでbinding.pryで値を確認してみました。
cart = current_userでは、orderインスタンスが表示され、すべてレコードの中身がすべてnillの状態でした。
これにidを入れる必要があると言うことでしょうか？

②formで値の送信しましたが、意図しない値が返ってきました。
商品一覧のindexで、formを使用して、弁当の個数をselectタグで送っています。
cartcontrollerのcreateで、binding.pryを使用してparams確認をしたところ、"cart"=>{"boxlunch_quantity"=>"1"}、 "commit"=>"カートの中に入れる", "boxlunch_id"=>"1"と表示されました。

なぜcartの中に弁当の個数がハッシュで入っているのか？の原因の見つけかたを教えていただきたいです
boxlunch_id => "1"のように、値をとれないので、テーブルに保存されないのではと考えています。

文章が長くなってしまいましたが、宜しくお願いいたします。

[winterboum]

①それ以前の問題です。どのタイミングでOrderが作られてますか？
沢山有るOrderのなかでどのOrderが current_cartを決めるのに使われますか？
それがあやふや
② form_for @cartid),。。。 do |f|　の中に　=f.select :boxlunch_quantity,　があるからです

このcodeはモデル名から想像される役割と実際の役割がまるで異なるのでともすると勘違いします。そこを治すのを最初にやる方が良いです。
cart ってその中に沢山ものを入れるものですよね。
それが品物と1:1になってる。　スーパーに行ってカートを沢山ガラガラ引いて、１品種毎に１カート使ってる。というイメージになります。そういうのはしないから、そういうCartの使い方だとcodeを読むとき混乱します。
名前と役割を一致させるのって大事です。
まずそれから着手しましょう

[nissyan]

＠winterboumさん
アソシエーションの設計を見直しER図書き直しました。
これでもう一度これで組みなおそうと考えています。
上記にあったER図を変更しています
一度ご確認いただけたら幸いです。

二日間もご指導いただき本当に感謝しております。
そして同じ問題を抱えた人にER図でわかりやすく見ていただけたら幸いです。

[winterboum]

だいぶすっきりしてきましたね。
二つ懸念があります。
1)Cartってどのように使うのでしょうか？　私の間隔だとOrderとCartが逆。
2)商品タイプ毎にmodelを作り、しまうidをOrderにもCartにも固定的に作ってます。
　これですと
a)商品タイプが増えた場合DBの作り直しから必要になる
b)２種類のBoxランチの注文とかが出来ません

[nissyan]

@winterboumさん
winderboumさんのおかげで、自分でもすっきりにER図を整理できたと感じています。
1)Orderで仮注文→Cartにいれると考えでしたが、Cartに入れる→注文決定Orderの方がわかりやすそうですね！
変更します。
2)各4つの商品をまとめるテーブル(Productテーブル)を作成するやり方はどうでしょうか？

[winterboum]

2)わたしでしたら、商品は全部一つのtableにして、商品カテゴリを示す項目を増やします。

[nissyan]

＠winterboumさん
ご回答ありがとうございます。
勉強不足でした。
そうゆうやり方があるのですね！

そのやり方を調べて実装してみます。
長い間お付き合い本当にありがとうございました。