“一个、另一个或两个”情况的更好解决方案

Question

我正在检查一些代码，然后出现了类似于以下内容：

def between_dates(date_1, date_2)
  if date_1 && date_2
    conditions "created_at >= date_1 AND created_at <= date_2"
  elseif date_1
    conditions "created_at >= date_1"
  elseif date_2
    conditions "created_at <= date_2"
  end
end

它看起来是可以改进的那种代码，但是对于这种琐碎而常见的条件语句，我找不到更优雅的解决方案。

当我们必须为一个、另一个或两者返回一个值时，我正在为这个问题寻找更好的答案。

Answer 1

Rails 允许您动态构建查询。这是一个使用scopes 和类方法的示例。由于作用域总是返回一个ActiveRecord::Relation 对象（即使块返回nil），它们是可链接的：

class Event < ApplicationRecord
  scope :created_before, -> (date) { where('created_at <= ?', date) if date }
  scope :created_after,  -> (date) { where('created_at >= ?', date) if date }

  def self.created_between(date_1, date_2)
    created_after(date_1).created_before(date_2)
  end
end

示例用法：

Event.created_between(nil, Date.today)
# SELECT `events`.* FROM `events` WHERE (created_at <= '2018-05-15')

Event.created_between(Date.yesterday, nil)
# SELECT `events`.* FROM `events` WHERE (created_at >= '2018-05-14')

Event.created_between(Date.yesterday, Date.today)
# SELECT `events`.* FROM `events` WHERE (created_at >= '2018-05-14') AND (created_at <= '2018-05-15')

Answer 2

我会使用这样的东西：

def between_dates(date_1, date_2)
  parts = []

  if date_1
    parts << "created_at >= date_1"
  end

  if date_2
    parts << "created_at <= date_2"
  end

  full = parts.join(' AND ')
  conditions(full)
end

这可以通过多种方式进一步美化，但你明白了。

Answer 3

def between_dates(date_1, date_2)
  date_conditions = []
  date_conditions << 'created_at >= date_1' if date_1
  date_conditions << 'created_at <= date_2' if date_2
  conditions date_conditions.join(' AND ') unless date_conditions.empty?
end

Answer 4

我不确定这是否更优雅，但我总是做reduce 一切以避免拼写错误：

[[date_1, '>='], [date_2, '<=']].
  select(&:first).
  map { |date, sign| "created_at #{sign} #{date}" }.
  join(' AND ')