Rails中如何编写模型方法实现带参查询？代码优化求助

1. 代码组织方式的合理性 #

两种思路都是合理的，各有适用场景：

• 最初的链式调用辅助方法是Rails社区推荐的常规写法，每个类方法（如around_date）只负责单一查询逻辑，直观易懂，也方便单独测试。控制器里的链式调用代码虽然看起来稍长，但逻辑清晰，完全符合ActiveRecord的设计理念。
• 改成统一的query_search入口方法是更优的进阶方向，尤其当后续查询条件增多时，能把所有查询逻辑封装在模型内部，让控制器代码更简洁，完美契合"胖模型瘦控制器"的原则。你之前遇到的问题不是思路错误，而是参数处理和方法实现的细节没做好。

2. 修复query_search方法的方案 #

核心问题是表单提交的空值是**空字符串""**而非nil，导致模型里的条件判断失效。我们需要先处理参数，再在query_search里链式拼接有效查询：

第一步：控制器中处理空参数 #

在flights#index里，先把空字符串转成nil，再传入模型方法：

def index
  # 用presence把空字符串/空白字符串转为nil
  search_params = params.permit(:departure_airport_id, :arrival_airport_id, :date)
                        .transform_values { |val| val.presence }
  @flights = Flight.query_search(search_params)
end

第二步：模型中实现query_search方法 #

把辅助方法设为私有，在query_search里只对有有效值的参数应用对应查询逻辑：

class Flight < ApplicationRecord
  def self.query_search(params)
    # 初始化为所有记录的查询关系
    results = all

    # 逐个处理参数，拼接查询条件
    results = results.where(departure_airport_id: params[:departure_airport_id]) if params[:departure_airport_id].present?
    results = results.where(arrival_airport_id: params[:arrival_airport_id]) if params[:arrival_airport_id].present?
    results = results.around_date(params[:date]) if params[:date].present?

    results
  end

  private

  # 保留你的日期查询逻辑
  def self.around_date(date)
    where(date: date.beginning_of_day..date.end_of_day)
  end
end

更优雅的写法（可选） #

如果后续要加更多查询条件，可以用迭代简化代码：

def self.query_search(params)
  params.each_with_object(all) do |(key, value), results|
    case key
    when :date
      results = results.around_date(value) if value.present?
    when :departure_airport_id, :arrival_airport_id
      results = results.where(key => value) if value.present?
      # 后续加其他参数时，在这里加分支即可
    end
  end
end

这样修改后，query_search就能正确执行所有辅助方法的逻辑，只有当参数有有效值时才会拼接对应的查询条件，空参数会被直接忽略。

内容的提问来源于stack exchange，提问作者blainefreestone