Rails中link_to指向错误ID及Share布尔属性更新错误问题排查

咱来一步步拆解你遇到的这两个问题：

问题1：Rails的link_to指向错误ID的常见原因 #

通常出现这种问题，大概率是以下几个场景之一：

• 参数传递或实例变量搞错了：比如你本来想链接到当前用户的Share对象，结果控制器里把@share赋值成了Share.last或者其他无关对象，link_to自然会用错误的ID。或者在link_to里写错了参数，比如把@share.id写成了@item.id。
• link_to的参数顺序搞混了：Rails里link_to的正确语法是link_to "显示文本", 目标对象/路径助手, 选项哈希。如果把路径助手和选项搞反，比如写成link_to "切换", {class: "btn"}, item_share_path(@item, @share)，就会生成错误的URL。
• 关联对象的查找逻辑没限定用户：比如你取@item.shares.first，但这个Share可能属于其他用户，导致link_to用了别人的ID。
• 路由定义或命名冲突：比如你有两个同名的路由助手，或者路由里的参数名重复（比如同时用了:id和:item_id但没区分），生成URL时就会错用ID值。

问题2：修改Share的active属性时更新错误记录 #

根据你的描述，访问ID为7的Item时正常，访问ID为3的Item时出错，核心问题肯定是你查找Share的逻辑没有限定当前用户。

举个例子，你的控制器代码可能是这样写的（错误版）：

def update
  @item = Item.find(params[:item_id])
  # 只按item_id找Share，会取到第一个匹配的，不管是谁的
  @share = @item.shares.find_by(item_id: params[:item_id])
  @share.update(active: !@share.active)
  redirect_to @item
end

同一个Item可能被多个用户分享，所以会存在多个item_id为3的Share记录。当你访问ID为3的Item时，代码取到的是数据库里第一个item_id=3的Share（可能属于其他用户），自然就更新了错误的记录。

解决方法： #

在查找Share时，一定要加上当前用户的条件，确保只操作当前用户拥有的那条记录：

def update
  @item = Item.find(params[:item_id])
  # 限定当前用户的Share
  @share = @item.shares.find_by(user_id: current_user.id)
  
  if @share
    @share.update(active: !@share.active)
  else
    # 可以加个提示，比如flash[:alert] = "你没有该Item的分享权限"
  end
  
  redirect_to @item
end

另外，建议在Share模型里给item_id和user_id加个唯一索引，避免同一个用户重复分享同一个Item：

# db/migrate/xxxx_add_unique_index_to_shares.rb
add_index :shares, [:item_id, :user_id], unique: true

这样不仅能保证数据唯一性，还能让查找更高效。

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