如何在Sinatra中检测路由是否含参数？日期参数处理报错求助

你的问题出在对路由参数的判断逻辑不正确，导致即使路由没有year和month参数时，创建Date对象的代码依然会执行，进而传入无效值触发错误。以下是具体问题分析和修复方案：

问题分析 #

1. 第一种写法的问题：params[:year].any?会在params[:year]为nil时抛出NoMethodError（因为nil没有any?方法），但如果参数存在但为空字符串（比如/foo/bar//12），any?会返回false，这部分是对的，但nil的情况会直接报错。
2. 第二种写法的问题：defined?(params[:year])永远返回true，因为即使参数不存在，params[:year]只是返回nil，表达式本身是合法定义的。所以这个判断完全无效，导致代码始终执行，传入nil.to_i得到的0来创建Date，触发Date::Error。

修复方案 #

需要正确判断参数是否存在且不为空，同时验证参数转换后的数值是否合法，最后加入异常处理避免意外错误：

方案1（使用ActiveSupport的present?方法，适合已引入ActiveSupport的项目） #

before do
  if params[:year].present? && params[:month].present?
    begin
      year = params[:year].to_i
      month = params[:month].to_i
      @requested_date = Date.new(year, month, 1)
    rescue Date::Error
      # 处理无效日期的情况，比如设为nil或跳转错误页面
      @requested_date = nil
    end
  end
end

方案2（不依赖ActiveSupport） #

before do
  year_param = params[:year]
  month_param = params[:month]
  
  # 先判断参数存在且不为空字符串
  if year_param && !year_param.empty? && month_param && !month_param.empty?
    year = year_param.to_i
    month = month_param.to_i
    
    # 提前验证年份和月份的合法性
    if year > 0 && month.between?(1, 12)
      @requested_date = Date.new(year, month, 1)
    else
      @requested_date = nil
    end
  end
end

关键要点 #

• 用params[:year] && !params[:year].empty?替代错误的判断逻辑，确保只有参数存在且非空时才执行后续代码。
• 对转换后的年份和月份做合法性检查（年份>0，月份在1-12之间），避免无效数值传入Date构造方法。
• 加入rescue块捕获Date::Error，处理极端情况（比如2月30日这类无效日期）。

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