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

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

ただいまの
回答率

88.64%

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

解決済

回答 1

投稿

  • 評価
  • クリップ 1
  • VIEW 1,953

hatt

score 14

動作環境

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オブジェクト

class SirabasuForm
  include Virtus.model
  include ActiveModel::Model

  attr_accessor :number, :name, :content, :userid, :cid, :image_path

  validates :name, :content, presence: true

  def initialize(attr = {}) #中身を書いていなくてもロールバックする
#updateを行うために準備
     unless attr["number"].nil?
       @sirabasu = Sirabasu.find_by(number: attr["number"])
       self[:name] = attr[:name].nil? ? @sirabasu.name : attr[:name]
       self[:content] = attr[:name].nil? ? @sirabasu.content : attr[:name]
     end
  end

  def save(sirabasu_params)
    sirabasu = Sirabasu.new(name: name, content: content, number: number, userid: userid, cid: cid)
    unless sirabasu_params[:image_path].nil? #画像複数登録
      sirabasu_params[:image_path].each do |path|
        sirabasu.images.new(image_path: path)
      end
    end
    sirabasu.save    #<---コンソール上で指摘されたのはここです
  end

  def update(sirabasu_params)

  end
end

 controller(Sirabasu、一部)

  def new
      @sirabasu_form = SirabasuForm.new
  end

  def create
    @sirabasu_form = SirabasuForm.new(sirabasu_params)
    if @sirabasu_form.save(sirabasu_params)
      redirect_to('/sirabasus')
    else
      render 'new'
    end
  end

  def edit
      @sirabasu_form = SirabasuForm.new("number" => params[:id])
  end

  def update
    @sirabasu_form = SirabasuForm.new(sirabasu_params.merge("number" => params[:id]))
    if @sirabasu_form.update(sirabasu_params)
      redirect_to action: 'index'
    else
      render 'edit'
    end
  end

  def sirabasu_params
    params.require(:sirabasu_form).permit(:number, :name, :content, :userid, :cid, image_path: [])
  end
  • 気になる質問をクリップする

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

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

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

+1

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

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

class SirabasuForm
  include Virtus.model
  include ActiveModel::Model

  attr_accessor :number, :name, :content, :userid, :cid, :image_path

  validates :name, :content, presence: true # テーブルでnot null制約をかけている属性を全て追記していいと思います。

  def initialize(attr = {}) #中身を書いていなくてもロールバックする
#updateを行うために準備
     unless attr["number"].nil?
       @sirabasu = Sirabasu.find_by(number: attr["number"])
       self[:name] = attr[:name].nil? ? @sirabasu.name : attr[:name]
       self[:content] = attr[:name].nil? ? @sirabasu.content : attr[:name]
     end
  end

  def save(sirabasu_params)
    return false if invalid? # ここにバリデーションを追加します

    sirabasu = Sirabasu.new(name: name, content: content, number: number, userid: userid, cid: cid)
    unless sirabasu_params[:image_path].nil? #画像複数登録
      sirabasu_params[:image_path].each do |path|
        sirabasu.images.new(image_path: path)
      end
    end
    sirabasu.save    #<---コンソール上で指摘されたのはここです
  end

  def update(sirabasu_params)

  end
end

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

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/11/19 00:10

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

    キャンセル

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

  • ただいまの回答率 88.64%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る