rieles "find_all_by" vs ".donde"

Question

Tengo el siguiente código:

def maturities 
    InfoItem.find_all_by_work_order(self.work_order).map(&:maturity) 
end 

estaba pensando en cambiarlo a:

def maturities 
    InfoItem.where(work_order: self.work_order).map(&:maturity) 
end 

Habría alguna ventaja para esto? Parece que .where es más común que find_all_by en la actualidad.

Answer 1

Mi opinión es que usar .where es una mejor opción.

Cuando utiliza buscadores basados ​​en atributos, tendrá que pasar por un túnel a través de una llamada faltante del método y finalmente definir un método de clase, a través del class_eval, que devuelve el resultado. Este es un procesamiento adicional que quizás no necesites hacer.

Además, encadenar juntos: find_by_this_and_this_and_this_and_this ... puede ponerse feo.

See how rails accomplishes attribute based finders here

Método falta de DynamicMatchers módulo en github:

def method_missing(name, *arguments, &block) 
    match = Method.match(self, name) 

    if match && match.valid? 
    match.define 
    send(name, *arguments, &block) 
    else 
    super 
    end 
end 

Answer 2

Creo que la principal ventaja es poder agregar criterios adicionales a donde, find_all_by está limitado al campo del selector dinámico. Si solo tienes una condición por la que estás buscando, creo que es un lavado, pero cuando comienzas a agregar 3 o 4, los buscadores dinámicos pueden ser feos. Los hash son agradables de mirar, y podrías pasar un hash de condiciones como parámetro si es necesario. Los buscadores dinámicos son geniales, pero creo que escalan de una manera más limpia y es más legible.