Rails持久化时忽略验证：City模型重复创建问题排查

看起来你在处理City模型重复创建、验证逻辑没生效的问题，我来帮你拆解下可能的原因和修复方案：

1. 先检查existing_city的查找逻辑 #

这是重复记录的重灾区！如果你的查找只匹配了城市名，没关联州名，那不同州的同名城市（甚至同一州因空格差异的同名城市）都会被当成新记录创建。比如：

• 错误的查找：existing_city = City.find_by(name: city_name)
• 正确的查找应该同时匹配城市和州，还要处理字符串前后空格：

existing_city = City.find_by(name: city_name.strip, state: state_name.strip)

2. 验证逻辑未生效的排查点 #

首先确认City模型的验证是否正确定义，尤其是唯一性验证（避免同州同名城市重复）：

class City < ApplicationRecord
  validates :name, :state, presence: true
  validates :name, uniqueness: { scope: :state, message: "该城市在当前州已存在" }
end

然后检查你在assign_city里创建City的方式：

• 如果用了City.new(...)然后save(validate: false)，会直接跳过验证，绝对会导致无效记录！
• 如果用City.create(...)，它会执行验证，但创建失败只会返回false而不抛出异常——要是你没检查返回值就直接赋值给Family，就会出现“看似忽略验证”的情况。

3. 优化后的assign_city方法实现 #

推荐用find_or_create_by简化逻辑，同时确保验证生效、避免重复：

def assign_city(city_name, state_name)
  raise(GeoException, 'Both city and state names must be present') unless city_name.present? && state_name.present?

  # 标准化输入，消除空格导致的重复
  normalized_city = city_name.strip
  normalized_state = state_name.strip

  # 查找或创建，创建时自动执行模型验证
  city = City.find_or_create_by(name: normalized_city, state: normalized_state)

  # 检查是否成功，验证失败时抛出异常提示
  unless city.persisted?
    error_msg = city.errors.full_messages.join(', ')
    raise(GeoException, "无法创建/获取城市: #{error_msg}")
  end

  self.city = city
end

4. 数据库层面加保险 #

即使模型验证没问题，并发场景下还是可能出现重复记录（比如两个请求同时创建同一个城市）。所以最好给City表加联合唯一索引：

# 生成迁移文件后，编辑内容如下
class AddUniqueIndexToCities < ActiveRecord::Migration[7.0]
  def change
    add_index :cities, [:name, :state], unique: true
  end
end

这样数据库会直接阻止重复数据插入，从底层保证数据一致性。

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