質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.50%
Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

Q&A

解決済

1回答

820閲覧

[rails]accepts_nested_attributes_for使った入力フォームの情報が保存できない

gawao

総合スコア5

Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

1グッド

0クリップ

投稿2020/01/18 10:58

現在個人でWEBサービスを開発している者です。
◾️やりたいこと
railsで、accepts_nested_attributes_forを使って子モデルも一緒に保存したい
◾️課題
入力フォームに値を入れてsubmitしてもDBに値が保存されない
(エラー等が起きるわけでもなく、画面もしない)

イメージ説明

お手数ですが、有識者の皆様のお力添えいただけますと幸いです。
サイト等みながら色々と試してみた関係で、
不要なコードやDB項目が含まれているかもしれませんがご了承ください。
また、不足している情報があればお出ししますのでご教示ください。

◾️テーブル一覧
概要:ユーザー(user)が、プラン名(plan)を投稿します。同時にnested modelのプラン詳細(posts)を登録するというイメージです。
例:ユーザ1(user)が箱根1泊旅行(plan)と同時に時間等(posts)を登録
users:
("id" INTEGER PRIMARY KEY AUTOINCREMENT NOT NULL, "name" varchar, "email" varchar, "created_at" datetime NOT NULL, "updated_at" datetime NOT NULL, "image_name" varchar, "password_digest" varchar);
plans:
("id" INTEGER PRIMARY KEY AUTOINCREMENT NOT NULL, "plan_name" varchar, "user_id" integer, "created_at" datetime NOT NULL, "updated_at" datetime NOT NULL);

posts:
("id" INTEGER PRIMARY KEY AUTOINCREMENT NOT NULL, "content" text, "created_at" datetime NOT NULL, "updated_at" datetime NOT NULL, "user_id" integer, "plan_id" integer, "plan_date" varchar, "arrival_time" varchar, "place_image" varchar, "place_name" varchar);

◾️ソース一覧
★plan.rb

class Plan < ApplicationRecord validates :user_id, {presence: true} belongs_to :user has_many :posts accepts_nested_attributes_for :posts end

★post.rb

class Post < ApplicationRecord validates :user_id, {presence: true} belongs_to :plan def user return User.find_by(id: self.user_id) end end

★user.rb

class User < ApplicationRecord has_secure_password validates :email,{uniqueness: true} validates :password,{presence: true} has_many :plans has_many :posts def posts return Post.where(user_id: self.id) end end

★/app/views/plans/new.html.erb

<div class="main posts-new"> <div class="container"> <h1 class="form-heading">プランを作成する</h1> <%= form_for @plan,:url => {:controller => :plans ,:action => :create} do |f| %> <div class="form"> <div class="form-body"> <div class="field"> <%= f.label :プラン名 %><br> <%= f.text_field :plan_name %> </div> <div class="field"> <%= f.label :posts, "プラン詳細" %><br /> <%= f.fields_for :posts do |posts_form| %> <%= posts_form.hidden_field :user_id, value: @current_user.id %> <%= posts_form.label :日付 %> <%= posts_form.date_field :plan_date %><br /> <%= posts_form.label :到着時間 %> <%= posts_form.time_field :arrival_time %><br /> <%= posts_form.label :地名 %> <%= posts_form.text_field :place_name %><br /> <%= posts_form.label :やること %> <%= posts_form.text_field :content %><br /> <% end %> </div> <%= f.submit "作成" %> </div> <% end %> </div>

★/app/controllers/plans_controller.rb

class PlansController < ApplicationController before_action :authenticate_user def create @plan = Plan.create(plan_params) if @plan.save flash[:success] = "投稿が完了しました!" redirect_to("/posts/index") else flash.now[:alert] = "投稿が失敗しました。" #render end end def new @plan = Plan.new @plan.posts.build end private def plan_params params.require(:plan).permit( :plan_name, :prefecture_id, posts_attributes: [:id, :plan_id, :user_id, :plan_date, :arrival_time, :place_name, :place_image, :content, :_destroy] ).merge(user_id: @current_user.id) end end

★route.rb

Rails.application.routes.draw do post 'users/create' => 'users#create' get "signup" => "users#new" get 'users/index' => 'users#index' get 'users/:id' => 'users#show' get 'users/:id/edit' => 'users#edit' get 'users/:id/likes' => 'users#likes' post 'users/:id/update' => 'users#update' post 'users/:id/destroy' => 'users#destroy' get 'login' => 'users#login_form' post 'login' => 'users#login' post 'logout' => 'users#logout' get 'posts/index' => 'posts#index' get 'posts/new' => 'posts#new' get 'posts/:id' => 'posts#show' post 'posts/create' => 'posts#create' get 'posts/:id/edit' => 'posts#edit' post 'posts/:id/update' => 'posts#update' post 'posts/:id/destroy' => 'posts#destroy' get 'plans/new' => 'plans#new' post 'plans/create' => 'plans#create' get '/' => 'home#top' get '/about' => 'home#about' # For details on the DSL available within this file, see http://guides.rubyonrails.org/routing.html end

◾️参考にしたサイト1
◾️参考にしたサイト2

shinoharat👍を押しています

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

shinoharat

2020/01/18 11:57

ざっと見ましたが、コードに大きな問題はなさそうな気がします。不思議ですね・・・。 ひとつ気になったのが、 create アクションの先頭部分です。 ``` @plan = Plan.create(plan_params) if @plan.save  ・・・ ``` みたいに書いてますが、これだと Plan.create で DB に保存した後、もう一回 save する形になってしまいます。 なので、 ``` @plan = Plan.new(plan_params) if @plan.save  ・・・ ``` とするのがいいかなと思います。
shinoharat

2020/01/18 12:12

一応、上に書いた部分を修正した上でもう一度確認していただいて、それでも動かなければ、以下の情報を追記お願いします。 【1】create アクションの先頭で params の値が何になってるか教えてください。 【2】create アクションの先頭で plan_params の戻り値が何になってるか教えてください。 【3】 ・Plan は保存されるが Posts が保存されない ・Plan も Posts も保存されない のどちらでしょうか? 【4】 > (エラー等が起きるわけでもなく、画面もしない) とありますが、「画面遷移もしない」の書き間違いでしょうか? つまり、作成ボタンを押した後の挙動は ・入力画面が再度表示される ・"投稿が失敗しました。" の flash メッセージが表示される ・しかし、バリデーションエラーは表示されない でしょうか? 【5】「新規作成時に DB に保存されない」というお悩みで間違いないですよね?(「新規はできるけど更新はできない」という話ではないですよね?)
shinoharat

2020/01/18 12:26

もう一つ気になる部分をみつけました。 app/views/plans/new.html.erb に、バリデーションエラーを表示するコードが無いように見えます。 もし未実装なら、以下のコードを追加してみてください。 ``` <% if @plan.errors.any? %> <div id="error_explanation"> <ul> <% @plan.errors.full_messages.each do |msg| %> <li><%= msg %></li> <% end %> </ul> </div> <% end %> ``` この状態で再度保存を実行すると、保存失敗の原因特定のヒントが得られるかもしれません。
gawao

2020/01/18 12:38

shinoharatさん 色々とご確認ありがとうございます!!!泣 先に回答できるものからお答えします!!!! ◾️1つ目のコメント @plan = Plan.new(plan_params)を試しましたが変わらずです。 ◾️2つ目のコメント 【1】【2】→申し訳ございませんが確認方法が分からずなのでもう少々お時間いただけますでしょうか、、 【3】 >・Plan は保存されるが Posts が保存されない >・Plan も Posts も保存されない >のどちらでしょうか? →どちらもです。 【4】 「画面遷移もしない」の書き間違いです、、、 >・入力画面が再度表示される →正しいです。 >・"投稿が失敗しました。" の flash メッセージが表示される →flashも表示されません >・しかし、バリデーションエラーは表示されない →正しいです。 【5】 「新規作成時に DB に保存されない」というお悩みで間違いないですよね?(「新規はできるけど更新はできない」という話ではないですよね?) →新規作成時にDBに保存されないで正しいです! ◾️3つ目のコメント 追記してみましたが、バリデーションエラーも表示されません、、、
gawao

2020/01/18 12:54

追記です。 ◾️3つ目のコメント plans_controller.rbで、render("plans/index")を追記すると、下記のメッセージが表示されました!! 「Posts plan must exist」 相変わらずflashメッセージは表示されません、、
gawao

2020/01/18 13:02

保存できました!!! optional: trueをbelongs_toに付けたら保存はできたのですが、本来このような書き方は良いのですかね?? belongs_to :user, optional: true belongs_to :plan, optional: true
guest

回答1

0

ベストアンサー

投稿2020/01/19 05:52

no1knows

総合スコア3365

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

gawao

2020/01/19 23:39

共有ありがとうございます。 あまり使うべきメソッドではないのですかね〜、、? まだまだ勉強が必要です、、 いずれにしても、様々な助言ありがとうございました、大変助かりました!
no1knows

2020/01/20 00:24

僕は初学者なのでなんとも言えないですが… どうしてもじゃなければ積極的に採用しない方向にしたいと思います。 将来的になくなる≒バージョン上げる際の負債になりそうといったコメントも頷けるので…
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問