rails @current_userがnilになってしまう

Question

プログラミング初学者です。
現在railsの学習がてらwebアプリの開発を行なっているのですが、posts_controllerのnewアクションをした後にcreateアクションをすると、@current_userがnilになってしまいます。
@current_userはapplication controllerでset_current_userとして定義していて、before_actionで全てのアクションに対応するようにしています。
ご教授お願いします。
エラーメッセージは

undefined method `id' for nil:NilClass

と出ています。

posts_controller

class PostsController < ApplicationController
  before_action :authenticate_user
  before_action :ensure_correct_user, {only: [:edit, :update, :destroy]}

  def index
    @posts = Post.all.order(created_at: :desc)
  end

  def new
    @post = Post.new
  end

  def create
    @post = Post.new(
      user_id: @current_user.id,
      post_image_name: Fire.binwrite("public/post_images/#{@post.id}.jpg", params[:post_image].read),
      title: params[:title],
      content: params[:content]
    )
    if @post.save
      flash[:notice] = "レシピが投稿されました"
      redirect_to("/posts/index")
    else
      render("posts/new")
    end
  end

  def show
    @post = Post.find_by(id: params[:id])
  end

  def edit
    @post = Post.find_by(id: params[:id])
  end

  def update
    @post = Post.find_by(id: params[:id])
    @post.content = params[:content]
    if params[:post_image]
      @post.post_image_name = "#{@post.id}.jpg"
      post_image = params[:post_image]
      Fire.binwrite("public/post_images/#{@post.post_image_name}", post_image.read)
    end

    if @post.save
      flash[:notice] = "レシピを変更しました"
      redirect_to("/posts/index")
    else
      render("posts/edit")
    end
  end

  def destroy
    @post = Post.find_by(id: params[:id])
    @post.destroy
    flash[:notice] = "レシピを削除しました"
    redirect_to("/posts/index")
  end

  def ensure_correct_user
    @post = Post.find_by(id: params[:id])
    if @current_user.id != @post.user_id
      flash[:notice] = "権限がありません"
      redirect_to("/posts/index")
    end
  end
end

posts/new.html.erb

<div class="main posts-new">
  <div class="container">
    <h1 class="form-heading">
        投稿する
    </h1>
    <%= form_tag("/posts/create") do %>
      <div class="form">
        <div class="form-body">
          <% @post.errors.full_messages.each do |message| %>
            <div class="form-error">
              <%= message %>
            </div>
          <% end %>
          <div class="post_image">
            料理の写真
            <input type="file" name="post_image">
          </div>
          <div class="post-title">
            タイトル
            <input name="title" value="<%= @post.title %>">
          </div>
          <div class="post-content">
            レシピ
            <textarea name="content"><%= @post.content %></textarea>
          </div>
          <input type="submit" value="投稿">
        </div>
      </div>
    <% end %>
  </div>
</div>

application_controller

class ApplicationController < ActionController::Base
  before_action :set_current_user
    
  def set_current_user
    @current_user = User.find_by(id: session[:user_id])
  end
    
  def authenticate_user
    if @current_user = nil
      flash[:notice] = "ログインが必要です"
      redirect_to("/login")
    end
  end
    
  def forbid_login_user
    if @current_user
      flash[:notice] = "すでにログインしています"
      redirect_to("/posts/index")
    end
  end
  
end
  

Answer 1

エラーメッセージは，それがどこで出たかが重要なので，そこも示されるとよいですね。

set_current_user メソッドの定義を見ると

  def set_current_user
    @current_user = User.find_by(id: session[:user_id])
  end

とありますが，当該の id が無ければ右辺の値は nil になりますね。
session[:user_id] の値を表示させてみて，そういうユーザーがいるかどうかを確認すると早いかもしれません。

なお，authenticate_user の定義で

    if @current_user = nil

とありますが，== でなく = になっている（つまり代入式になっている）のはバグでしょうね。
「代入式の値は代入された値」というルールがあるので，この if の条件式の値は常に nil です。つまり，if の中身が実行されることはありません。

Comments

[winterboum]

if @current_user = nil　ここで値が入っていた @current_user　に nilを入れてしまってるんですね。

[dfasdf]

返信が遅くなり申し訳ありません。
回答ありがとうございます。
if @current_user == nilにしたところエラーが解除されました。ありがとうございます。