バリデーションエラーメッセージを表示させたい

Question

実現させたいこと

バリデーションエラーでフォーム画面に戻ってきたときに、バリデーションエラーメッセージを表示させたい。

【ビュー】
<%= form_tag("/time_cards/#{params[:id]}", method: "patch" ) do %>

...

<% @time_cards.each.with_index(1) do |time_card, i| %>
  <tr>
    <td><%= params[:month] %>/<%= i %></td>
    <td><%= get_weekday(params[:year], params[:month], i) %></td>
    <%= fields_for "time_cards[]", time_card do |f| %>
      <%= f.error_messages %>
      <td><%= f.time_field :in_at %></td>
      <td><%= f.time_field :out_at %></td>
      <td><%= get_stay_time(params[:id], params[:year], params[:month], i) %></td>
      <td><%= f.text_field :remarks %></td>
    <% end %>
  </tr>
<% end %>

・・・
<% end %>

【コントローラー】
def edit
    today = Date.current
    
    (1..view_context.get_days).each do |day|
      TimeCard.find_or_create_by(user_id: params[:user_id], date: "#{params[:year]}-#{params[:month]}-#{day}")
    end
    @time_cards = TimeCard.where(user_id: params[:user_id]).where(date: today.in_time_zone.all_month).order("date")
end

def update
    @time_cards = time_card_params.keys.each do |id|
      time_card = TimeCard.find(id)
      if !time_card.update_attributes(time_card_params[id]) then
        render action: :edit
      end
    end
    flash[:success] = "完了しました。"
    redirect_to action: 'show', user_id: params[:id], year: params[:year], month: params[:month]
end

editアクションで@time_cardsを取得してビューで表示させていますが、
バリデーションエラーによりrenderでビューを表示した時に、@time_cardsを取得する過程が無いため、
@time_cardsなんて無い！というエラーなのだと考えています。
ただどうコードを書けば解決できるかわかりません。
教えていただけないでしょうか。

Answer 1

@time_cardsを取得する過程が無いため

そこまでわかっているなら簡単です @time_cards を取得する過程があれば良いのです
また、件数分 find していた処理が１回の where で済むのでパフォーマンスも良いです

def update
  @time_cards = TimeCard.where(id: time_card_params.keys)

  @time_cards.each do |time_card|
    if !time_card.update_attributes(time_card_params[time_card.id])
      render action: :edit
    end
  end
end

補足: 失敗時の挙動を考えることを推奨

ただし上記では以下のような問題が起きる可能性があります

1. 最初に失敗したデータしかメッセージがでず、失敗後の入力が消える（失敗時点で render）

　全て更新処理を行って、一つでも失敗があれば render という処理にすべきでは？

1. 成功した部分は保存されてしまう

　全て更新処理を行って、一つでも失敗があればデータはロールバックすべきでは？

本筋ではないので詳細は省きますが、本題の描画が成功したら確認してみるのはどうでしょう

Comments

[awa]

おかげさまでエラーメッセージを表示することができました。
ありがとうございます。
補足で挙げてくださった内容は、確かに解決すべき問題なので、
こちらも着手します。
ありがとうございました。