レコード取得の記述をもっと綺麗に書きたい

Question

前提

フォームで入力された条件で、レコードの検索をかけるときにフォームの各項目に値があるかどうかでwhere文が変わります。
当初は、各項目を必須にすることはできないため、すべてのwhere文でワイルドカードであいまい検索を行っていました。
そうすると、処理が重くなってしまうため各項目の有無によってwhere文をわけるようにしました。

フォーム内容を必須にすることは考えていません。

実現したいこと

もう少し、簡潔に記述できるのではないかと思っているのですが、その方法があれば教えていただきたいです。

該当のソースコード

/model/history.rb

def self.find_records(params)
  logger.info(params)

  shop_name    = params[:shop].present? ? params[:shop] : ''
  start_time   = params[:start_time]
  end_time     = params[:end_time]
  first_name   = params[:first_name].present? ? params[:first_name] : ''
  last_name    = params[:last_name].present? ? params[:last_name] : ''
  status       = params[:status].present? ? params[:status] : ''

  production_flg = (params[:first_name].present? && params[:last_name].present?) == true ? true : false

  # shop_id、status、prodction
  if shop_id != '' && status != '' && production_flg
    record =
    self.joins(:shop, :production)
      .select('histories.*, productions.first_name, productions.last_name, shops.name')
      .where("first_name = ?",  "#{first_name}")
      .where("last_name = ?",   "#{last_name}")
      .where("name = ?",        "#{shop_name}")
      .where("status = ?",      "#{status}")
      .where("start_time BETWEEN '#{start_time}' and '#{end_time}'")
      .order("start_time")
  # shop_id, status
  elsif shop_id != '' && status != ''
    record =
    self.joins(:shop, :production)
      .select('histories.*, productions.first_name, productions.last_name, shops.name')
      .where("first_name LIKE ?", "%#{first_name}%")
      .where("last_name LIKE ?",  "%#{last_name}%")
      .where("name = ?",          "#{shop_name}")
      .where("status = ?",        "#{status}")
      .where("start_time BETWEEN '#{start_time}' and '#{end_time}'")
      .order("start_time")
  # shop_id
  elsif shop_id != ''
    record =
    self.joins(:shop, :production)
      .select('histories.*, productions.first_name, productions.last_name, shops.name')
      .where("first_name LIKE ?",  "%#{first_name}%")
      .where("last_name LIKE ?",   "%#{last_name}%")
      .where("name = ?",           "#{shop_name}")
      .where("status LIKE ?",      "%#{status}%")
      .where("start_time BETWEEN '#{start_time}' and '#{end_time}'")
      .order("start_time")
  else
    record =
    self.joins(:shop, :production)
      .select('histories.*, production.first_name, production.last_name, shops.name')
      .where("first_name LIKE ?",  "%#{first_name}%")
      .where("last_name LIKE ?",   "%#{last_name}%")
      .where("name LIKE ?",        "%#{shop_name}%")
      .where("status LIKE ?",      "%#{status}%")
      .where("start_time BETWEEN '#{start_time}' and '#{end_time}'")
      .order("start_time")
   end
   return record
  end

Answer 1

if文を使った検索条件の組み立て

検索条件が入力されたときだけ where を追加したい場合、if を使って以下のように書くことができます。

# 共通部分は先頭でまとめて指定
records = self.joins(:shop, :production)
              .select('histories.*, ... , shops.name')
              .order(:start_time)

# 送付されたパラメータに応じて where を追加
if xxxx
  records = records.where("xxxx = ?", xxxx)
end

if yyyy
  records = records.where("yyyy = ?", yyyy)
end

if zzzz
  records = records.where("zzzz = ?", zzzz)
end

return records 

これを使うと、パラメータが一部だけ指定された場合でも、すべて指定された場合でも、同じコードで処理できます。
production_flg や LIKE を一通りコードから除去できるはずです。

# ------------------------------------------
#  修正前
# ------------------------------------------

  production_flg = params[:first_name].present? && params[:last_name].present?

  if production_flg
    record = self.joins(...)
      .select(...)
      .where("first_name = ?",  "#{first_name}")
      .where("last_name = ?",  "#{last_name}")
      .order(...)
  else
    record = self.joins(...)
      .select(...)
      .where("first_name LIKE ?",  "%#{first_name}%")
      .where("last_name LIKE ?",  "%#{last_name}%")
      .order(...)
  end
  return record


# ------------------------------------------
#  修正後
# ------------------------------------------

  records = self.joins(...)
                .select(...)
                .order(...)
  
  if params[:first_name].present?
    records = records.where("first_name = ?", params[:first_name])
  end
  
  if params[:last_name].present?
    records = records.where("last_name = ?", params[:last_name])
  end

  return records

範囲指定系の検索条件の組み立て

現状のコードでは start_time と end_time を BETWEEN で処理されていますが、どちらか片方だけ指定された場合に備えて <= と >= に分解した方が良いです。

# ------------------------------------------
#  修正前
# ------------------------------------------

    record = ...
      .where("start_time BETWEEN '#{start_time}' and '#{end_time}'")


# ------------------------------------------
#  修正後
# ------------------------------------------

  if params[:start_time].present?
    records = records.where("start_time >= ?", params[:start_time])
  end
  
  if params[:end_time].present?
    records = records.where("start_time <= ?", params[:end_time])
  end

後置if

人により好みが分かれますが、 後置ifを使うと更にコードをコンパクトにできます。

-  if xxxx
-    records = records.where("xxxx = ?", xxxx)
-  end

+  records = records.where("xxxx = ?", xxxx) if xxxx

最終的なコード

def self.find_records(params)
  logger.info(params)

  records = self.joins(:shop, :production)
                .select('histories.*, productions.first_name, productions.last_name, shops.name')
                .order(:start_time)

  records = records.where("name = ?", params[:shop])              if params[:shop].present?
  records = records.where("start_time >= ?", params[:start_time]) if params[:start_time].present?
  records = records.where("start_time <= ?", params[:end_time])   if params[:end_time].present?
  records = records.where("first_name = ?", params[:first_name])  if params[:first_name].present?
  records = records.where("last_name = ?", params[:last_name])    if params[:last_name].present?
  records = records.where("status = ?", params[:status])          if params[:status].present?

  return records
end