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

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

新規登録して質問してみよう
ただいま回答率
85.35%
HTML5

HTML5 (Hyper Text Markup Language、バージョン 5)は、マークアップ言語であるHTMLの第5版です。

Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

Q&A

解決済

2回答

703閲覧

Ruby on rails paramsの使い方

mtbkr422

総合スコア1

HTML5

HTML5 (Hyper Text Markup Language、バージョン 5)は、マークアップ言語であるHTMLの第5版です。

Ruby on Rails

Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

0グッド

0クリップ

投稿2021/08/08 12:54

paramsの使い方

新規登録・ログイン機能を作る練習をしているのですが、コードの関係性がわからなくなってしまいました。

loginアクションの中で@userインスタンスを作成しているのでログインできなかった場合でも、login_form.html.erbファイルで@user.emailと@use.passwordを<input>タグの初期値として設定できるのではないかと思うのですが、できない理由がわかりません。
できない理由を教えていただきたいです。よろしくお願いいたします。

新規登録機能を作るときのコードを比較コードとして書きました。
新規登録機能は正常に動作しております。

ソースコード

Ruby

1 def login 2 @user = User.find_by(email: params[:email],password: params[:password]) 3 4 if @user 5 session[:user_id] = @user.id 6 flash[:notice] = "ログインしました" 7 redirect_to("/posts/index") 8 else 9 @message = "メールアドレスまたはパスワードが間違っています" 10 render("users/login_form") 11 end 12 end

↓ファイル名 login_form.html.erb

HTML

1 <%= form_tag("/login") do %> 2 <p>メールアドレス</p> 3 <input name="email" value="<%= @user.email %>"> 4 <p>パスワード</p> 5 <input type="password" name="password" value="<%= @user.password %>"> 6 <input type="submit" value="ログイン"> 7 <% end %>

エラーコード

NoMethodError in Users#login_form Showing /home/progate/tweet_app/app/views/users/login_form.html.erb where line #15 raised: undefined method `email' for nil:NilClass Extracted source (around line #15); 13 <%= form_tag("/login") do %> 14 <p>メールアドレス</p> 15 <input name="email" value="<%= @user.email %>"> 16 <p>パスワード</p> 17 <input type="password" name="password" value="<%= @user.password %>"> 18 <input type="submit" value="ログイン">

比較したいコード

Ruby

1 def create 2 @user = User.new( 3 name: params[:name], 4 email: params[:email], 5 password: params[:password], 6 image_name: "default_user.jpg" 7 ) 8 if @user.save 9 session[:user_id] = @user.id 10 flash[:notice] = "ユーザー登録が完了しました" 11 redirect_to("/users/#{@user.id}") 12 else 13 render("users/new") 14 end 15 end

↓ファイル名 new.html.erb

HTML

1 <%= form_tag("/users/create") do %> 2 <p>ユーザー名</p> 3 <input name="name" value="<%= @user.name %>"> 4 <p>メールアドレス</p> 5 <input name="email" value="<%= @user.email %>"> 6 <p>パスワード</p> 7 <input name="password" type="password" value="<%= @user.password %>"> 8 <input type="submit" value="新規登録"> 9 <% end %>

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

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答2

0

それはだめですね
@user = User.find_by(email: params[:email],password: params[:password])

ごめん間違えて送信してしまいました

tomoharuさんの通りであるのですが
このcodeもう一つ問題があります。
Userには password というDB項目がないので、これは必ずエラーになるでしょう

投稿2021/08/08 13:09

編集2021/08/08 13:12
winterboum

総合スコア23567

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

mtbkr422

2021/08/08 13:19

コメントありがとうございます。 エラーを見つけるためには色々なところをチェックしないといけないですね、、 アドバイスのおかげで解決することができました、ありがとうございました!
guest

0

ベストアンサー

User.find_by(email: params[:email],password: params[:password])

の返り値がnil すなわち @user がnilだからではないですかね?
ログインできたかできなかったかではなく、@userがnilかそうでないかで考える感じです。

投稿2021/08/08 13:07

tomoharu

総合スコア107

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

mtbkr422

2021/08/08 13:17

コメントありがとうございます! 皆様のアドバイスのおかげで納得することができました! 長時間悩んでいたのでとてもスッキリしました。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.35%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問