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

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

ただいまの
回答率

88.58%

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

解決済

回答 1

投稿 編集

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

yume0409

score 27

twitterのような簡易ツイートアプリを作成しています。
やりたい内容は、ツイートが3文字未満、200文字より大きければ、送信時にエラーメッセージを出すようにしたいです。

バリデーションは定義しています。
tweet.rb

class Tweet < ApplicationRecord
  validates :text,    length: { in: 3..200 } 
  belongs_to :user
  has_many :comments
end

こちらはツイート送信のビューです。
3行目にrenderを記述しました。
new.html.haml

.new-field
  = form_tag('/tweets', method: :post) do
    = render 'layouts/error'
    %h3.new-tweets
      ツイートする
    %textarea{placeholder: "ツイート文を入力してください", name: "text", cols: "50", rows: "20", class: "new-text"}
    %input{type: "submit", class: "new-submit"}/  
    %i.fas.fa-camera

renderから引っ張り出しているのはこのファイルです。
今は仮置きしていますが、ツイートが3文字未満、200文字より大きければ、2行目を表示させたいです。
_error.html.haml

- if @tweet.errors.any?
  .aaa エラー

バリデーションで新たに記述したのは4行目から8行目です。
tweets_controller.rb

  def create
    @tweet = Tweet.create(text: tweet_params[:text], user_id: current_user.id)
    redirect_to tweets_path
    if @tweet.save
      redirect_to :root
    else
      render 'new'
    end
  end

上記の4つのファイルを編集すると必ずこのエラーが発生します。

NoMethodError in Tweets#new
undefined method `errors' for nil:NilClass

- if @tweet.errors.any?
  .aaa エラー

バリデーションに関することを調べましたが、どの記事でも同じエラーが出ていて困っています。。。
よろしくお願いします。

補足

tweets_controller.rb

class TweetsController < ApplicationController

  def index
    # @tweet = Tweet.find(params[:id])
    @tweets = Tweet.includes(:user).order("created_at DESC")
  end

  def show
    # @tweet = Tweet.find(params[:id])
  end

  def new
  end

  def create
    @tweet = Tweet.create(text: tweet_params[:text], user_id: current_user.id)
    redirect_to tweets_path
     if @tweet.save
       redirect_to :root
     else
       render 'new'
     end
  end

private
  def tweet_params
    params.permit(:text)
  end

end


newは何も定義していないです。
よろしくお願いします。

補足2

遅くなり、すみません。
回答ありがとうございます!

  def new
    @tweet = Tweet.new
  end


こう付け足してみましたが、先ほどと違うエラーが出てしまいました。
エラーはrender 'new'に赤マークが出ていました。

AbstractController::DoubleRenderError in TweetsController#create
Render and/or redirect were called multiple times in this action. Please note that you may only call render OR redirect, and at most once per action. Also note that neither redirect nor render terminate execution of the action, so if you want to exit an action after redirecting, you need to do something like "redirect_to(...) and return".


調べると

  def create
    @tweet = Tweet.create(text: tweet_params[:text], user_id: current_user.id)
    redirect_to tweets_path and return
    if @tweet.save
      redirect_to :root
    else
      render 'new'
    end
  end


redirect_to tweets_pathの後ろにand returnを付け足すとエラーは消えました。

今は バリデーションの設定はしてありますので、エラーメッセージは出ないだけでツイート送信したらツイート画面に遷移されますが、ツイートは送信されません。
エラーメッセージが出てエラーだったらそのページから動かない設定をしたいです。。。
よろしくお願いします。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

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

  • winterboum

    2020/01/24 21:23

    controllerを全体を見せてください。少なくとも newを

    キャンセル

  • bamboo-nova

    2020/01/24 22:32

    winterboumさんと同じ指摘になりますが、おそらくコントローラのnewが原因だと思います。NilClass自体が@tweetに値が入ってないことから生じるエラーなので...。

    スペルミスや@tweetが定義されているか確認してみてください。

    キャンセル

回答 1

checkベストアンサー

0

def new  で
@tweet = Tweet.new
してください

追記
createに

  redirect_to tweets_path
     if @tweet.save
       redirect_to :root
     else
       render 'new'
     end


とあるので、ifがどちらに転んでも redirect か render かが2回定義されてしまいます。
これが原因です。
どのパスを通っても、render, redirect が一つになるようにロジックしてください

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2020/01/27 15:55

    回答ありがとうございます!
    render 'new'
    を消したらand returnなくてもエラーは消えます。
    ただエラーメッセージは表示されないのですが、それはまた別の記述がおかしいのでしょうか?

    キャンセル

  • 2020/01/27 16:04

    そっちを消したんですか。redirect_to tweets_path って何のため?どういう時に?
    saveが成功するとまたさっきのエラーになりますよ。

    で、失敗してるので本来 new に行ってそこで = render 'layouts/error' でエラー表示される所、それを削除しちゃったので redirect_to tweets_path に行ってしまった

    キャンセル

  • 2020/01/27 16:11

    ありがとうございます!
    redirect_to tweets_pathを消したらメッセージ表示されました。
    redirect_to tweets_pathはメッセージ送信されたらtweets_pathへ遷移するつもりで記述してましたが、
    redirect_to :rootが代わりの役割を果たしているのでtweets_pathは消しても良かったんですね。
    今理解できました!
    ありがとうございます!

    キャンセル

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

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

関連した質問

同じタグがついた質問を見る