Rails のredirect_to でエラーメッセージを出したいのですが

redirect_to :back ,notice: '入力されていない項目があります'

コントローラーで前ページに戻るように設定しているのですが、戻ったページでエラーメッセージを出したいです。
def create
@content = Content.new(create_params)
if @content.valid?

 @content.save
 @@number = @@number + 1

 if @@number == 19
 @contents = Content.where(game_id:params[:game_id])
 @@number = 1
 @par = 0
 @putts = 0
 @score = 0
 else
 redirect_to :back
 end

else
redirect_to :back ,notice: '入力されていない項目があります'

end

end
ちなみにこれがメソッドの記述です。

hana-da

2016/10/14 09:53

redirect_to :back での遷移先となる view テンプレート(erbなど) を提示ください

行動規範の内容に同意します

回答2件

ベストアンサー

という事で flash メッセージの設定方法は間違えていませんので、表示されないのは遷移先のviewテンプレートに flash を表示する部分がないからだと思います。

msyk_tymさんも書いてくださっていますが flash メッセージを表示したいところに

erb
1 <%= flash[:notice] %>

とかけば大丈夫です。

msyk_tymさんも書かれていますが変数の扱いがrubyっぽくないです。
もし、まだ Ruby on Rails チュートリアル実例を使ってRailsを学ぼうを試した事がなければ最初から順に試す事をオススメいたします。

投稿2016/10/14 15:47

hana-da

総合スコア1728

自分がよくやる手法としては、flashに入れますね。flashなら任意のパラメータを設定できるので、
エラーの切り分けも簡単にできます。
使い方は簡単で、

 redirect_to index_path, flash: {error: 'メッセージ'}

とやればいけます。
ご提示されているコードを改修すると以下になります。

def create 
   @content = Content.new(create_params) 
   if @content.valid?

      @content.save 
      @@number = @@number + 1

      if @@number == 19 
         @contents = Content.where(game_id:params[:game_id]) 
         @@number = 1 
         @par = 0 
         @putts = 0 
         @score = 0 
      else 
         redirect_to :back 
      end

   else 
      redirect_to :back ,flash: {notice: "入力されていない項目があります"}
   end
end

これで、リダイレクト先で、flash[:notice]で値が取れます。
あとは、View側で

<% if flash[:notice] %>
   <%= flash[:notice] %>
<% end %>

ってやればエラーメッセージ出せます。

余談ですがクラス変数がめっちゃ使われているのが気になります。
クラス変数だと、もし違うアクションで同じ名前の変数を定義しちゃった時にデグレが発生する危険があるので、
バッドプラクティスだと思います。
インスタンス変数あるいはローカル変数を使いましょう。変数のスコープは狭いほど良いです。

投稿2016/10/14 15:21

msyk_tym

総合スコア39

hana-da

2016/10/14 15:32

redirect_to :back ,flash: {notice: "入力されていない項目があります"} これ redirect_to :back ,notice: "入力されていない項目があります" と書いても同じなんです。標準では :alert と :notice だけなので error: 'エラーだよ' とかはダメですが、add_flash_typesすると :error も使えるようにできたりします。