🎄teratailクリスマスプレゼントキャンペーン2024🎄』開催中!

\teratail特別グッズやAmazonギフトカード最大2,000円分が当たる!/

詳細はこちら
Cloud9

Cloud9は、クラウドからのプログラミングが可能になるWebサービス。IDEとしての機能が搭載されており、GitHubやHerokuなど他ツールとの連携も可能です。ブラウザ上で動くため、デバイスに関係なく開発環境を準備できます。

Ruby

Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

Ruby on Rails

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

ログイン

ログインは、ユーザーがコンピューターシステムにアクセスするプロセスの事を呼びます。

Q&A

解決済

1回答

3777閲覧

ログイン、ログアウトでのheaderの切り替え

takumi0703

総合スコア5

Cloud9

Cloud9は、クラウドからのプログラミングが可能になるWebサービス。IDEとしての機能が搭載されており、GitHubやHerokuなど他ツールとの連携も可能です。ブラウザ上で動くため、デバイスに関係なく開発環境を準備できます。

Ruby

Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

Ruby on Rails

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

ログイン

ログインは、ユーザーがコンピューターシステムにアクセスするプロセスの事を呼びます。

0グッド

0クリップ

投稿2021/03/15 03:45

前提・実現したいこと

ログイン済とそうでない時のheaderのレイアウトを切り替えたいです。

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

if文でlogged_in(ログインユーザー情報)メソッドがtrueであればloginを非表示にしheaderを切り替えたいがログインしてもうまく切り替わらず、headerがずっとログインしていない状態になっています。

試したこと

・まずログイン後のレイアウトに問題がないかチェックするため、if文を消した所、ドロップダウンなど思った通りの表示はされました。

・試しに<% if logged_in? %>を<% if !logged_in? %>にしてみた所、ログイン状態の表示がうまくいき、loginリンクも非表示にできました。ですが、logged_inメソッドにはログインしてればtrueになるように、定義していたつもりだったので、なぜ逆の挙動になったのか理解できませんでした。

該当のソースコード

/sessions_helper.rb

Rails

1module SessionsHelper 2 def log_in(user) 3 session[:user_id] = user.id 4 end 5 def current_user 6 if session[:user_id] 7 @current_user ||= User.find_by(id: session[:user_id] 8 end 9 end 10 def logged_in? 11 !current_user.nil? 12 end 13 14end

/header.html.erb

<header class="navbar navbar-fixed-top navbar-inverse"> <div class="container"> <%= link_to "sample app", root_path, id: "logo" %> <nav> <ul class="nav navbar-nav navbar-right"> <li><%= link_to "Home", root_path %></li> <li><%= link_to "Help", help_path %></li> <% if logged_in? %> <li><%= link_to "Users", '#' %></li> <li class="dropdown"> <a href="#" class="dropdown-toggle" data-toggle="dropdown"> Account <b class="caret"></b> </a> <ul class="dropdown-menu"> <li><%= link_to "Profile", current_user %></li> <li><%= link_to "Settings", '#' %></li> <li class="divider"></li> <li> <%= link_to "Log out", logout_path, method: :delete %> </li> </ul> </li> <% else %> <li><%= link_to "Log in", login_path %></li> <% end %> </ul> </nav> </div> </header>

/sessions_controller.rb

class SessionsController < ApplicationController def new end def create user = User.find_by(email: params[:session][:email].downcase) if !user.nil? && user.authenticate(params[:session][:email]) log_in user redirect_to user else flash.now[:danger] = 'Invalid email/password combination' render 'new' end end def destroy end end

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

cloud9のrails sevrer
Rails 6.0.3

ご教授のほどよろしくお願いしたします。

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

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

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

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

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

guest

回答1

0

自己解決

/sessions_controller.rbに問題がありました。
× if !user.nil? && user.authenticate(params[:session][:email])

○ if !user.nil? && user.authenticate(params[:session][:password])

そもそもloginボタンからのログインができずにいたため、urlを指定してログイン状態を作っていました。この状態では本当のログインができていなかったため、if logged_in?はelseを返していたんだと気付きました。

原因はlogin自体に問題があり、paramsの引数にpasswordを渡していなかったため、loginがうまくいかずにいました。

単なるケアレスミスでした。
ご協力ありがとうございました。

投稿2021/03/15 05:52

takumi0703

総合スコア5

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

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

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.36%

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

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

質問する

関連した質問