Question

我有以下方法从我的Rails应用程序中的Ruby哈希中删除指定的值：

def remove_hash_values(hash, target = nil)
    hash.delete_if {|key, value| value == target}
    hash.each_value {|obj| remove_hash_values(obj, target) if obj.is_a?(Hash)}
end

这样的工作：

remove_hash_values(some_hash, :some_symbol)

但是，不工作：

remove_hash_values(some_hash, {})

它适用于irb，这让我很困惑。我很肯定正在传递正确的哈希（用许多puts语句检查）。我的Ruby版本是ruby-2.0.0-p247，我正在使用Rails 3.非常感谢任何帮助。

编辑：这不起作用：

def remove_hash_values(hash, target = nil)
    hash.each do |key, value|
        hash.delete(key) if value == target
    end
    hash.each_value {|obj| remove_hash_values(obj, target) if obj.is_a?(Hash)}
end

我到底做错了什么？！

编辑2 ：刚才意识到我实际上使用的是HashWithIndifferentAccess，而不是Hash。这可能会引起一些欺骗，所以我会先尝试将其转换为Hash并报告回来。

Answer 1

我尝试过类似的东西，例如

2.0.0-p195 :092 > class Hash
2.0.0-p195 :093?>   def delete_values! target
2.0.0-p195 :094?>     delete_if { |k, v| v == target }
2.0.0-p195 :095?>     each_value { |v| v.delete_values!(target) if v.is_a?(Hash) }
2.0.0-p195 :096?>   end
2.0.0-p195 :097?> end

工作正常：

2.0.0-p195 :098 > { a: 1, b: 2, c: 3 }.delete_values! 2
 => {:a=>1, :c=>3}

也有效：

2.0.0-p195 :101 > { a: 1, b: 2, c: { d: {}, e: 5 } }.delete_values!({})
 => {:a=>1, :b=>2, :c=>{:e=>5}}

请注意，如果您应该在结构中进一步向下进行空哈希，则此方法不会删除空哈希 - 如果您的哈希只包含空哈希 - 您需要切换递归和删除的顺序，例如

2.0.0-p195 :092 > class Hash
2.0.0-p195 :093?>   def delete_values! target
2.0.0-p195 :094?>     each_value { |v| v.delete_values!(target) if v.is_a?(Hash) }
2.0.0-p195 :095?>     delete_if { |k, v| v == target }
2.0.0-p195 :096?>   end
2.0.0-p195 :097?> end

Answer 2

（显示为答案而不是评论以启用代码显示。）

请分享您的失败测试。从使用Ruby 2.0p247的Rails 4.0控制台运行时，以下工作原理：

def remove_hash_values(hash, target = nil)
  hash.delete_if {|key, value| value == target}
  hash.each_value {|obj| remove_hash_values(obj, target) if obj.is_a?(Hash)}
end

some_hash = HashWithIndifferentAccess.new
some_hash[:foo] = :some_symbol
some_hash[:bar] = {}

remove_hash_values(some_hash, {})

puts some_hash.inspect   # => {"foo"=>:some_symbol}

Answer 3

Isaac，我喜欢你使用递归的想法，因为它处理嵌套到任何级别的哈希。我想我明白为什么你的代码不起作用，以及你如何解决它。这是你的代码：

def remove_hash_values(hash, target = nil)
  hash.delete_if {|key, value| value == target}
  hash.each_value {|obj| remove_hash_values(obj, target) if obj.is_a?(Hash)}
end

假设

hash = {a: 10, b: {c: 10}}

我们调用：

remove_hash_values(hash, 10)

我希望您希望remove_hash_values(hash, 10)返回{}，但我相信它会返回{b: {}}。我们来看看计算：

remove_hash_values(hash, 10)
hash.delete_if {|key, value| value == target}       # hash => {b: {c: 10}} 
hash.each_value {|obj| remove_hash_values(obj, target) if obj.is_a?(Hash)} calls (next line)

  remove_hash_values({c: 10})
  hash.delete_if {|key, value| value == target} # hash => {} 
  hash.each_value {|obj| remove_hash_values(obj, target) if obj.is_a?(Hash)} # hash => {}

hash => {b: {}}

我认为你可以做到这一点，虽然我还没有检查过我的代码。首先假设target不是hash或其他结构，只是一个简单的值。试试这个：

def remove_hash_values(hash, target = nil)
  hash.keys.each do |key|
    value = hash[key]
    case value
    when Hash
      val = remove_hash_values(hash[key], target)  
      if val == {}
        hash.delete(key)
      else
        hash[key] = val
      end
    else
      hash.delete(key) if val == target
    end
  end
  hash
end

如果你想概括这一点，那么target可能是一个（可能是嵌套的）哈希，我认为这可能有用：

def remove_hash_values(hash, target = nil)
  hash.keys.each do |key|
    value = hash[key]
    if value.is_a? Hash
      if target.is_a? Hash && hash[key] == target
        hash.delete(key)
        next
      end
    else
      # value is not a hash
      hash.delete(key) if !(target.is_a? Hash) && if hash[key] == target)
      next
    end
    # Value is a hash, target may or may not be a hash, value != target
    val = remove_hash_values(hash[key], target)  
    if val == target
      hash.delete(key)
    else    
      hash(key)= val
    end
  end
  hash
end

Ruby哈希'delete_if'在与空哈希比较时不起作用

3 个答案: