在 Rails 5 中，“model.save”和“model.errors.empty”之间是否存在功能差异？

Question

我正在使用 Rails 5。我的控制器中有这个逻辑......

@user = UserService.create_user(params)
if @user.errors.empty?
  render :create, :status => :ok

上面和上面有什么区别吗

@user = UserService.create_user(params)
if @user.save
  render :create, :status => :ok

我正在尝试决定是使用“.save”还是“.errors.empty？” 逻辑。

Answer 1

两者都只是劣等的做事方式：

@user = UserService.build_user(params) # just instanciate the record
if @user.save
   # ...
else
   # ...
end

功能上有区别吗？ 是的。 .save触发模型上与保存可能产生副作用的记录相关的所有回调。 如果您只想检查它是否已保存，请使用.persisted? . .errors.empty? 是对.valid?的不良模仿.valid? .

两者都不会炸毁任何东西，但它仍然困扰着我，因为它使您的意图变得不那么清晰。 如果您执行以下操作，它还会创建两个查询（并触发更新回调）：

@user = UserService.create_user(params)
@user.foo = bar
if @user.save
   # ...
else
   # ...
end

当您想将参数与来自其他地方（如会话）的信息合并时，这非常常见。

render :create, status: :ok也很奇怪。 通常你只想重定向到新创建的资源，因为这会影响历史记录，这样浏览器实际上会返回到正确的页面而不是/users如果用户点击后退按钮。 对于 api，您可以使用status: :created, location: @user或资源的 JSON 表示进行响应。