Czy jest jakiś dobry sposób, aby PODNOSIć powielanie zakresu/predykatu logicznego duplikowania?

Question

Na przykład rozważmy następujący kod (w modelu):

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 

Pierwszy warunek jest SQL, drugi jest rubin, ale wciąż istnieje oczywista powielanie. Czy jest jakiś dobry sposób, aby to WYSOKIE? Jaka jest najlepsza praktyka w takich przypadkach?

Answer 1

Istnieją dobre powody, aby mieć zarówno więc mogę uznać to (wszystkie w modelu):

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