親のない、レコードの更新について

Question

#回答よろしくお願いします
エラーのページ

<div class="col-md-6 col-md-offset-3">  
     <h1>テスト作成</h1>
     <%= form_tag words_path, method: :create do %>
         <%= fields_for "words[]",@words, index: nil do |f| %>
            <%= render 'shared/error_messages', object: f.object %>

            <%= f.hidden_field :user_id, value:current_user.id %>

            <%= f.label :title,"タイトル"%>
            <%= f.text_field :title, class: 'form-control' %>

            <%= f.label :question,"問題"%>
            <%= f.text_area :question, class: 'form-control' %>

            <%= f.label :answer, "回答" %>
            <%= f.text_area :answer, class: 'form-control' %>

            <%= f.label :image,"ホーム画像" %>
            <%= f.file_field :image,class: 'form_control'%

         <% end %>

        <div id='key'></div>
        <%= link_to '問題を追加',add_path, remote: true ,class: 'btn btn-success'%>
         <br>
       <%= submit_tag '完了',class: "btn btn-primary"  %>
      <% end %>
  </div>
</div>

##バリデーションを描いた後、上のページだけエラーが出るようになった。

class Word < ActiveRecord::Base

  validates :question,length: { minimum: 1 }
  validates :answer,length: { minimum: 1 }
  validates  :answer ,length:{minimum:1}
  validates  :title ,length:{minimum:1}

  acts_as_taggable
  belongs_to :group
  belongs_to :user
  has_many  :favorites, dependent: :destroy

  def favorite?(user)
    favorites.where(user_id: user.id).exists?# exists?の説明 user.idが一致する人を探す。いたら、true いなかったら false
  end
end

コントローラー

def new
    @words = Word.new
  end

  def create
      word=words_params
      word.each do |word|
        @word = Word.create word
      end
      if @word.save
        flash[:success] = "success!"
        redirect_to @word
      else
        render 'new'
      end
  end

Comments

[moke]

少し情報が足りない気がします。controllerもさらしてください。あとvalidatesを全て削ってcreateできるか試してください

[NishidaRyu416_]

もちろん、バリデーションを書かなければ動きます。

[moke]

@wordを@wordsにすれば動くとは思いますが…

[moke]

過去の質問nested_form_fieldsというgemについてというのを見てやりたいことがわかりました。親のないレコードの複数更新をしたいのですね。

[moke]

rails(レールズ)はその名の通りレールから外れると大変です。親のないレコードを一括更新したいで再度質問を立てた方がいいかもしれません。

[NishidaRyu416_]

ありがとうございます。　@wordsにしたら、うまくいきました。

[moke]

今更ここまで作りこんだものを変える気はないでしょうが、次やる時の指針にでもなればと回答をしときますねー

Answer 1

ここに書きましたか
では方針だけでも
このままだとvalidationに失敗したレコードがどれかわからないと思うので

nested_formを使い一気にやる方法

①無理やり特に必要のない親レコードを作る作戦
word_basesとか作ってword_base_idをwordsに追加word_base_idは全部1とかpostごとに採番するとかテキトーで

②どうせ更新するときは〇〇だろ作戦
このケースの場合投稿日と投稿者ごとにしか更新しないだろ、とかあたりをつけてそれを親にして
posted_day,posted_user_idをComposite Primary Keysを使ってPrimary Keyにして紐付ける

③ActiveModelで親Modelを作って登録後破棄
登録更新時だけ親を作りゴミデータを残さない
親Model名_attributes methodを定義する必要がある

この中でいうと②がオススメですかね

Comments

[NishidaRyu416_]

いや、レコードもきちんと表示されます、完璧な解決策だったと思ったのですが・・

[moke]

複数登録する際に一番上のレコードのバリデーションを崩し
他を通るようにしてもsuccessって出て保存されていないような気がしますが…

[NishidaRyu416_]

いえ、複数保存できてます

[moke]

そうですか、ならいいのですが

この例でいくと例えば①つ目のレコードの:questionを空欄にして（バリデーションを崩して）
登録しようとすると①つ目が登録されずに②つ目いこうだけ登録されsuccessだけ表示されて
なぜ①つ目が登録されなかったのかユーザーに分からないような気がするのですが

[NishidaRyu416_]

そうですね、それは問題ですね。
バリデーションだけでなく、jsを使えばどうですかね？

[NishidaRyu416_]

そもそも、ボタンを押せないように、隠す等
しかし、jsがわからないんで、できませんが・・・