Ruby on Rails：組織大搜索方法

Question

我的模型中有很大的搜索方法，對應於搜索合適的汽車。 它看起來像這樣：

def self.search(params) 
    cars = joins(:reservations).where.not("reservations.reception_time <= ? AND reservations.return_time >= ?", 
     params[:return_date], params[:handover_date]) 
    cars = joins(:car_class).where("car_classes.id= ?", params[:car_class]) 
    cars = cars_at_both_locations(params[:handover_location], params[:return_location]) 
    cars = params[:car_body_style] == [""] ? cars : joins(:car_configuration). 
     where("car_configurations.body_style_id = ?", params[:car_body_style]) 
    cars = params[:car_fuel] == [""] ? cars : where(fuel: params[:car_fuel]) 
    cars = params[:car_transmission] == [""] ? cars : where(transmission: params[:car_transmission]) 
    cars = params [:car_seats] == [""] ? cars : car_seats(params[:car_seats]) 
    cars = Car.joins(:prices).where('prices.to_days >= ?', 
     (Date.parse(params[:return_date]) - Date.parse(params[:handover_date])).to_i) 
    end 

這是非常不可讀... 有沒有人有任何想法，以重構這個方法是更具可讀性？

Answer 1

你應該在你的汽車模型

使用適當的名稱範圍爲每個方法，例如你的第一個兩行，加在你的模型

scope :available, ->(handover_date,return_date) { joins(:reservations).where.not("reservations.reception_time <= ? AND reservations.return_time >= ?", 
     return_date, handover_date)} 
scope :with_class, ->(car_class_id) {where("car_classes.id= ?", car_class_id)} 

然後您將只需要編寫

def self.search(opts) 
    available(opts[:handover_date],opts[:return_date]). 
    with_class(opts[:car_class_id]). 
[...] 
end 

您還可以使用隨處可用的示波器和類，這也非常酷。