क्या ड्रॉ अप करने के लिए कोई अच्छा तरीका है/तर्क तर्क डुप्लिकेट?

Question

उदाहरण के लिए, निम्नलिखित कोड (एक मॉडल में) पर विचार करें:

scope :popular, where("views >= 250 OR (views >= 10 AND avg_rating >= 4.75)") 

def popular? 
    views >= 250 or views >= 10 && avg_rating >= 4.75 
end 

पहली शर्त एसक्यूएल है, दूसरा एक गहरे लाल रंग का है, लेकिन अभी भी, वहाँ एक स्पष्ट दोहराव है। क्या इसे सूखने का कोई अच्छा तरीका है? ऐसे मामलों के लिए सबसे अच्छा अभ्यास क्या है?

Answer 1

अच्छे कारणों के लिए दोनों तो मैं (सभी मॉडल में) इस पर विचार हो सकता हैं:

VIEWS_QUALIFIER = 250 
RATING_VIEWS_QUALIFIER = 10 
RATING_QUALIFIER = 4.75 

scope :popular, where("views >= ? OR (views >= ? AND avg_rating >= ? ", 
       VIEWS_QUALIFIER, RATING_VIEWS_QUALIFIER, RATING_QUALIFIER) 

def popular? 
    views >= VIEWS_QUALIFIER or 
    (views >= RATING_VIEWS_QUALIFIER && avg_rating >= RATING_QUALIFIER) 
end 

Answer 2

def popular? 
    !!self.class.popular.includes? self 
end