Ruby按顺序将货币数组值分配至Country模型currency字段问题

你的需求是按时间顺序将货币数组的元素依次赋值给每条Country记录，但现有代码存在逻辑问题，以下是修正方案：

现有代码的问题 #

• 第一个循环中，Country.update(currency: currencies[i])是调用类方法update，会尝试更新所有Country记录，而非当前遍历的单个记录，应该使用实例方法country.update。同时Country.all默认顺序不保证是时间顺序，需要显式排序。
• 第二个循环逻辑完全错误：它会把数组中的每个货币依次赋值给所有Country记录，最终所有记录的currency都会被覆盖为数组最后一个元素。

正确实现方案 #

基础版（逐条更新，适合记录量不大的场景） #

currencies = [...] # 你的货币数组

# 按创建时间升序获取记录，确保顺序符合预期
Country.order(created_at: :asc).each_with_index do |country, index|
  # 避免数组元素不足时出现索引越界错误
  if index < currencies.length
    country.update(currency: currencies[index])
  else
    # 可选：数组元素不足时的默认处理，比如设为nil
    country.update(currency: nil)
  end
end

批量更新版（性能更优，适合大量记录场景） #

如果Country记录数量较多，逐条更新会产生大量SQL请求，推荐用批量更新减少数据库交互：

currencies = [...]

# 生成包含国家ID和对应货币的哈希数组
country_updates = Country.order(created_at: :asc).pluck(:id).zip(currencies).map do |id, currency|
  { id: id, currency: currency }
end

# 批量更新，仅需1条SQL语句
Country.update(country_updates.map { |item| item[:id] }, country_updates)

关键说明 #

• order(created_at: :asc)确保记录按创建时间从早到晚排序，严格匹配你要求的"时间顺序"。
• each_with_index自动维护索引，无需手动管理i变量，代码更简洁可靠。
• 加入索引越界判断，避免当货币数组长度小于Country记录数时抛出错误。

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