Railsアプリで、ログイン状態により表示する項目を変更したい

Question

#解決したいこと
現在、Ruby on RailsでTodoアプリを作成しています。ログイン機能を実装しているのですが、ログインしている場合としていない場合で、ヘッダーの表示を以下のように変更するよう実装したいです。
・ログインしている時 =>トップページ、タスク一覧、ログアウト
・ログインしていない時 =>トップページ、新規登録、ログイン
しかし、ログインしても表示が変わらないので、どこを修正したら良いかを質問させていただきました。

#実装したこと
Sessionを利用して、ユーザーの判別を行いました。実装したコードとディレクトリの関係を以下に挙げます。
app/controllers/sessions_controller.rb
app/controllers/sessions_helper.rb
app/controllers/users_controller.rb
app/views/layouts/application.html.erb
config/routes.rb

sessions_controller.rb

class SessionsController < ApplicationController
  def new
  end

  def create
    #DBからユーザーを取り出す。
    user = User.find_by(email: params[:session][:email].downcase)
    if user && user.authenticate(params[:session][:password])
      #パスワードが登録されたユーザー情報と一致したら、セッションにユーザー情報を登録する。
      log_in user
      redirect_to '/'
    else
      flash[:alert] = 'Invalid email/password combination'
      render 'new'
    end

  end

  def destroy
    log_out
    flash[:notice] = "Logged out!"     #追加が成功した際に、flashメッセージを表示する。
    redirect_to '/'
  end
end

application.html.erb ヘッダーの部分のみ掲載します。

  <!--ヘッダー-->
  <header>
    <nav class="navbar navbar-expand-lg navbar-light bg-light">
      <div class="container-fluid">
        <a class="navbar-brand"><%= link_to("Todoアプリ","/")%></a>
        <div class="collapse navbar-collapse justify-content-end" id="navbarNavAltMarkup">
          <div class="navbar-nav">
            <% if logged_in? %>
            <a class="nav-link"><%= link_to "タスク一覧", tasks_path %></a>
            <a class="nav-link"><%= link_to "ログアウト", logout_path %></a>
            <% else %>
            <a class="nav-link"><%= link_to "新規登録", new_user_path %></a>
            <a class="nav-link"><%= link_to "ログイン", login_path %></a>
            <% end %>
          </div>
        </div>
      </div>
    </nav>

  </header>

routes.rb

Rails.application.routes.draw do
  get 'sessions/new'
  get '/' => "home#top" #トップページのルーティング home:コントローラー名 top:アクション名
  get 'login' => 'sessions#new'         #新しいセッションのページ(ログイン)
  post 'login' => 'sessions#create'     #新しいセッションの作成
  delete 'logout' => 'sessions#delete'  #セッションの削除(ログアウト)
  #resourcesメソッドを用いることで、HTTPメソッドとURLとアクションを自動的に紐付ける。
  #Tasksコントローラーのルーティングが自動生成される。
  resources :tasks
  resources :users
end

sessions_helper.rb

module SessionsHelper
  #渡されたユーザーでログインする。
  #攻撃者がこの情報をcookiesから盗むことができたとしても、本物のユーザーとしてログインすることはできない。
  def log_in(user)
    #ユーザーのブラウザ内の一時cookiesに暗号済みのユーザーIDが自動で生成される。
    #session[:名前]により、名前をつけてセッションに登録できる。
    session[:user_id] = user.id
  end

  #セッションに含まれる現在のユーザーを検索する。
  def current_user
    #@current_user = @current_user || User.find_by(id:params[:user_id])
    if session[:user_id]
      @current_user ||= User.find_by(id:params[:user_id])
    end
  end

  #ユーザーがログインしていればtrue,そうでなければfalseを返す。
  def logged_in?
    #current_user.nil?がfalseの場合(すなわち・・・)にtrueとなる。
    !current_user.nil?
  end

  #ログイン中のユーザーをログアウトする.
  def log_out
    sessions.delete(:user_id)
    @current_user = nil
  end
end

users_controller.rb newとcreateアクションを掲載します。

  def new
    @user = User.new
  end

  def create
    @user = User.new(user_params)
    #ユーザーの保存が成功した時
    if @user.save
      #登録の終わったユーザーがデフォルトでログインするように変更する。
      log_in @user
      flash[:notice] = "Successfully registered a new user!"     #追加が成功した際に、flashメッセージを表示する。
      redirect_to '/'
    else
      #flash[:alert] = "Please enter at least one character."     #追加が成功した際に、flashメッセージを表示する。
      render 'users/new'
    end
  end

どうぞよろしくお願いします。

#追記
実際の画像を添付します。
まず、rails consoleでダミーデータを作成しておきます。

irb(main):004:0> User.create(name: "sampleitirou", email: "[e-mail]", password: "hogehoge", password_confirmation: "hogehoge")
  TRANSACTION (0.1ms)  begin transaction
  User Exists? (0.2ms)  SELECT 1 AS one FROM "users" WHERE "users"."email" = ? LIMIT ?  [["email", "[e-mail]"], ["LIMIT", 1]]
  User Create (1.1ms)  INSERT INTO "users" ("name", "email", "created_at", "updated_at", "password_digest") VALUES (?, ?, ?, ?, ?)  [["name", "sampleitirou"], ["email", "[e-mail]"], ["created_at", "2021-08-22 13:00:23.183764"], ["updated_at", "2021-08-22 13:00:23.183764"], ["password_digest", "$2a$12$sPiY0t7zYoRvdzDdLk.4L.iDlvL3srRgFTIVW3ymtQFoTc1TWZ4Qe"]]
  TRANSACTION (1.4ms)  commit transaction
=> #<User id: 11, name: "sampleitirou", email: "[e-mail]", created_at: "2021-08-22 13:00:23.183764000 +0000", updated_at: "2021-08-22 13:00:23.183764000 +0000", password_digest: [FILTERED]>
irb(main):005:0> 

そして、ログイン画面で登録した情報を入力します。パスワードは同じパスワードを打っています。

ログインボタンを押した後の画面は以下のようになりました。もしログインに失敗していたら、フラッシュメッセージを出してnewページのままなので、ログインは出来ていると思います。

Comments

[winterboum]

viewもその他もおかしいところは見当たりません。
本当に login成功してますか？

[suzuryu.07]

追記欄に画像を挿入しました。ログインに失敗していたら、render 'new'でログインページのままになるので、恐らくログインは成功していると思います。

[winterboum]

できてますね。。。
わからん、これでなぜ駄目なのだろう

[suzuryu.07]

ログイン自体はできているので、sessions_controller.rbは問題ないと思います。sessions_helper.rbのlogged_inメソッドかcurrent_userメソッドに問題あると思いますが、見当たらない、、、

Answer 1

session_helper内のcurrent_userメソッドの部分が間違っていました。id:paramsではなく、id:sessionでした。session[:user_id]に保存したので、そこからfind_byすることで現在のユーザーを探索することができることを考えると、凡ミスしていました。
(誤)
@current_user ||= User.find_by(id:params[:user_id])
(正)
@current_user ||= User.find_by(id:session[:user_id])