Rails ActiveRecord where clause

Question

我想从带有where子句的数据库中选择Cars，为我的问题寻找最佳DRY方法。

例如，我有两个参数

params[:car_model_id] (int)
params[:transmission_id] (int)
params[:from_date]
params[:to_date]

但我不知道哪一个会是空的

if params[:car_model_id].nil? && !params[:transmission_id].nil?
    if params[:from_date].nil? && params[:from_date].nil?
        return Car.where(:transmission_id => params[:transmission_id])
    else
        return Car.where(:transmission_id => params[:transmission_id], :date => params[:from_date]..params[:to_date])
    end
elseif !params[:car_model_id].nil? && params[:transmission_id].nil?
    if params[:from_date].nil? && params[:from_date].nil?
        return Car.where(:car_model_id=> params[:car_model_id])
    else
        return Car.where(:car_model_id=> params[:car_model_id], :date => params[:from_date]..params[:to_date])
    end
else
   return Car.where(:car_model_id=> params[:car_model_id], :transmission_id => params[:transmission_id], :date => params[:from_date]..params[:to_date])
end

避免这种错误代码的最佳方法是什么，并检查参数是否为nil内联（在where中）

Answer 1

你可以做：

car_params = params.slice(:car_model_id, :transmission_id).reject{|k, v| v.nil? }

然后：

Car.where(car_params)

解释：由于，您正在检查params中是否存在特定密钥，即：:car_model_id和transmission_id。当params:中只有:transimission_id时，上面的代码会是这样的

Car.where(:transmission_id => '1')

或者当params:中有:car_model_id时

Car.where(:car_model_id => '3')

或者当你同时拥有时

Car.where(:transmission_id => '1', :car_model_id => '3')

注意：只有当您将params键作为要运行查询的列名时，这才会起作用。如果您打算在params中有一个与列名不匹配的不同键，那么我建议您在slice之前将其键更改为控制器本身中的列名。

更新：从那以后，OP编辑了他的问题，现在引入了更多的if.. else条件。解决这个问题的一种方法是始终记住一件事，那就是让user_params的值正确，您希望在模型类上运行查询，这里是Car。因此，在这种情况下：

car_params = params.slice(:car_model_id, :transmission_id).reject{|k, v| v.nil? }
if params[:from_date].present? && params[:from_date].present?
  car_params.merge!(date: params[:from_date]..params[:to_date])
end

然后：

Car.where(car_params)

Answer 2

避免这种错误代码的最佳方法是什么，并检查参数是否为无内联

好问题！

我将使用两个额外的布尔变量（transmission_id_is_valid和car_model_id_is_valid）

transmission_id_is_valid = params[:car_model_id].nil? && !params[:transmission_id].nil?
car_model_id_is_valid = !params[:car_model_id].nil? && params[:transmission_id].nil?
if transmission_id_is_valid
   return Car.where(:transmission_id => params[:transmission_id])
elseif car_model_id_is_valid
   return Car.where(:car_model_id=> params[:car_model_id])
....
end

我认为现在更容易被人类阅读。

Answer 3

首先，我会将此代码更改为Car模型，我认为没有必要检查params是否不存在。

# using Rails 4 methods
class Car < ActiveRecord::Base
  def self.find_by_transmission_id_or_model_id(trasmission_id, model_id)
     if transmission_id
       find_by trasmission_id: trasmission_id
     elsif model_id
       find_by model_id: model_id
     end
  end
end

控制器内：

def action
   car = Car.find_by_transmission_id_or_model_id params[:trasmission_id], params[:car_model_id]
end

编辑：

当您只有两个参数时，此代码很好。对于许多条件参数，请查看ransak-gem。