Ruby on Rails页面编号验证绕过与PostgreSQL约束冲突处理求助

一、提前避免数据库错误的方案 #

1. 在Book模型添加自定义验证，检查所有待生效页面编号的唯一性 #

Rails默认的validates_uniqueness_of只会查询数据库，不会检查本次提交中嵌套属性里的重复（比如新增多个同编号页面），也不会考虑现有页面的修改。我们可以在Book模型中添加自定义验证，收集所有即将生效的页面编号（包括现有未删除的、待更新的、待新增的），统一检查是否重复：

class Book < ActiveRecord::Base
  has_many :pages
  accepts_nested_attributes_for :pages

  validate :check_page_numbers_uniqueness

  private

  def check_page_numbers_uniqueness
    # 收集当前未标记删除的页面编号
    existing_valid_numbers = pages.reject(&:marked_for_destruction?).pluck(:page_number)
    
    # 收集嵌套属性中待处理的页面编号（跳过标记删除的）
    nested_valid_numbers = pages_attributes&.values&.map do |attrs|
      next if attrs['_destroy'] == '1'
      attrs['page_number'].to_i if attrs['page_number'].present?
    end&.compact || []

    # 合并所有即将生效的编号，检查重复
    all_numbers = existing_valid_numbers + nested_valid_numbers
    duplicate_numbers = all_numbers.group_by { |n| n }.select { |_, count| count.size > 1 }.keys

    if duplicate_numbers.present?
      errors.add(:base, "页面编号 #{duplicate_numbers.join(', ')} 重复，请修改")
    end
  end
end

2. 调整嵌套属性的处理顺序 #

Rails默认处理嵌套属性的顺序是新增→更新→删除，这会导致场景2（修改现有页面编号+新增同编号页面）先触发新增操作，和数据库中未修改的旧编号冲突。我们可以重写嵌套属性赋值方法，改成删除→更新→新增的顺序：

class Book < ActiveRecord::Base
  has_many :pages
  accepts_nested_attributes_for :pages

  def pages_attributes=(attributes)
    return super if attributes.blank?

    # 拆分嵌套属性为三类：待删除、待更新、待新增
    delete_attrs = attributes.values.select { |a| a['_destroy'] == '1' }
    update_attrs = attributes.values.select { |a| a['id'].present? && a['_destroy'] != '1' }
    create_attrs = attributes.values.select { |a| a['id'].blank? && a['_destroy'] != '1' }

    # 先处理删除
    delete_attrs.each { |attrs| pages.find(attrs['id']).mark_for_destruction }
    # 再处理更新
    update_attrs.each { |attrs| pages.find(attrs['id']).assign_attributes(attrs) }
    # 最后处理新增
    create_attrs.each { |attrs| pages.build(attrs) }
  end
end

调整顺序后，场景2中会先把原有页面的编号修改完成，再新增新页面，避免数据库冲突。

二、捕获数据库错误并友好提示 #

即使做了上述处理，仍可能存在并发场景（比如两个请求同时修改同一本书的页面）触发数据库唯一索引错误。此时需要在控制器中捕获错误，转换成用户友好的提示：

def update
  @book = Book.find(params[:id])
  if @book.update_attributes(params[:book])
    flash[:notice] = '书籍修改成功！'
    redirect_to @book
  end
rescue ActiveRecord::RecordNotUnique, PG::UniqueViolation => e
  # 匹配页面编号重复的错误信息，生成精准提示
  if e.message =~ /page_number.*book_id/
    # 尝试提取重复的页面编号（根据PostgreSQL错误信息格式调整正则）
    if match = e.message.match(/key \(page_number, book_id\)=\((\d+), \d+\)/)
      @book.errors.add(:base, "页面编号 #{match[1]} 已存在，请修改")
    else
      @book.errors.add(:base, '存在重复的页面编号，请检查后重新提交')
    end
  else
    @book.errors.add(:base, '更新失败，请稍后重试')
  end
  render :edit
end

注意事项 #

• 数据库层面的唯一索引必须保留，它是最后一道防线，防止并发场景下的数据不一致。
• 自定义验证中要注意处理marked_for_destruction?的页面，避免把即将删除的页面编号算入重复检查。

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