搜索紅寶石哈希值爲空

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