Как я могу упростить условия, когда у меня много параметров в моем поиске?

@search = Policy.scoped
@search = @search.where('num_policy LIKE ?', params[:num_policy]) if params[:num_policy].present?
@search = @search.where('type_id LIKE ?', params[:type_id]) if params[:type_id].present?
@search = @search.where('doc LIKE ?', params[:doc]) if params[:doc].present?
@search = @search.where('name LIKE ?', params[:name]) if params[:name].present?
@policies = @search.paginate(page: params[:page], per_page: 2)

Кстати: проблема в том, что вы создаете @search внутри операторов if, поэтому, если параметры не существуют, @search не существует.

Это версия рельсов 2:

conditions_str = []
conditions_values = {}
['num_policy', 'type_id', 'doc', 'name'].each do |cond|
  if params[cond.to_sym].present?
    conditions_str << "#{cond} LIKE :#{cond}"
    conditions_values[cond.to_sym] = params[cond.to_sym]
  end
end
@policies = Policy.find(:all, :conditions => [conditions_str.join(" AND "), conditions_values])

Вам нужно очистить свои параметры и извлечь хэш только с вашими критериями поиска, тогда вы можете написать простые для чтения условия. Пример:

# define what are the search keys that can be found in params[]
search_keys = [:a, :b, :c, :d, :e, :f]

# extract hash values that concern only the search criteria
search_params = params.select { |k,v| search_keys.include? k and ! v.nil? and ! v.empty? }

# Write query according to search criteria presents
case search_params.keys.sort
  when [:a]
    # Write the query when only :a search criteria is present
    Something.where("blahblah = ?", params[:a])
  when [:a, :c]
    # Write the query when only :a and :c search criteria are present
    Something.where("a_field = ? AND stuff = ?", params[:a], params[:c])
  when ...
    ...
end

Я не знаю, правильно ли я понял ваш вопрос, но я все равно опубликую свой ответ.

Я бы сделал функцию:

def simplified_conditions conds, prms, perform_and_on_hashes = false
  conds.each do |h|
    h.each do |method_name, symbols_array|
      symbols_array.each do |symb|
        return false unless prms[symb].send method_name
      end
    end
    return true unless perform_and_on_hashes
  end
  return true
end

И передайте ему структуру условий, которая выглядит так:

[
  { 'boolean_function_name' => [:array,:of,:symbols],
    'another_function_name' => [:array,:of,:symbols],
    'another_function_name' => [:array,:of,:symbols]
  },
  { 'boolean_function_name' => [:array,:of,:symbols],
    'another_function_name' => [:array,:of,:symbols],
    'another_function_name' => [:array,:of,:symbols]
  },
  { 'boolean_function_name' => [:array,:of,:symbols],
    'another_function_name' => [:array,:of,:symbols],
    'another_function_name' => [:array,:of,:symbols]
  }
]

Итак, в ваших примерных условиях массив будет выглядеть так:

conds = [
  { 'present?' => [:a], 'nil?' => [:b] },
  { 'present?' => [:a], 'nil?' => [:c] },
  { 'present?' => [:a], 'nil?' => [:e] },
  { 'present?' => [:b], 'nil?' => [:c] },
  { 'present?' => [:b], 'nil?' => [:d] }
]

# And then to call it:
final_boolean_result = simplified_conditions(conds, params, false)

Это вернет true, если любая из строк в вашем примере верна. Если вы хотите, чтобы ВСЕ они были правдой, просто измените последний параметр на true.

Если я неправильно понял ваш вопрос, пожалуйста, оставьте комментарий.