Suche Rubin Hash für leeren Wert

Question

ich eine rubin Hash habe wie diese
h = {"a" => "1", "b" => "", "c" => "2"}
Jetzt habe ich eine Ruby-Funktion, die diesen Hash auswertet und gibt true zurück, wenn sie einen Schlüssel mit einem leeren Wert finden. Ich habe die folgende Funktion, die immer true zurück, auch wenn alle Schlüssel in der Hash nicht leer

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

Was mache ich hier falsch? Bitte helfen

Danke, Abhi

Answer 1

Ich hoffe‘ bereit, hier etwas Rubinkarikatur zu lernen. Ich würde eine solche Funktion nicht global definieren wie Sie. Wenn es auf einem Hash eine Operation ist, als sollte es eine Instanz-Methode für die Hash-Klasse sein, das Sie es wie folgt tun:

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

reject einen neuen Hash mit allen leeren Strings zurückgeben, und als es sein wird, überprüft, wie groß dieser neue Hash ist.

eine möglicherweise effizientere Art und Weise (es sollte das gesamte Array nicht durchqueren):

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

Aber das wird noch die ganze Hash durchqueren, wenn kein leerer Wert ist

ich habe mich verändert Die empty? zu !nil? || length >0 weil ich nicht weiß, wie Ihre empty Methode funktioniert.

Answer 2

Try this:

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

Oder:

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

Wenn Sie eine alte 1.8.x Rubin verwenden

Answer 3

Wenn Sie wollen einfach nur überprüfen, ob einer der Werte ein leerer String könnte man

h.has_value?('') 

tun, aber Ihre Funktion scheint gut zu funktionieren.

Answer 4

Ich würde eine Refactoring Ihrer Modelldomäne erwägen. Offensichtlich repräsentiert das Hash etwas Greifbares. Warum nicht ein Objekt machen? Wenn das Element vollständig durch einen Hash dargestellt werden kann, möchten Sie vielleicht Hash ableiten. Wenn es komplizierter ist, kann der Hash ein Attribut sein.

Zweitens kann der Grund, für den Sie Leerzeichen überprüfen, benannt werden, um Ihre Domäne besser widerzuspiegeln. Sie haben uns nicht das "Warum" gesagt, aber nehmen wir an, dass Ihr Artikel nur gültig ist, wenn er keine leeren Werte enthält.

class MyItem < Hash 

    def valid? 
    !invalid? 
    end 

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

Der Punkt ist, wenn Sie ein Vokabular, das Sinn herstellen können in Ihrer Domain macht, wird Ihr Code sauberer und verständlich sein. Die Verwendung eines Hash ist nur ein Mittel zum Zweck, und Sie verwenden besser beschreibende, domänenspezifische Begriffe.

Mit dem obigen Beispiel würden Sie in der Lage zu tun: für mich

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

my_item.valid? #=> false