Ruby on Rails中destroy函数无法删除条目，报nil:NilClass无user方法错误

嘿，这个问题我太熟了！咱们一步步来拆解解决～

你遇到的 undefined method 'user' for nil:NilClass 错误，核心原因超级明确：@article 变量是 nil——也就是说，当你在 require_same_user 里调用 @article.user 时，这个@article根本没被赋值，是空的，Ruby自然就报错啦。

1. 先检查你的destroy方法 #

首先确认，你在执行销毁操作前，有没有先找到对应的文章实例？正常的destroy方法应该是这样的：

def destroy
  # 关键！先通过ID把文章找出来赋值给@article
  @article = Article.find(params[:id])
  
  # 然后才是销毁逻辑
  if @article.destroy
    flash[:success] = "文章删除成功啦"
  else
    flash[:danger] = "删除失败，请重试"
  end
  redirect_to articles_path
end

如果你的destroy方法里没有这行@article = Article.find(params[:id])，那@article肯定是nil，后续调用@article.user必报错。

2. 检查before_action的执行顺序 #

很多人会用before_action把「查找文章」的逻辑抽出来复用，比如：

class ArticlesController < ApplicationController
  # 重点！必须先执行set_article，再执行require_same_user
  before_action :set_article, only: [:show, :edit, :update, :destroy]
  before_action :require_same_user, only: [:edit, :update, :destroy]

  # 其他控制器方法...

  private

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

  def require_same_user
    if current_user != @article.user && !current_user.admin?
      flash[:danger] = "你只能编辑或删除自己的文章哦"
      redirect_to root_path
    end
  end
end

这里一定要注意before_action的顺序：得先让set_article把@article赋值好，再执行require_same_user做权限校验。如果顺序搞反了，require_same_user执行时@article还没被初始化，就会出现nil错误。

3. 额外的边界情况优化 #

为了让代码更健壮，还可以处理两个常见的边界问题：

• 文章不存在的情况：如果用户传的params[:id]对应的文章根本不存在，Article.find会直接抛出RecordNotFound错误。可以改成find_by并添加判断：

def set_article
  @article = Article.find_by(id: params[:id])
  unless @article
    flash[:danger] = "找不到这篇文章哦"
    redirect_to articles_path
  end
end

• 未登录用户访问的情况：如果用户没登录就想删文章，current_user会是nil，这时候current_user != @article.user也会出问题。可以在require_same_user里先加登录校验：

def require_same_user
  # 先检查用户是否登录
  unless current_user
    flash[:danger] = "请先登录再操作"
    redirect_to login_path
    return
  end

  # 再做权限校验
  if current_user != @article.user && !current_user.admin?
    flash[:danger] = "你只能编辑或删除自己的文章哦"
    redirect_to root_path
  end
end

最常见的问题就是destroy方法里没初始化@article，或者before_action的顺序搞反了。先排查这两点，基本就能解决这个nil错误啦！

内容的提问来源于stack exchange，提问作者goodjobbin85