Buscar ruby ​​hash de valor vacío

Question

tengo un hash de rubí como esto
h = {"a" => "1", "b" => "", "c" => "2"}
Ahora tiene una función de rubí que evalúa este hash y devuelve verdadero si encuentra una clave con un valor vacío. Tengo la siguiente función, que siempre devuelve cierto incluso si todas las claves en el hash no están vacíos

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

¿Qué estoy haciendo mal aquí? Por favor, ayuda

Gracias, Abhi

Answer 1

Te espero' estoy listo para aprender un poco de magia de rubí aquí. No definiría esa función globalmente como lo hizo. Si se trata de una operación en un hash, de lo que debería ser un método de instancia en la clase Hash puede hacerlo de esta manera:

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

reject volverá un nuevo hash con todas las cadenas vacías, y de lo que será comprobó qué tan grande es este nuevo hash.

ser un medio más eficiente (que no debe atravesar toda la matriz):

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

Pero esto todavía va a recorrer todo el hachís, si no hay un valor vacío

He cambiado empty? a !nil? || length >0 porque no sé cómo funciona su método empty.

Answer 2

Prueba esto:

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

O:

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

Si está utilizando un viejo 1.8.x Rubí

Answer 3

Si lo que desea es comprobar si alguno de los valores es una cadena vacía que podría hacer

h.has_value?('') 

pero su función parece funcionar bien.

Answer 4

Considero refactorizar el dominio de su modelo. Obviamente, el hash representa algo tangible. ¿Por qué no convertirlo en un objeto? Si el elemento puede estar completamente representado por un hash, es posible que desee crear una subclase Hash. Si es más complicado, el hash puede ser un atributo.

En segundo lugar, se puede nombrar el motivo por el que está marcando espacios en blanco para reflejar mejor su dominio. No nos ha dicho el "por qué", pero supongamos que su artículo solo es válido si no tiene ningún valor en blanco.

class MyItem < Hash 

    def valid? 
    !invalid? 
    end 

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

El punto es, si se puede establecer un vocabulario que tiene sentido en su dominio, su código será más limpio y más comprensible. Usar un Hash es solo un medio para un fin y sería mejor usar términos más descriptivos y específicos del dominio.

Utilizando el ejemplo anterior, usted sería capaz de hacer:

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

my_item.valid? #=> false