Rails "find_all_by" vs ".where"

Question

Tôi có đoạn mã sau:

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

Tôi đã suy nghĩ về việc thay đổi nó để:

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

Nên có bất kỳ lợi thế này? Có vẻ như .where phổ biến hơn find_all_by ngày nay.

Answer 1

Ý kiến ​​của tôi là sử dụng .where là cách tiếp cận tốt hơn.

Khi bạn sử dụng công cụ tìm thuộc tính, bạn sẽ phải thông qua phương thức thiếu cuộc gọi và cuối cùng xác định phương thức lớp, qua class_eval, trả về kết quả của bạn. Đây là xử lý bổ sung mà bạn có thể không cần phải làm.

Ngoài ra, xâu chuỗi lại với nhau: find_by_this_and_this_and_this_and_this ... có thể trở nên xấu xí.

See how rails accomplishes attribute based finders here

Phương mất tích từ DynamicMatchers mô-đun trên 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

Tôi nghĩ lợi thế chính là có thể thêm tiêu chí bổ sung vào vị trí, find_all_by được giới hạn trong trường của công cụ chọn động. Nếu bạn chỉ có một điều kiện mà bạn đang tìm kiếm thì tôi nghĩ đó là một lần rửa, nhưng khi bạn bắt đầu thêm 3 hoặc 4, những người tìm kiếm động có thể xấu xí. Hashes là tốt đẹp để xem xét, và bạn có thể vượt qua một hash của điều kiện như một tham số nếu cần thiết. Công cụ tìm kiếm động rất thú vị, nhưng tôi nghĩ quy mô một cách rõ ràng hơn và dễ đọc hơn.