无法重构代码 rails 4 查询

Question

场景是，我想获取 2 个案例的所有时间段。

1. 如果门卫为真，则查询将相同

2. 如果 doorman 为 false 则需要在查询中添加参数

因此，几乎相同的查询几乎可以在两种情况下都适用，只需稍作修改。

这是查询和代码：

def self.latest_pickup_date current_zone,doorman
    if doorman
      latest_timeslot = Timeslot.where(dropoff_slots: '-1', zone_id: current_zone).order(:slot_date).last
    else
      latest_timeslot = Timeslot.where(dropoff_slots: '-1', zone_id: current_zone, doorman_type: "none").order(:slot_date).last
    end
    latest_timeslot.nil? ? Date.current : latest_timeslot.slot_date
  end

我想以使用 DRY 方法的方式重构我的代码和查询。

我不想在这两种情况下都写两次这些查询。我需要使用代码实践的更好解决方案。或者，如果我这样做是正确的，您也可以提出建议。

如果有人可以提供帮助，还需要良好的专业代码实践和代码重构。

Answer 1

您可以对现有查询执行 where 以添加附加条件，并使用 try 以防查询为空

def self.latest_pickup_date current_zone,doormam
  latest_timeslot = Timeslot.where(dropoff_slots: '-1', zone_id: current_zone).order(:slot_date)
  latest_timeslot = latest_timeslot.where(doorman_type: 'none') unless doorman
  latest_timeslot.last.try(:slot_date) || Date.current
end

Answer 2

您能否检查以下重构代码，如果您喜欢这种方法，请告诉我。

def self.latest_pickup_date current_zone,doorman
  filters = {dropoff_slots: '-1', zone_id: current_zone}
  filters[:doorman_type] = "none" unless doorman

  latest_timeslot = Timeslot.where(filters).order(:slot_date).last

  latest_timeslot.nil? ? Date.current : latest_timeslot.slot_date
end