搜索红宝石哈希值为空

Question

我有一个红宝石散列这样
h = {"a" => "1", "b" => "", "c" => "2"}
现在我有一个评估该哈希，如果找到一个空值一键返回true红宝石功能。我有以下函数始终返回true，即使在哈希的所有键都没有空

def hash_has_blank(hsh) 
    hsh.each do |k,v| 
    if v.empty? 
     return true 
    end 
    end 
    return false 
end 

我在做什么错在这里？请帮助

感谢，ABHI

Answer 1

我希望你”

def hash_has_blank hsh 
    hsh.values.any?{|i|i.empty?} 
end 

：

def hash_has_blank hsh 
    hsh.values.any? &:empty? 
end 

或者准备在这里学习一些红宝石魔法。我不会像你那样在全球范围内定义这样的功能。如果它是一个散列的操作，比它应该是在哈希类的实例方法，你可以做这样的：

class Hash 
    def has_blank? 
    self.reject{|k,v| !v.nil? || v.length > 0}.size > 0 
    end 
end 

reject会返回一个新的哈希值与所有的空字符串，而且比这将是检查这个新散列有多大。

一个可能更有效的方式（它不应该遍历整个数组）：

class Hash 
    def has_blank? 
    self.values.any?{|v| v.nil? || v.length == 0} 
    end 
end 

但是，这仍然会遍历整个哈希，如果没有空值

我已经改变empty?到!nil? || length >0，因为我不知道你的empty方法是如何工作的。

Answer 2

试试这个：如果您使用的是旧1.8.x的红宝石

Answer 3

如果你只是要检查是否有任何价值的是一个空字符串，你可以做

h.has_value?('') 

，但你的功能似乎很好地工作。

Answer 4

我会考虑重构你的模型域。显然，哈希表示一些有形的东西。为什么不把它作为一个对象？如果该项目可以完全由散列表示，那么您可能希望子类化哈希。如果它更复杂，散列可以是一个属性。

其次，检查空白的原因可以命名为更好地反映您的域名。你没有告诉我们“为什么”，但是让我们假设你的物品只有在没有任何空白值的情况下才有效。

class MyItem < Hash 

    def valid? 
    !invalid? 
    end 

    def invalid? 
    values.any?{|i| i.empty?} 
    end 
end 

问题是，如果您可以建立一个在您的域中有意义的词汇表，那么您的代码将变得更加清晰和易于理解。使用散列只是达到目的的一种手段，您最好使用更具描述性的，特定领域的术语。

使用上面的例子，你能够做到：

my_item = MyItem["a" => "1", "b" => "", "c" => "2"] 

my_item.valid? #=> false