将 AssociatedModel 或 AssociatedModel.id 分配给 ActiveRecordModel 的最佳方法是什么？

Question

我正在尝试构建一个辅助方法，该方法将采用关键字 args 并基于某些逻辑对它们进行处理，这是一个示例：

def update_with_logic(widget:, user:)
  # do some logic here
  widget.user = user
end

现在这工作得很好，但是，我想支持 user 也被设置为用户的 ID。

但是，如果我们将 user: 3 发送到我们的 update_with_logic 方法中，我会收到以下错误：

ActiveRecord::AssociationTypeMismatch: User(#47287814961100) expected, got 3 which is an instance of Integer(#47287745166420)

如果您换一种方式尝试：widget.user_id = user 和 user 是 ActiveRecord::Model，它会将 user_id 设置为 nil（并且不会引发异常）

这让我很吃惊，因为我可以在 ActiveRecord::Query 设置中毫无问题地执行以下操作：

userModel = User.find 3

# Looking up by user_id while sending a user model as the value
Widget.where(user_id: userModel).to_sql
# => SELECT "widgets".* FROM "widgets" WHERE "widgets"."user_id" = 3

# Looking up by user association while sending a user_id (integer) as the value
Widget.where(user: 3).to_sql
# => SELECT "widgets".* FROM "widgets" WHERE "widgets"."user_id" = 3

我尝试过使用widget.attributes = { user: user }，但是使用user = 3 时效果相同。 （引发 ActiveRecord::AssociationTypeMismatch）。

我找到的唯一解决方案是执行以下操作：

widget.user_id = user.is_a?(User) ? user.id : user

上面的解决方案有效，但是，它并不像一个干净的实现。我希望有人可以推荐一些更符合“The Rails Way®” ;)

Answer 1

您可以提供ActiveRecord 这样一团糟，只是因为它确实some heavylifting 来构建正确的查询。实际上并没有什么魔法——所有这些方便的多态性都以代码复杂性为代价。

您的带有try 的代码有点危险——如果提供的不是用户实例而是任何其他模型（它也会响应id）怎么办？例如，你可以让它更防弹

case user
when Integer
  widget.user_id = user
when User
  widget.user = user
else
  raise "BOOM!"
end

但这也不理想 - 如果提供的 id 用户不存在怎么办？当然，我们可以很容易地添加检查（只是像widget.user = User.find(user) 那样做），但我们真的需要所有这些意外的复杂性吗？

如果我是你，我会停下来问自己一个问题：为什么这个方法的调用不一致？如果我们希望它为一个小部件分配一个用户，为什么我们要提供的不是用户而是其他东西（反之亦然）？真的无法避免吗？

在大多数实际情况下，答案都是“否”：可以在调用此方法的位置进行必要的更改，以便它始终接收一致的参数 - 无论是用户或有效的用户 ID，- 并保持方法本身简洁明了。