Rails中更新has_many through关联数据时重复创建Instrument问题

核心问题 #

你的update方法中处理关联的代码存在严重错误：

params[:instrument_ids].each do |instrument_id|
  InstrumentSetup.update(instrument_id: instrument_id)
end

InstrumentSetup.update如果不指定查询条件，会批量更新所有InstrumentSetup记录。每次循环都会把数据库里所有InstrumentSetup的instrument_id改成当前循环的id，最终所有关联都会变成最后一个选中的instrument_id，导致前端显示重复的同一Instrument。

修复方案 #

1. 修正update方法逻辑 #

更新多对多关联时，正确的做法是先清除当前Setup已有的Instrument关联，再重新创建新的关联：

def update
  setup = Setup.find(params[:id])
  if setup && setup.user_id == current_user.id
    # 先清除现有关联
    setup.instrument_setups.destroy_all
    # 重新创建新的关联
    params[:instrument_ids].each do |instrument_id|
      setup.instrument_setups.create(instrument_id: instrument_id)
    end
    setup.update!(setup_params)
    render json: setup, status: :ok
  else
    render json: "Invalid Credentials", status: :unauthorized
  end
end

或者更简洁的方式，利用Rails的关联批量赋值（前提是setup_params正确配置）：

def update
  setup = Setup.find(params[:id])
  if setup && setup.user_id == current_user.id
    setup.update!(setup_params)
    render json: setup, status: :ok
  else
    render json: "Invalid Credentials", status: :unauthorized
  end
end

2. 修正setup_params参数 #

确保instrument_ids允许数组形式的参数，这样Rails可以自动处理多对多关联的更新：

def setup_params
  params.permit(:name, :description, :photo, :genre, instrument_ids: [])
end

修改后，调用setup.update!(setup_params)时，Rails会自动替换该Setup关联的所有Instrument，无需手动循环创建InstrumentSetup。

额外优化建议 #

• show方法建议包含关联的Instrument数据，方便前端展示：

  def show
    setup = Setup.find(params[:id])
    render json: setup, include: :instruments
  end
  

• create方法可以简化，利用关联批量赋值，和update保持一致：

  def create
    setup = current_user.setups.create!(setup_params)
    render json: setup, status: :created
  end
  

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