質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

89.64%

Railsのform_withでパリデーションエラーが二重に表示される

受付中

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 320

stks56

score 9

errors.full_messages.eachを回してformにバリデーションエラーを表示しているのですが、二重に表示されてしまい困っています。

![イメージ説明]

new.html.erb

<div class="main-wrapper">
  <div class="post-new-container">
    <h2>投稿する</h2>
    <%= render "form", post: @post %>
  </div>
</div>


_form.html.erb

<div class="form">
  <% post.errors.full_messages.each do |message| %>
    <div class="error">
      <%= message %>
    </div>
  <% end %>
  <%= form_with(model: post, local: true) do |f| %>

    <div class="field">
      <%= f.label :title %><br />
      <%= f.text_field  :title, class: "title"%>
    </div>

    <div class="field">
      <%= f.label :game_id %>
      <%= f.collection_select :game_id, Game.all, :id, :name, prompt: true %>
    </div>

    <div class="field">
      <%= f.label :content %>
      <%= f.text_area :content %>
    </div>

    <div class="actions">
      <%= f.submit "投稿", class: "submit-btn" %>
    </div>
  <% end %>
</div>

ルーティング

resources :posts

posts_controller.rb

class PostsController < ApplicationController
  before_action :authenticate_user!, {only: [:new, :create, :edit, :update, :destroy]}
  before_action :ensure_correct_user, {only: [:edit, :update, :destroy]}

  def index
    @posts = Post.page(params[:page]).per(10).order("created_at desc")
  end

  def show
    @post = Post.find_by(id: params[:id])
  end

  def new
    @post = Post.new
  end

  def create
    @post = Post.new(post_params)
    @post.user_id = current_user.id
    if @post.save
      flash[:notice] = "記事を投稿しました"
      redirect_to("/posts")
    else
      render("posts/new")
    end
  end

  def edit
  end

  def update
    if @post.update(post_params)
      flash[:notice] = "記事を編集しました"
      redirect_to("/posts/#{@post.id}")
    else
      render("posts/edit")
    end
  end

  def destroy
    @post = Post.find_by(id: params[:id])
    @post.destroy
    flash[:notice] = "記事を削除しました"
    redirect_to("/posts")
  end

  def ensure_correct_user
    @post = Post.find_by(id: params[:id])
    if current_user.id != @post.user_id
      flash[:notice] = "権限がありません"
      redirect_to("/")
    end
  end

  private
  def post_params
    params.require(:post).permit(:title, :game_id, :content)
  end
end

postsのスキーマ

create_table "posts", options: "ENGINE=InnoDB DEFAULT CHARSET=utf8", force: :cascade do |t|
    t.text "title"
    t.text "content"
    t.integer "game_id"
    t.integer "user_id"
    t.datetime "created_at", null: false
    t.datetime "updated_at", null: false
  end

考えられること
恐らくcollection_selectで他モデルのDBのデータを参照して表示している点がおかしいと思うのですが二重になる意味がちょっとわかりません。
form自体は正常に投稿できます。

アドバイスいただけると幸いです。
よろしくお願いします。

追記
Postモデルのバリデーション内容です

class Post < ApplicationRecord
  validates :title, {presence: true}
  validates :content, {presence: true}
  validates :game_id, {presence: true}
  validates :user_id, {presence: true}

  belongs_to :user
  belongs_to :game

  has_many :likes
  accepts_nested_attributes_for :likes
  has_many :users, through: :likes
end

追記

  def create
    console
    @post = Post.new(post_params)
    @post.user_id = current_user.id
    if @post.save
      flash[:notice] = "記事を投稿しました"
      redirect_to("/posts")
    else
      raise
      render("posts/new")
    end
  end


とした時のエラー内容です
イメージ説明

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • haneru

    2019/05/02 22:09

    Postモデルのバリデーションの実装を見せてください

    キャンセル

  • stks56

    2019/05/03 04:49

    Postモデルのバリデーション内容を追記しました!よろしくお願いします。

    キャンセル

  • haneru

    2019/05/03 20:56

    バリデーションの実装自体は問題なさそうですね。
    それでは次はcreateアクションの`else`と`render("posts/new")`の間に`raise`を記述し、エラー画面上のコンソールで`@post.errors`をした際の出力結果を見せていただいてもよろしいでしょうか?

    キャンセル

  • stks56

    2019/05/03 22:39

    web-consoleを知らなかったので調べました。
    メソッドの下にconsoleと入れるとエラー画面が出るらしいのですが出ませんでした。なぜでしょう?
    gemfileにweb-consoleの記述はあります。

    キャンセル

回答 1

0

(かなり古い質問のため、もう解決しておられるかもしれませんが、同じ悩みでこの質問にたどり着いた方もいらっしゃるかもしれませんので、一応回答させていただきます)

原因

Rails 5 から、

belongs_to :game

と書くと自動的に game に対する必須バリデーションを設定するようになっています。

今回は以下のように自前のバリデーションもつけちゃってるので、

validates :game_id, {presence: true}

必須のエラーメッセージが二重に出てしまっているものと思われます。

対策

モデルから以下の2行を削除してください。

  validates :game_id, {presence: true}
  validates :user_id, {presence: true}

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 89.64%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる