[Rails6] updateで発生したバリデーションエラーをindexで表示したい

Question

前提

Ruby 2.7.2
Rails 6.0.3.4
gem 'bootstrap_form'を利用

状況をシンプルにするためにscafforldでpost title:stringのみ作った状況です。
実際にやりたいのはpostではありませんが、同意であり、わかりやすいのでpostとして進めています。
使っていないcreateやdestroyは消してあります。

実現したいこと

1. indexにて、postsの個別の編集ができる
2. indexから編集した時、遷移先はindex
3. 失敗時、バリデーションエラーがviewに反映される
4. 遷移後のURLはindex
5. editにて個別編集もできる
6. @posts = Post.allの量は多くないものとする（100以下）

上記の条件をクリアした実装はできたが、良い実装なのか不安です。

実装方法

例: indexからpost(id:1)のupdateが失敗した時

1. flash[:error]にpost.errorsの値を渡す
2. redirect_to posts_pathでindexにリダイレクトする
3. indexの@postsからposts(id:1)を探す
4. @posts(id:1)にerrorsを追加（flash[:error]からエラー内容を取得する）
5. @posts(id:1).errorsに値が入っているため、view上にエラーが反映する

疑問

1.画面の遷移先をrefererで確認して条件分岐しているが、もっと良いやり方がないか？

referer_uri = URI.parse(request.referer)
if referer_uri.path == posts_path
  ...
end

2.redirect_toでindexにリダイレクトし、バリデーションエラー内容をflash[:error]で渡しているが、よい方法がないか？
render :indexだと、URLがposts/1になる。
その状態でリロードするとeditに遷移してしまうので、URLはpostsにしたい。

def index
  @posts = Post.all
  @posts.each { |post| post.if_eq_add_errors_to_self(flash[:error]['post']) } if flash[:error]&.key?('post')
end

def update
  @post = Post.find(params[:id])
  if @post.update(post_params)
    redirect_to posts_path, notice: 'Post was successfully updated.'
  else
    redirect_to posts_path, flash: { error: @post.flash_error_hash }
  end
end

該当のソースコード

posts_controller.rb

class PostsController < ApplicationController
  def index
    @posts = Post.all
    @posts.each { |post| post.if_eq_add_errors_to_self(flash[:error]['post']) } if flash[:error]&.key?('post')
  end

  def edit
    @post = Post.find(params[:id])
  end

  def update
    @post = Post.find(params[:id])
    referer_uri = URI.parse(request.referer)

    if @post.update(post_params)
      if referer_uri.path == posts_path
        redirect_to posts_path, notice: 'Post was successfully updated.'
      else
        redirect_to @post, notice: 'Post was successfully updated.'
      end
    elsif referer_uri.path == posts_path
      redirect_to posts_path, flash: { error: @post.flash_error_hash }
    else
      render :edit
    end
  end

  private

  def post_params
    params.require(:post).permit(:title)
  end
end

post.rb

class Post < ApplicationRecord
  validates :title, presence: true

  def if_eq_add_errors_to_self(flash_error_post)
    return unless id == flash_error_post['id']

    flash_error_post['details'].each do |col_key, details|
      details.each { |detail| errors.add(col_key, detail['error'].to_sym) }
    end
  end

  def flash_error_hash
    { post: { id: id, details: errors.details } }
  end
end

index.html.erb

<%= render partial: 'form', collection: @posts, as: 'post' %>

edit.html.erb

<%= render 'form', post: @post %>

_form.html.erb

<%= bootstrap_form_with(model: post, local: true, inline_errors: false) do |f| %>
  <%= f.text_field :title %>
  <%= f.errors_on :title %>
  <%= f.submit '更新する' %>
<% end %>

実際の画面イメージ

Comments

[no1knows]

> 良い実装なのか不安です。
良い実装の定義を質問に追記してください。

[necocoa]

実現したいことに書いてしまったのでわかりづらくなってしまったかもしれません。
聞きたいこととしては、疑問に書いてある2点です。

Answer 1

1.画面の遷移先をrefererで確認して条件分岐しているが、もっと良いやり方がないか？
「良い」の判断基準、物差しが何なのかを言っていただかないと比べられないです。
他に方法は無いか　なら沢山答えがあるとは思いますが。
link情報に indexなのかeditなのかも持たせるという方法もあります。

2.redirect_toでindexにリダイレクトし、バリデーションエラー内容をflash[:error]で渡しているが、よい方法がないか？
ここはちょっと、、ですね。flashをそのまま表示ではまずいのですか？

どうしても errorsで渡したいなら
@posts = Post.all
して、
@post = @posts.select{|post| post.id == params[:id].to_i}.first
して、その@postをなおし、saveして　render action: :index

select でなく find にすれば firstは要らないけど、find が RelationのfindかEnumrableのかどっちにとられるか分からないので日和った

Comments

[necocoa]

ありがとうございます！

`@post = @posts.select{|post| post.id == params[:id].to_i}.first`
`@post = @posts.find{|post| post.id == params[:id].to_i}`
は同意でした。
詳しく実装みてないですが、findにblockを渡すとselect{}.firstと同意になるようです。（rubocop上で同意だよっていう指摘があったので合ってると思います。）