रेल "find_all_by" बनाम ".where"

Question

मैं निम्नलिखित कोड है:

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

मैं करने के लिए इसे बदलने के बारे में सोच रहा था:

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

वहाँ इस के लिए किसी भी लाभ होगा? ऐसा लगता है कि .where आजकल find_all_by से अधिक आम है।

Answer 1

मेरी राय यह है कि .where का उपयोग करना एक बेहतर तरीका है।

जब आप विशेषता आधारित खोजकर्ताओं का उपयोग करते हैं, तो आपको class_eval के माध्यम से एक विधि विधि को परिभाषित करने के लिए एक विधि विधि को परिभाषित करना होगा, जो आपका परिणाम देता है। यह अतिरिक्त प्रसंस्करण है जिसे आपको करने की आवश्यकता नहीं हो सकती है।

इसके अलावा, एक साथ स्ट्रिंग: find_by_this_and_this_and_this_and_this ... बदसूरत हो सकता है।

See how rails accomplishes attribute based finders here

विधि मॉड्यूल DynamicMatchers 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

मुझे लगता है कि मुख्य लाभ डायनामिक चयनकर्ता के क्षेत्र तक सीमित है, जहां अतिरिक्त मानदंड जोड़ने में सक्षम है। यदि आपके पास केवल एक शर्त है जिसे आप खोज रहे हैं तो मुझे लगता है कि यह एक धो है, लेकिन जब आप 3 या 4 जोड़ना शुरू करते हैं, तो गतिशील खोजकर्ता बदसूरत हो सकते हैं। हैश को देखने के लिए अच्छा लगा है, और यदि आवश्यक हो तो आप पैरामीटर के रूप में स्थितियों के हैंश पास कर सकते हैं। गतिशील खोजक शांत हैं, लेकिन मुझे लगता है कि क्लीनर तरीके से स्केल कहां और अधिक पठनीय है।