バリデーションによるエラーメッセージが表示されない

Question

前提

プログラミング初心者な為専門用語等わからないことが多くあります
調べながらの対応になる為返答が遅くなる場合があります。
誤字、脱字等もあるかと思いますがご指摘ください。

実現したいこと

本の感想を投稿するサイトを作成し、バリデーションを実装。
バリデーションはcreate、updateが失敗したときにエラーメッセージも表示させる

発生している問題・エラーメッセージ

エラーメッセージは表示されず、それぞれのリダイレクト先にリダイレクトされる

該当のソースコード

<h1>books</h1>
  <table>
    <thead>
      <tr>
        <th>title</th>
        <th>body</th>
        <th></th>
      </tr>
    </thead>
    <tbody>
        <% @books.each do |book| %>
        <tr>
          <td><%= book.title %></td>
          <td><%= book.body %></td>
          <td><%= link_to "show",book_path(book.id) %></td>
          <td><%= link_to "edit",edit_book_path(book.id) %></td>
          <td><%= link_to "destroy",book_path(book.id), method: :delete,"data-confirm" => "Are you sure?" %></td>
        </tr>
        <% end %>
    </tbody>
  </table>

  <h3>New Books</h3>
  <% if @book.errors.any? %>
    <%= @book.errors.count %>errors prohibited this book from being saved
    <ul>
      <% @book.errors.full_messages.each do |message| %>
        <li><%= message %></li>
    <% end %>
    </ul>
  <% end %>

<%= form_with model: @book do |f| %>

  <h4>title</h4>
  <%= f.text_field :title %>

  <h4>body</h4>
  <%= f.text_area :body %>

  <%= f.submit'Create Book' %>
<% end %>

<p id="notice"><%= notice %></p>
  <h1>Editing Book</h1>
  <% if @book.errors.any? %>
    <%= @book.errors.count %>error prohibited this book from being saved
    <ul>
      <% @book.errors.full_messages.each do |message| %>
        <li><%= message %></li>
      <% end %>
    </ul>
  <% end %>

<%= form_with model: @book, url: update_book_path(@book.id), method: :patch do |f| %>

  <h4>Title</h4>
  <%= f.text_field :title %>

  <h4>Body</h4>
  <%= f.text_area :body %>
  <h4></h4>
  <%= f.submit'Update Book' %>
<% end %>

  <span>
    <%= link_to "show",book_path(@book.id) %>
  </span>
  <span>|</span>
  <span>
    <%= link_to "back","/books" %>
  </span>

class BooksController < ApplicationController
  def new
  end

  def index
    @book = Book.new
    @books = Book.all
  end
  def create
    book = Book.new(book_params)
  if  book.save
    flash[:notice] = "Book was successfully created."
    redirect_to book_path(book.id)#, notice: "You have created book successfully."
  else
    @book = Book.new
    @books = Book.all
    render :index
  end
  end
  def show
    @book = Book.find(params[:id])
  end

  def edit
    @book = Book.find(params[:id])
  end
  def update
    book =Book.find(params[:id])
  if book.update(book_params)
    flash[:notice] = "Book was successfully updated."
    redirect_to book_path(book.id)
  else
    @book = Book.find(params[:id])
    render :edit
  end
  end

  def destroy
    book = Book.find(params[:id])
    book.destroy
    redirect_to '/books'
  end

  private
  def book_params
    params.require(:book).permit(:title, :body)
  end
end

試したこと

特にないです。
これを実施してみたら？等あればご教授いただけると幸いです。

補足情報（FW/ツールのバージョンなど）

ruby 3.1.2p20
Rails 6.1.7
cloud9

railsに関して知識不足を痛感しているので自分自身の勉強の為にも
なにが悪くてこういう理由でこう処置する等ご教授いただけると幸いです。

Answer 1

これを実施してみたら？等あればご教授いただけると幸いです。

Railsの書き方の基本はScaffoldです。
なので、個人的におすすめなのは、このScaffoldをベースにカスタマイズしていくという流れをおすすめします。

1. 新しくプロジェクトを作る
2. rails g scaffold book title:string body:textを実行する
3. rails db:migrateを実行する

上記で、「バリデーションによるエラーメッセージが表示されて、CRUDのできるBookページ」が出来上がります。
それをベースにして既存のコードの振る舞い（例えばcreateのあとindexに遷移するなど）へ修正していくのが良いかと思います。

---

初学者の方がCRUDなどのRailsの作法の理解が十分でないなかで、１からコードを書いて、それを理解するのは難しいと思います。
そのうちScaffoldって何をしているんだろう・・・とか、そもそも処理の流れってどうなっているんだろうってことが気になってきたら少しずつ下記を読み込んでいただくと理解も深まりやすいかと思います。

• railsアプリケーションはどのように動いているのか？
• Railsのscaffoldで生成されたコードを1行1行説明する

Comments

[nimu]

コメントありがとうございます。
前提に記述していなかったのですがScaffoldを使用しないで開発するのが条件となっていました。
Scaffoldを使用する際に利用させて頂きます。
繰り返しになりますがものすごく丁寧なコメントありがとうございました！
また困って質問した際に助けてください。笑

[no1knows]

以下、おっさんのたわごとです。

ちょっと考えてみてください。
お手本があって、お手本の存在をしっているのに、お手本を見ずに、人に聞いて、教えてもらう。
そんな人、どう思いますか？わからないことがあったら、毎回、聞いてくる人と一緒に仕事したいですか？

> Scaffoldを使用する際に利用させて頂きます。
DHHさんがScaffoldを使わずに書いたコードを、コミュニティのみんなが、よりわかりやすく、より共通化して、より見やすくした結果、出来上がったのが、Scaffoldなのです。
だからわからないことがあれば、もしくは自作コードがうまく動かなければ、まずはScaffoldを見て、自分のコードと比較して理解して書き方を学んでいくのが良いと思います。

さて、あなたは何をすると一番早く、Ruby on Railsの設定より規約を遵守したコードがかけるようになるのでしょうか？

[nimu]

ご指摘ありがとうございます！

説明不足だらけですいません！
課題に取り組んでいまして、その条件で
Scaffoldを使用せずに開発するというのが
ありました！
Scaffoldとは何か現段階で理解していません。
なにもかもわからないままで次の事に
進んでいいのかわからずにScaffoldについて
まだ調べられていないのが本音です！
なのでScaffoldを理解したらもっとRuby on Railsについて理解が深まるのかもしれませんが
そこは後から遠回りしたなぁと感じる部分かもしれません！

なんでもかんでも聞くのは正しくはないと思います！自分でもわからないところがわからない状態なので正しい処置を教えてもらってもなんでそうなるのか、自分の今の知識と照らし合わせみたいなことが出来ればいいなと考えてます！

コメントにいただいたリンクは目を通して
ちょっとでも理解ができる様努力します！
コメントありがとうございました！

[no1knows]

たわごとに丁寧なコメントありがとうございます。
なんのための課題なんでしょうね。
まず難しい課題を出して、ほら、Scaffoldをやるとすごいだろうって言いたいのかな。。。
そんなことしても時間ばっかりかかって、得るものが少ないのに。。。

まぁ、気が向いて10分ぐらいあったら、一度、1〜3までをやって、Scaffoldを試してみてください。
Scaffoldとは何かとか調べる必要もなく、Terminalで打てば、Scaffoldとは何かがわかると思います。

そしたらcreate時のバリデーションエラーの解決の糸口も見えると思いますよ！

Answer 2

update で @book を定義していなかったり、再代入してたりするじゃないでしょうか？
@book をいちど定義したらそれを使いまわしてはいかがでしょう

def update
  @book = Book.find(params[:id])

  if @book.update(book_params)
    flash[:notice] = "Book was successfully updated."
    redirect_to book_path(book.id)
  else
    render :edit
  end
end

Comments

[nimu]

update時のバリデーション表示されるようになりました！！
ありがとうございます！！

さらに質問になって申し訳ないのですが
create時にもバリデーションを表示させたい時
```controller
def create
book = Book.new(book_params)
if book.save
flash[:notice] = "Book was successfully created."
redirect_to book_path(book.id)#, notice: "You have created book successfully."
else
@book = Book.new
@books = Book.all
render :index
end
end
```
updateの時同様に@bookを再代入せずにrails sすると
NoMethodError in Books#create
Showing /home/ec2-user/environment/bookers/app/views/books/index.html.erb where line #11 raised:エラーになるのですがこれの対処についても
ご教授いただけますか？
お手数かけてすみません。

[yuma.inaura]

どの行のエラーでしょうか？

[yuma.inaura]

updateと構造が異なる箇所を比べてみましょう
しばらく頑張って無理なら別の質問を作ってみてはいかがでしょうか