Rails中link_to调用错误控制器动作问题求助

你的问题核心是Rails路由的匹配优先级导致请求被错误拦截了，咱们一步步拆解解决：

1. 路由顺序是问题根源 #

在你的routes.rb里，Rails是按从上到下的顺序匹配路由的：

get '/weeks/:id', to: 'weeks#setup', as: 'weeks_setup'
get 'weeks/draw_day'

当你点击link_to生成的/weeks/draw_day链接时，Rails会先匹配到/weeks/:id这个动态路由——它把draw_day当成了:id参数的值，直接把请求转发到了setup方法，完全跳过了draw_day的路由规则。这就是为什么你在draw_day里加的byebug从来没触发，反而setup因为拿到非法的id参数报错。

2. 修复步骤 #

第一步：调整路由顺序 #

把明确的静态路由（比如weeks/draw_day）放在动态路由（/weeks/:id）的前面，让Rails先匹配到精准的路径：

########### Week actions #############
get 'weeks/schedule'
get 'weeks/draw_day'  # 把这个移到动态路由前面
get '/weeks/:id', to: 'weeks#setup', as: 'weeks_setup'

resources :weeks, only: [:schedule, ] do
  get :schedule, on: :collection
end

第二步：验证路由匹配结果 #

在终端执行rails routes命令，检查输出里的weeks_draw_day路由，确认它现在指向正确的控制器动作：

weeks_draw_day GET    /weeks/draw_day(.:format)    weeks#draw_day

第三步：确认前端链接路径 #

在浏览器里右键检查下拉菜单的链接，查看它的href属性，确认是/weeks/draw_day?test=xxx的格式，而不是被错误解析成/weeks/draw_day（当作id的情况）。

3. 额外优化建议 #

• 如果需要给draw_day传递参数（比如你的:test => d），可以把路由改成更RESTful的形式：

  get 'weeks/draw_day/:test', to: 'weeks#draw_day', as: 'weeks_draw_day'
  

  这样生成的路径会是/weeks/draw_day/2024-05-20，比查询参数更规范，当然具体看你的业务需求。
• 在setup方法里加参数校验，避免非法id导致报错：

  def setup
    @w = Week.find_by_id(params[:id])
    unless @w
      redirect_to root_path, alert: "无效的周ID"
      return
    end
    # 后续业务逻辑
  end
  

调整完路由顺序后，你的link_to应该就能正确触发draw_day方法，byebug也会正常执行了~

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