Rails自定义POST成员路由下如何正确传递Variant参数解决ID复用问题

当前实现的核心漏洞是参数逻辑完全复用，没有给变体ID留独立的传参通道：

• select_variant作为questions资源的member路由，路径固定为/questions/:id/select_variant，这里的:id占位符是Rails框架约定好的、专门存储当前问题ID的位置，本身没有额外空间存储变体ID
• 原视图代码把变体对象直接作为位置参数传给路径Helper时，Rails会自动把ActiveRecord对象的ID替换掉路径里原本的:id值，导致原本存问题ID的位置被改成了变体ID。最终Controller里两个查询都从params[:id]取值，拿到的永远是同一个值——你看到的SQL日志里两个ID都是1，纯粹是测试数据巧合：刚好第一条问题的ID和第一条变体的ID都是1，根本不是代码正确传递了参数。一旦换到ID不同的测试数据，就会出现找错记录、甚至直接404的问题。

1. 调整Controller参数取值逻辑 #

不要复用:id参数查询变体，改为用独立的参数名接收变体ID，从根源上避免参数混淆：

def select_variant
  # :id 是member路由默认绑定的问题ID，取值逻辑无需修改
  @question = Question.find(params[:id])
  # 用独立的variant_id参数查询所选变体，不与问题ID的参数复用
  variant = Variant.find(params[:variant_id])
  Answer.create!(question: @question, variant: variant)
  redirect_to question_path(@question), notice: "Question was successfully answered."
end

2. 修正视图层链接的参数传递方式 #

传递参数时明确指定variant_id的键名，不要靠位置参数隐式传递对象，避免触发Rails的自动参数覆盖逻辑：

<% @question.variants.each do |variant| %>
  <%= link_to variant.body, 
      select_variant_question_path(@question, variant_id: variant.id), 
      method: :post %>
<% end %>

3. 路由无需做核心结构修改 #

原有路由定义完全可以正常工作——Rails路由只会匹配路径部分的占位符，额外通过query string传递的variant_id参数会自动进入params哈希，不需要在路由块中声明额外的占位符。如果希望路径参数的绑定逻辑更明确，原有的路由定义可以保留不变：

resources :questions do
  member do
    post :select_variant, 
      to: "questions#select_variant"
  end
end

注意：实际生产环境中，建议在创建Answer前加一层校验，确认拿到的variant是当前问题下关联的变体，避免用户手动篡改请求参数，传入不属于当前问题的变体ID。

修改后点击对应变体链接，生成的请求路径会类似/questions/1/select_variant?variant_id=3，参数传递逻辑完全隔离：

• 路径中的1对应问题ID，绑定到params[:id]
• query参数里的3对应所选变体ID，绑定到params[:variant_id]
  最终创建Answer记录时，两个关联记录的ID都会和用户实际选择的内容对应，不会再出现ID混淆的问题。

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