redirect_to でエラーメッセージを表示したいです

Question

バリデーションで空白の場合にエラーメッセージを出したいのですが、render、redirect_to、のどちらが良いでしょうか。

redirect_toの場合 Shutaコントローラ↓

def create
    book = Book.new(book_params)
    if book.save
      redirect_to shutax_path(book.id)
      flash[:notice] = "Book was successfully created."
    else 
      redirect_to index_path
    end
end

投稿画面index.html.erb（投稿画面と閲覧画面を同じindexページに載せています）で空白のまま投稿した場合は、またredirect_toでindex.html.erbを表示させるようにしています。しかしredirect_toの仕様上エラーメッセージが表示されないらしいので悩んでおります。

renderの場合　Shutaコントローラ↓

def create
    book = Book.new(book_params)
    if book.save
      redirect_to shutax_path(book.id)
      flash[:notice] = "Book was successfully created."
    else 
      render "index"
    end
end

renderでエラーメッセージを表示させようとすると、index.html.erbに書かれているeachメソッドがエラーを起こしてしまいます。

index.html.erb↓

<h1>Books</h1>

<table class="table">
  <thead>
    <tr>
      <th>Title</th>
      <th>Body</th>
      <th colspan="3"></th>
    </tr>
  </thead>
  <tbody>
    <% @book.each do |book| %>
     <tr>
       <td>
        <%= book.title %>
       </td>
       <td>
        <%= book.body %>
       </td>
       <td>
         <%= link_to "Show", shutax_path(book.id) %>
         <%= link_to "Edit", edit_shutax_path(book.id) %>
         <%= link_to "Destroy", destroy_shutax_path(book.id), method: :delete %>
       </td>
     </tr>
    <% end %>
  </tbody>
</table>

<h1>New book</h1>

<% if @booker.errors.any? %>
  <%= @booker.errors.count %>
  <% @booker.errors.full_messages.each do |message| %>
    <%= message %>
    <% end %>
<% end %>



<%= form_with model: @book, url:'/books', local:true do |f| %>
  <h4>Title</h4>
  <%= f.text_field :title %>
  <h4>Body</h4>
  <%= f.text_area :body %>
  <%= f.submit 'Create Book' %>
<% end %>

エラーメッセージ↓

redirect_toを使ってもエラーメッセージを表示することができる方法があるのか、また、NoMethodErrorを解決してrenderを使うのが良いのか、教えてください。よろしくお願いいたします。

Answer 1

redirect では errors.each を使ったエラーメッセージは使えません。
redirectする前にflachに積んでおくのでしょうか

Comments

[shuta-a]

なるほど。flashで表示させることも考えましたが、バリデーションでのエラーメッセージの表示はrenderで行おうと思います。

Answer 2

renderでエラーメッセージを表示させようとすると、index.html.erbに書かれているeachメソッドがエラーを起こしてしまいます。

きちんと必要な値を供給しましょう。

Comments

[maisumakun]

フォームに与えるのとリストを表示するのが同じ@bookでは成立しません。

リストは@booksとしたほうがいいでしょう。

[shuta-a]

はい。errorsメソッドへのインスタンス変数に値が格納されていないのが原因だと推測し、以下のようにelse〜end間にerrorsメソッドに対する@ booker変数をまた定義したのですが、

else
@book = Book.all
@booker = Book.all
render :index
end

これでもエラーが消えません。
必要な値の供給とはこのことではないのでしょうか。

[shuta-a]

フォームに与えるのとリストを表示するのが同じ@bookでは成立しません。

リストは@booksとしたほうがいいでしょう。

ありがとうございます。試します。

[shuta-a]

厚かましいのは承知でお聞きするのですが、具体的にどの部分を@booksに変えればよいのでしょうか。あれこれ変数の名前を変えたところ、errors.any?の部分でNoMethodErrorが起きてしまいました。

[maisumakun]

上に書いたとおり、「フォームに表示する値」と「ほかの案件のリスト」で値を分けてください。

[shuta-a]

なるほど、閲覧のリストの変数名と投稿フォームの変数名を分かるということですか？