Question

I am trying to delete an array of users but the way I have it it is deleting one by one. Is there a better way to do it?

My code is:

@users ||= User.where("clicks_given - clicks_received < ?", -5).to_a
@users.each do |user|
  user.destroy
end

Answer 1

You can just use Rails’ built-in methods. Note that you need to wrap your query in an array (if you’re interpolating variables) when using these methods.

To iterate over each one calling destroy (which will run callbacks, etc.):

User.destroy_all(["clicks_given - clicks_received < ?", -5])

Or to just delete these in the database in a single query (no iteration over each item), you can do this, but keep in mind it won’t run your callbacks:

User.delete_all(["clicks_given - clicks_received < ?", -5])