before_actionを実装した後、削除機能ができなくなった

Question

前提・実現したいこと

オリジナルアプリ（ブログ的な）を作成しています。
before_action move_to_indexで「URLに直接入力（編集など）して、ログインしていないユーザーが他のユーザーの記事を編集しようとするとトップページに遷移する」機能を実装した後、アプリの挙動を確認していたところ、記事の削除をしようとすると、エラーになってしまい、削除ができなくなってしまいました。

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

NoMethodError in ArticlesController#destroy

undefined method `user' for nil:NilClass Did you mean? super

def move_to_index←ここがおかしいと言われている
    unless current_user.id == @article.user.id
      redirect_to action: :index
    end
  end

articles_controller

class ArticlesController < ApplicationController
  before_action :authenticate_user!, except: [:index, :show]
  before_action :set_article, only: [:edit, :show]
  before_action :move_to_index, except: [:index, :new, :create, :show,]

  def index
    @articles = Article.order("created_at DESC")
  end

  def new
    @article = Article.new
  end

  def create
    @article = Article.new(article_params)
    if @article.save
      redirect_to root_path
    else
      render :new
    end
  end

  def destroy
    article = Article.find(params[:id])
    article.destroy

  end

  def edit
  end

  def update
    article = Article.find(params[:id])
    article.update(article_params)
    if article.valid?
      redirect_to article_path(article.id)
    else
      render :edit
    end
  end

  def show
    @comment = Comment.new
    @comments = @article.comments.includes(:user)
  end

  private

  def article_params
    params.require(:article).permit(:image, :title, :profile, :text, :category_id, ).merge(user_id: current_user.id)
  end

  def set_article
    begin
      @article = Article.find(params[:id])
    rescue
      render :destroy
    end
  end

  def move_to_index
    unless current_user.id == @article.user.id
      redirect_to action: :index
    end
  end
end

destroy.rb

<div class="article-delete">
      <%= link_to 'B-blog', root_path, class:"delete-header-text" %>
  <div class="delete-main">
    <div class="success">
      <div class="delete-text">
        削除完了しました
      </div>
        <%= link_to 'トップページに戻る', root_path, class: "return-btn" %>
    </div>
  </div>
</div>

試したこと

エラーの内容を調べたところ、「ユーザーが空だからわからない」と言われています。move_to_indexにuserは記述しているのですが、それでも「空です」と言われているので、コントローラーで正しくuserの定義ができていない？のかと思います。

どなたかエラーの解決を教えて頂けると幸いです。

Answer 1

エラーの内容を調べたところ、「ユーザーが空だからわからない」と言われています。

いえ、そのエラーメッセージの意味は
「存在しないメソッドを呼び出そうとしました。Nil クラスに user というメソッドは存在しません。」
です。

つまり、move_to_index メソッドの

@article.user.id

の部分を

(nil).user.id

と解釈しています。

@article が nil になっているのが原因なので、 destroy のときも set_article が呼び出されるようにすれば解決すると思います。

- before_action :set_article, only: [:edit, :show]
+ before_action :set_article, only: [:edit, :show, :destroy]

Comments

[YugoTokonami]

回答ありがとうございます！
回答を参考にした結果、解決ができました！
・set_articleにdestroy追加→article.rbのcommentsのアソシエーションに、dependent: :destroyを追加で正しく削除ができるようになりました！
ありがとうございます。