0

在我的项目中，我有一个具有四个属性的模型：名称、描述、内容和完成。

我想在保存之前检查所有属性，除了已完成是否为空白。如果不为空，则将已完成设置为 1，否则设置为 0。

我使用 ActiveRecord 和 Mysql，所以它也会有 id、created_at 和 updated_at 列。

我写了一个这样的 before_save 回调：

def check_completed
  if self.attributes.except("id", "created_at", "updated_at").all? {|k, v| v.present?}
    self.completed = 1
  else
    self.completed = 0
  end
end

它有效，但看起来很难看。我想删除 except 函数调用。

有没有更好的方法来做到这一点？

4

2 回答 2

0

我建议不要依赖，self.attributes因为您将来可能希望为模型添加其他属性。如果是这样，您必须将新属性添加到您的except列表中。

相反，您可以执行以下操作：

self.completed = %w(name description content).all? { |attr| !send(attr).blank? } ? 1 : 0
于 2013-08-29T06:58:42.003 回答
0

我假设使用 .except 方法是因为 id、created_at 和 updated_at 都是由 MySQL 内部生成和管理的。因此，该列表扩展或更改将是不寻常的。我同意提供的代码很好。如果您想缩短它，您可以使用三元组，如下所示：

def check_completed
  self.attributes.except("id", "created_at", "updated_at").all? {|k, v| v.present?} ? 1 : 0
end

消除 .except 方法可以让您在模型更改时随时管理此方法。

扩展此讨论：我很好奇您希望返回 1 还是 0？没有看到更多代码，我不确定您的意图。但是，如果“之前”回调返回 false，则停止执行并回滚事务。在任何其他情况下，继续执行。在 Ruby 中，0 不是假的。False 仅由 false 或 nil 触发。我的期望是它更有可能使用 true 代替 1 和 false 代替 0？如果是这样，代码将是：

def check_completed
  self.attributes.except("id", "created_at", "updated_at").all? {|k, v| v.present?} ? true : false
end

这样，如果任何用户属性不存在，事务将被取消并回滚。但是，这取决于你。

于 2013-08-29T23:26:23.700 回答