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

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

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

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

Ruby

Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

Q&A

解決済

1回答

3599閲覧

【RubyOnRails5】Formオブジェクトを用いたModel操作

退会済みユーザー

退会済みユーザー

総合スコア0

Ruby on Rails 5

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

Ruby

Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

0グッド

1クリップ

投稿2018/11/09 11:41

###動作環境
ruby 2.4.4p296
Rails 5.2.0

現在行っていること・問題

・フォームオブジェクトを用いて、2つのモデルに対してnew、create、edit、updateを行います。
SirabasuモデルとImageモデルがあり、1対多の関連付けがあります。
new、createはうまくいきました。
・edit、updateを実装しようと、初期化用にinitializeを設けたところで問題が発生しました。
・それまでうまくいっていたcreaterollback transactionが起こり、DBに保存できなくなりました。中身を一切書いていなくても、発生します。

調べても原因がわからなかったので、質問させていただきました。お詳しい方々、どうか回答のほど、よろしくおねがいします。

参考にしたサイト

(How) Can I use a form object for the edit/update routines? - stack overflow

Formオブジェクト

ruby

1class SirabasuForm 2 include Virtus.model 3 include ActiveModel::Model 4 5 attr_accessor :number, :name, :content, :userid, :cid, :image_path 6 7 validates :name, :content, presence: true 8 9 def initialize(attr = {}) #中身を書いていなくてもロールバックする 10#updateを行うために準備 11 unless attr["number"].nil? 12 @sirabasu = Sirabasu.find_by(number: attr["number"]) 13 self[:name] = attr[:name].nil? ? @sirabasu.name : attr[:name] 14 self[:content] = attr[:name].nil? ? @sirabasu.content : attr[:name] 15 end 16 end 17 18 def save(sirabasu_params) 19 sirabasu = Sirabasu.new(name: name, content: content, number: number, userid: userid, cid: cid) 20 unless sirabasu_params[:image_path].nil? #画像複数登録 21 sirabasu_params[:image_path].each do |path| 22 sirabasu.images.new(image_path: path) 23 end 24 end 25 sirabasu.save #<---コンソール上で指摘されたのはここです 26 end 27 28 def update(sirabasu_params) 29 30 end 31end

controller(Sirabasu、一部)

ruby

1 def new 2 @sirabasu_form = SirabasuForm.new 3 end 4 5 def create 6 @sirabasu_form = SirabasuForm.new(sirabasu_params) 7 if @sirabasu_form.save(sirabasu_params) 8 redirect_to('/sirabasus') 9 else 10 render 'new' 11 end 12 end 13 14 def edit 15 @sirabasu_form = SirabasuForm.new("number" => params[:id]) 16 end 17 18 def update 19 @sirabasu_form = SirabasuForm.new(sirabasu_params.merge("number" => params[:id])) 20 if @sirabasu_form.update(sirabasu_params) 21 redirect_to action: 'index' 22 else 23 render 'edit' 24 end 25 end 26 27 def sirabasu_params 28 params.require(:sirabasu_form).permit(:number, :name, :content, :userid, :cid, image_path: []) 29 end

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

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

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

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

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

guest

回答1

0

ベストアンサー

rollbackした後のログはどのようなメッセージが出ていますか?保存できなかった理由が表示されていると思います。

仮説ですがsaveするためのデータが足りないのだとするとsaveの前にバリデーションを実行すると気付けるかもしれません。
以下ではnameとcontentがnilのときはfalseを返してsaveしません。

Ruby

1class SirabasuForm 2 include Virtus.model 3 include ActiveModel::Model 4 5 attr_accessor :number, :name, :content, :userid, :cid, :image_path 6 7 validates :name, :content, presence: true # テーブルでnot null制約をかけている属性を全て追記していいと思います。 8 9 def initialize(attr = {}) #中身を書いていなくてもロールバックする 10#updateを行うために準備 11 unless attr["number"].nil? 12 @sirabasu = Sirabasu.find_by(number: attr["number"]) 13 self[:name] = attr[:name].nil? ? @sirabasu.name : attr[:name] 14 self[:content] = attr[:name].nil? ? @sirabasu.content : attr[:name] 15 end 16 end 17 18 def save(sirabasu_params) 19 return false if invalid? # ここにバリデーションを追加します 20 21 sirabasu = Sirabasu.new(name: name, content: content, number: number, userid: userid, cid: cid) 22 unless sirabasu_params[:image_path].nil? #画像複数登録 23 sirabasu_params[:image_path].each do |path| 24 sirabasu.images.new(image_path: path) 25 end 26 end 27 sirabasu.save #<---コンソール上で指摘されたのはここです 28 end 29 30 def update(sirabasu_params) 31 32 end 33end 34

余談ですが、editのときにあらかじめ属性を渡すときはコントローラでSirabasuFormのインスタンスを生成する時に渡してしまうのがいいかと思います。
こうすれば特にintializeを使わなくても動作すると思います。
image_pathもインスタンスをnewする時に渡せばsaveの引数で渡す必要はなさそうですね。

Ruby

1def edit 2 sirabasu = Sirabasu.find_by(number: params[:id]) 3 @sirabasu_form = SirabasuForm.new(number: siranasu.number, name: sirabasu.name, content: sirabasu.content) 4end

投稿2018/11/16 09:28

KiyoKuro

総合スコア51

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

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

退会済みユーザー

退会済みユーザー

2018/11/18 15:10

回答ありがとうございます。バリデーションは考えていませんでした。試してみます!
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

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

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

質問する

関連した質問