如何确保多对多双方的关系是唯一的

Question

我有一个充当多对多关系的模型。类名是RelatedDocument，这是不言自明的，我基本上用它来关联Document类实例。

我遇到了验证问题，例如我在 RelatedDocument 类中有这个问题：

validates :document, presence: true, uniqueness: { scope: :related_document }
validates :related_document, presence: true

这项工作我无法创建重复的document_id/related_document_id 行。但是，如果我想从关系的另一端使其独一无二并将验证更改为：

validates :document, presence: true, uniqueness: { scope: :related_document }
validates :related_document, presence: true, uniqueness: { scope: :document }

这在另一边是不一样的。当我注意到这一点时，我正在编写 rspec 测试。如何编写验证或自定义验证方法来防止保存相同的 id 组合，无论它们来自哪一侧？

更新

根据评论部分的第一条评论，第一次唯一性验证将照顾双方，我说这不仅仅是因为我的 rspec 测试失败，它们是：

describe 'relation uniqueness' do
    let!(:base_doc) { create(:document) }
    let!(:another_doc) { create(:document) }
    let!(:related_document) { described_class.create(document: another_doc, related_document: base_doc) }

    it 'raises ActiveRecord::RecordInvalid, not allowing duplicate relation links' do
      expect { described_class.create!(document: another_doc, related_document: base_doc) }
        .to raise_error(ActiveRecord::RecordInvalid)
    end

    it 'raises ActiveRecord::RecordInvalid, not allowing duplicate relation links' do
      expect { described_class.create!(document: base_doc, related_document: another_doc) }
        .to raise_error(ActiveRecord::RecordInvalid)
    end
  end

第二次测试失败。

Answer 1

如果您想考虑替代已提供的自定义验证，您可以在每次添加新记录时创建互惠关系并使用现有验证。

例如，当我说“文档 A 与文档 B 相关”时，我也会插入一条记录，说明“文档 B 与文档 A 相关”。您可以保持验证简单，并且可以在将来更轻松地实现逻辑，以便在某些情况下不想要互惠关系（也许只有当文档来自不同的文档时，关系才会互惠）作者）。

以下是您将实现的模型回调类型的未经测试的示例：

create_table :documents do |t|
    t.string :title, null: false
end

create_table :related_documents do |t|
    t.integer :source_document_id, null: false
    t.integer :related_document_id, null: false
end

add_index :related_documents, [:source_document_id, :related_document_id], unique: true
add_foreign_key :related_documents, :documents, column: :source_document_id
add_foreign_key :related_documents, :documents, column: :related_document_id

class Document < ActiveRecord::Base
    # We have many document relationships where this document is the source document
    has_many :related_documents, foreign_key: :source_document_id

    validates :title,
        presence: true
end

class RelatedDocument < ActiveRecord::Base
    after_create :add_reciprocal_relationship
    after_destroy :remove_reciprocal_relationship

    belongs_to :source_document, class_name: Document
    belongs_to :related_document, class_name: Document

    validates :source_document,
        presence: true

    validates :related_document,
        presence: true,
        uniqueness: {
            scope: :source_document
        }

    private

    # Creates a reciprocal relationship
    def add_reciprocal_relationship
        RelatedDocument.find_or_create_by(
            related_document: self.source_document,
            source_document: self.related_document
        )
    end

    # Safely removes a reciprocal relationship if it exists
    def remove_reciprocal_relationship
        RelatedDocument.find_by(
            related_document: self.source_document,
            source_document: self.related_document
        )&.destroy
    end
end

Answer 2

自定义验证器应该在这里工作

validate :unique_document_pair

def unique_document_pair
  if RelatedDocument.exists?(:document => [self.document,self.related_document], :related_document => [self.document, self.related_document])
      errors.add :base, "error"
  end
end