Rails中最小数量大于最大数量的范围验证问题

一、验证位置选择 #

优先在控制器层或者表单对象做输入校验（前端也可加一层即时提示提升体验，但后端必须做校验，防止恶意请求），同时可以在数据库查询层做兜底处理。

二、控制器层直接校验（快速实现） #

在处理筛选请求的控制器action里，先获取参数并做校验，示例代码：

def index
  min_qty = params[:total_quantity_min].to_i
  max_qty = params[:total_quantity_max].to_i

  # 检查最小数量是否大于最大数量
  if params[:total_quantity_min].present? && params[:total_quantity_max].present? && min_qty > max_qty
    flash[:alert] = "最小数量不能大于最大数量"
    # 跳回原页面或列表页
    return redirect_to request.referer || your_resource_index_path
  end

  # 应用筛选条件
  @resources = YourModel.filter_total_quantity_min(params)
                        .filter_total_quantity_max(params)
                        .all
end

三、用表单对象做校验（规范写法，适合复杂筛选） #

如果筛选条件较多，建议用表单对象做集中校验，符合Rails关注点分离原则：

1. 创建表单对象文件：

# app/forms/search_form.rb
class SearchForm
  include ActiveModel::Model

  attr_accessor :total_quantity_min, :total_quantity_max

  # 先校验参数格式
  validates :total_quantity_min, numericality: { only_integer: true, allow_nil: true }
  validates :total_quantity_max, numericality: { only_integer: true, allow_nil: true }
  # 自定义范围校验
  validate :min_quantity_not_greater_than_max

  private

  def min_quantity_not_greater_than_max
    return if total_quantity_min.blank? || total_quantity_max.blank?

    if total_quantity_min.to_i > total_quantity_max.to_i
      errors.add(:base, "最小数量不能大于最大数量")
    end
  end
end

2. 控制器中使用表单对象：

def index
  @search_form = SearchForm.new(search_params)

  if @search_form.invalid?
    flash[:alert] = @search_form.errors.full_messages.join(", ")
    # 校验失败时返回空列表或原列表
    @resources = YourModel.all
    return render :index
  end

  # 应用筛选条件
  @resources = YourModel.filter_total_quantity_min(@search_form.attributes)
                        .filter_total_quantity_max(@search_form.attributes)
                        .all
end

private

def search_params
  params.permit(:total_quantity_min, :total_quantity_max)
end

四、修正你的Scope代码 #

注意你提供的scope里有拼写错误：Onde应该改成Rails的查询方法where，修正后代码：

scope :filter_total_quantity_min, -> (params) {
  params[:total_quantity_min].present? ?
    where(
      "total_quantity >= ?",
      params[:total_quantity_min]
    )
  :
    all
}

scope :filter_total_quantity_max, -> (params) {
  params[:total_quantity_max].present? ?
    where(
      "total_quantity <= ?",
      params[:total_quantity_max]
    )
  :
    all
}

五、数据库层兜底（可选） #

可以把两个scope合并成一个，在查询时直接处理范围逻辑，避免无效查询：

scope :filter_total_quantity_range, -> (min, max) {
  if min.present? && max.present?
    min_val = min.to_i
    max_val = max.to_i
    # 如果min>max，直接返回空集合
    min_val <= max_val ? where("total_quantity BETWEEN ? AND ?", min_val, max_val) : none
  elsif min.present?
    where("total_quantity >= ?", min.to_i)
  elsif max.present?
    where("total_quantity <= ?", max.to_i)
  else
    all
  end
}

调用时直接传min和max参数即可：

@resources = YourModel.filter_total_quantity_range(min_qty, max_qty).all

内容的提问来源于stack exchange，提问作者Roberto da Silva