如何遍历含多值的Hash并正确执行destroy操作？

这个问题的核心是搞清楚destroy和destroy_all在ActiveRecord中的用法差异，咱们来一步步修正你的代码：

问题根源 #

• 当Model.where(...)返回单条记录的Relation时，调用record.destroy其实是隐式调用了该记录的实例方法destroy，所以能正常执行；
• 当返回多条记录的Relation时，调用record.destroy会触发ActiveRecord::Relation#destroy方法，这个方法需要传入具体的id参数，而你没传，所以报“参数数量错误”；
• 你说改用destroy_all前两次失败，其实这是个误解——destroy_all本身支持处理单条或多条记录的Relation，它会遍历所有匹配的记录，逐个调用实例的destroy方法，空Relation调用它也不会报错。

修正方案1：统一使用destroy_all #

直接把你的代码改成用destroy_all，去掉多余的判断（因为空Relation调用destroy_all不会有任何操作）：

a = {"type1"=>[11], "type2"=>[12], "type3"=>[13, 14]}
a.each do |type, ids|
  Model.where(type: type, id: ids).destroy_all
end

这样不管ids是单个元素的数组还是多个元素的数组，都能正常执行删除操作，同时会触发每个记录的before_destroy和after_destroy回调（这也是destroy_all和delete_all的关键区别）。

修正方案2：逐个处理每个ID（更直观） #

如果你想明确逐个处理每个id，也可以把数组展开，用实例方法destroy：

a = {"type1"=>[11], "type2"=>[12], "type3"=>[13, 14]}
a.each do |type, ids|
  ids.each do |id|
    # 使用find_by找到记录后调用destroy，&.确保记录不存在时不报错
    Model.find_by(type: type, id: id)&.destroy
  end
end

这个方案的好处是你能清晰看到每个id的处理过程，适合需要对单个记录做额外操作的场景。

为什么之前用destroy_all会失败？ #

如果之前你尝试destroy_all时前两次失败，大概率是因为代码里的判断逻辑问题——比如你写了record.destroy_all if record，但record是Relation对象，在Ruby中只要不是空的Relation，if record都会返回true，空的话返回false，所以其实不会有问题。可能是测试时的其他因素导致的，比如记录不存在或者权限问题，可以检查一下日志确认。

内容的提问来源于stack exchange，提问作者user3260316