ruby-on-rails - 给定 id 列表，如何更新所有模型的 bool 属性？

Question

我正在做类似于这些 railscast 剧集的事情:

http://railscasts.com/episodes/165-edit-multiple

http://railscasts.com/episodes/52-update-through-checkboxes

问题是那些只是试图修改选定的模型。我需要更新每个模型。

我发现的第一件事是 id not in () 并没有像我预期的那样返回所有内容，所以我不得不为空列表做一个特例。

此代码有效，但看起来不是很干。至少我应该能够将正常情况合并到一行中。

def update_published
  if params[:book_ids].empty?
    Book.update_all(published: false)
  else
    Book.where(id: params[:book_ids]).update_all(published: true)
    Book.where("id not in (?)", params[:book_ids]).update_all(published: false)
  end
  redirect_to books_path
end

如有任何改进意见，我们将不胜感激。

Answer 1

为什么不只执行以下操作:

def update_published
  Book.update_all(published: false)
  Book.where(id: params[:book_ids]).update_all(published: true)
  redirect_to books_path
end

它会更快，而且非常简单明了。