Как улучшить код?

Question

[john_bob]

if date_range_present? && search_query_present?
        domain_list = domain_list&.select { |hash|
          hash['date'] >= params[:from_date] &&
          hash['date'] <= params[:to_date] &&
          hash['domainName'].match(params[:search_query])
        }
      elsif date_range_present?
        domain_list = domain_list&.select { |hash|
          hash['date'] >= params[:from_date] &&
          hash['date'] <= params[:to_date]
        }
      elsif search_query_present?
        domain_list = domain_list&.select { |hash|
          hash['domainName'].match(params[:search_query])
        }
      end

Имеется такая логика,
наверняка можно ее переписать, чтобы было меньше повторений.
Подскажите пожалуста, как лучше переписать!

Comments

[Василий Банников]

Можно заменить цепочку из if-elif на pattern matching
Дублирование кода с условиями можно заменить на паттерн specification
А ещё select можно применять по цепочке, типа
(псевдокод)

if a?
  domain_list = domain_list&.select { ... }
if b?
  domain_list = domain_list&.select { ... }

Answer 1

[Everything_is_not_so_bad]

domain_list.select! do |hash|
  v = true
  if date_range_present?
    v &= hash['date'] >= params[:from_date]
    v &= hash['date'] <= params[:to_date]
  end
  if search_query_present?
    v &= hash['domainName'].match(params[:search_query])
  end
  v
end if date_range_present? || search_query_present?

Это не самая короткая запись. Оставил для понимания.
P.S. писал ночью с мобильного телефона, не проверял. Так что на свой страх и риск.

Comments

[Василий Банников]

это же не эквивалентная запись.

[Everything_is_not_so_bad]

Василий Банников, ага, обновил ответ. Если ничего дополнительно не упустил, то должно быть достаточно.

[john_bob]

еще немного укоротил:

domains&.select! do |h|
        res = true
        res &= h['date'].between?(params[:from_date], params[:to_date]) if date_range_present?
        res &= h['domainName'].match(params[:search_query]) if search_query_present?
        res
      end

[john_bob]

Василий Банников, почему не эквивалетная ?

[Everything_is_not_so_bad]

john_bob, кстати, если методы со знаком вопроса вычислительно дороги, то лучше вынести их перед началом проверок в булевы переменные.

[john_bob]

Василий Банников, понял, блок выполнялся даже когда не приходили параметры.
исправил, Спасибо!