需要当前密码才能设置新密码

Question

我希望用户在编辑个人资料时输入其当前密码。

这是我的用户模型的样子：

attr_accessor :current_password

def current_password
  errors[:current_password] << 'Incorrect password' unless self.current_password == self.password
end

validate :current_password, on: :update

和我的控制器参数：

def user_params
  params.require(:user).permit(:email, :name, :current_password, :password, :password_confirmation, :phone)
end

和用户表格部分：

<div class="form-group">
  <%= f.label :password, "Current password" %>
  <%= f.password_field :current_password, class: "form-control", placeholder: "Current password" %>
</div>

但是我的堆栈级别太深了，它进入了验证循环。

我究竟做错了什么？

Answer 1

这个方法

def current_password
  errors[:current_password] << 'Incorrect password' unless self.current_password == self.password
end

自我调用：这就是为什么您会得到堆栈太深的错误的原因-它陷入无限循环的循环中。

您正在尝试做的是令人困惑的，因为current_password方法会覆盖attr_accessor :current_password创建的方法，实际上是在进行一些验证，而不仅仅是返回值。 我会这样做，而不是：

attr_accessor :current_password

validate :current_password_same_as_password, on: :update

def current_password_same_as_password
  errors[:current_password] << 'Incorrect password' unless self.current_password == self.password
end

因此，在这里，我只剩下attr_accessor方法：它们是对实例变量方法的简单读写。 验证具有不同的方法名称，因此不会与访问器方法名称冲突。

Answer 2

current_password正在调用自身。 您具有无限递归。

您基本上是在这样做：

def current_password
  self.current_password
end

Answer 3

正如其他答案所说，您在current_password中存在无限循环。 我将验证逻辑引入其自己的方法中。 类似于以下内容。

validate :has_correct_current_password, on: :update

def has_correct_current_password
  errors[:current_password] << 'Incorrect password' unless current_password == password
end