0

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

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

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>

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

我究竟做错了什么？

4

3 回答 3

2

这种方法

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 方法：它们是简单的读写到实例变量方法的方法。验证具有不同的方法名称，因此它不会与访问器方法名称发生冲突。

于 2015-05-12T15:12:17.233 回答
1

current_password 正在调用自己。你有一个无限递归。

你基本上是这样做的：

def current_password
  self.current_password
end
于 2015-05-12T15:13:56.370 回答
1

正如其他答案所说，您在 current_password 中有一个无限循环。我会将验证逻辑提取到它自己的方法中。类似于以下内容。

validate :has_correct_current_password, on: :update

def has_correct_current_password
  errors[:current_password] << 'Incorrect password' unless current_password == password
end
于 2015-05-12T15:19:19.177 回答