Devise的destroy_with_password始终返回false，无法删除用户账号

嘿，我之前也碰到过一模一样的坑！咱们一步步拆解问题根源：

核心问题：参数传递错误 + 不必要的验证干扰 #

你现在把完整的user_params对象传给了destroy_with_password，这里有两个关键问题：

1. Devise的destroy_with_password本质上只需要当前用户的密码——要么直接传密码字符串，要么传仅包含:current_password键的哈希。你传入的user_params里包含了:first_name、:last_name等其他字段，但删除表单并没有提交这些值，导致这些字段被设为nil。
2. 如果你的User模型对这些字段（比如first_name）有presence: true的验证，调用destroy_with_password时会触发这些验证，验证失败就会返回false，账号自然不会被删除。

快速修复方案 #

方案1：直接传递密码字符串（最简单）

修改destroy动作，只提取current_password传给方法：

def destroy
  @user = current_user # 这里不用find(current_user.id)，直接用current_user更简洁
  if @user.destroy_with_password(user_params[:current_password])
    redirect_to root_url, notice: "User deleted."
  else
    flash[:alert] = "Couldn't delete: #{@user.errors.full_messages.join(', ')}"
    redirect_to root_url
  end
end

这样完全避免了多余字段带来的验证问题。

方案2：为销毁动作单独定义参数白名单

如果更习惯用哈希传递参数，可以专门写一个仅允许:current_password的参数方法：

def destroy_user_params
  params.require(:user).permit(:current_password)
end

def destroy
  @user = current_user
  if @user.destroy_with_password(destroy_user_params)
    redirect_to root_url, notice: "User deleted."
  else
    flash[:alert] = "Couldn't delete: #{@user.errors.full_messages.join(', ')}"
    redirect_to root_url
  end
end

关键调试技巧 #

下次遇到方法返回false的情况，一定要在binding.pry里查看具体错误：

binding.pry
@user.errors.full_messages # 执行这个就能看到失败原因，比如"First name can't be blank"

这能帮你瞬间定位问题！

额外优化：调整模型验证时机 #

如果User模型有必填字段验证，建议指定仅在创建/更新时触发，销毁时跳过：

class User < ApplicationRecord
  validates :first_name, :last_name, presence: true, on: [:create, :update]
  # 其他Devise配置...
end

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