Rails自定义验证限制了has_many的数量：允许通过关联？

Question

我有一个带有“产品变体”表单的 Rails 项目。产品变体模型称为Variant，在Variant 表单上，用户应该能够为每个可用选项选择一个选项。例如，一件 T 恤可能有一个名为“尺寸”的“选项”，其中“选择”为小、中或大，另一个“选项”名为“颜色”，“选择”为红色、绿色、蓝色。因此，创建的Variant 是一个独特的 SKU，例如“T 恤 — 尺寸：小号，颜色：绿色”。或者，如果产品有 3 个选项而不是 2 个选项，则该变体将需要每个选项 3 个选项，例如“吉他背带 - 尺寸：长款，面料颜色：红色，皮革颜色：棕色”。

我不知道如何编写只允许用户为每个选项保存一个选项的自定义验证。每个选项应该只为每个变体选择一个选项。这是一个插图。

这是我的模型与相关的关联...

models/variant.rb

class Variant < ApplicationRecord    
  has_many :selections
  has_many :choices, through: :selections

  validate :one_choice_per_option

  private
    def one_choice_per_option
      # can't figure out how to do this custom validation here
    end

end

models/choice.rb

class Choice < ApplicationRecord
  has_many :variants, through: :selections

  belongs_to :option
end

models/selection.rb

class Selection < ApplicationRecord
  belongs_to :choice
  belongs_to :variant
end

models/option.rb

class Option < ApplicationRecord
  has_many :choices, dependent: :destroy

  accepts_nested_attributes_for :choices, allow_destroy: true
end

我设法做的最好的事情是在models/variant.rb 中进行此自定义验证

def one_choice_per_option
  self.product.options.each do |option|
    if option.choices.count > 1
      errors.add(:choice, 'Error: select one choice for each option')
    end
  end
end

但这只允许一个Choice 总数通过变体形式。我想要做的是让每组选项都有一个选择。

我知道这可以在 UI 中使用 Javascript 来完成，但这对于保持数据库清洁和防止用户错误至关重要，所以我认为它应该是模型级别的 Rails 验证。

进行此类自定义验证的“Railsy”方式是什么？我应该尝试对Selection 模型进行自定义验证吗？如果有，怎么做？

---

更新

基于 cmets 中的讨论。看来我需要结合Active Record querying 来完成这项工作。 @sevensidemarble 下面的“EDIT 2”更接近，但这给了我这个错误：Type Error compared with non class/module

如果我将错误的行为保存到数据库中，然后在控制台中调用Variant.last.choices，那感觉就像我越来越接近了：

所以本质上，如果有多个Selection 具有相同的option_id，我需要做的是不允许保存Variant 表单。除非option_id 对关联的Variant 是唯一的，否则不应保存选择。

我正在尝试做这样的事情：

validate :selections_must_have_unique_option

  private

    def selections_must_have_unique_option
      unless self.choices.distinct(:option_id)
        errors.add(:options, 'can only have one choice per option')
      end
    end

但该代码不起作用。它只是保存表单，就好像验证不存在一样。

Answer 1

这类似于唯一性约束，但 Rails 的内置 validates_uniqueness_of 无法处理此问题：所需的验证在每个 Selection 对象上，但它是由选项决定的，而 selections 表没有 有一个option_id 列来约束。

您也不能在数据库中轻松做到这一点，因为唯一索引不会跨越表边界。

我建议您让每个 Selection 对象查找冲突的兄弟，并在结果上使用 absence validation。像这样的：

class Variant < ApplicationRecord    
  has_many :selections
  has_many :choices, through: :selections
end

class Selection < ApplicationRecord
  belongs_to :choice
  belongs_to :variant

  validates_absence_of :conflicting_selection

  protected
    def option
      choice.option
    end

    def conflicting_selection
      variant.selections.excluding(self).detect { |other| option == other.option }
    end
end

鹰眼会注意到我使用了数组方法而不是 ActiveRecord 查询。这不是避免数据库往返的俗气的噱头。它确保验证在未保存的选择以及那些持久的选择上正常工作，这对于表单处理可能是必不可少的。我通常想写成has_one :option, through: :choice 的Selection#option 方法与此类似。

如果你需要的话，带有单元测试的示例要点here。

Answer 2

这样的事情应该可以工作：

def one_choice_per_option
      errors.add(:choice, "can't be more then one per selection") if selections.any { |selection| selection.choices.count > 1 }
end

或者，我怀疑你也可以只选择has_one :choice，然后你的生活可能会更简单。

---

编辑：

根据 cmets 的要求，我认为会按照要求进行更新：

def one_choice_per_option
      errors.add(:choice, "can't be more then one per selection") if selections.joins(:choice).where(selections: :choice).count > 3
end

我认为这会得到你想要的。如果不是，我们只需稍微弄乱joins 和where 部分，请告诉我。

编辑 2：

---

好的，经过再次讨论，我对要求的内容更加清楚了。抱歉，花了一些时间，如果不在您自己的应用程序中看到它，就很难理解架构。

这是我认为适用于指定的内容。我是根据Product 的上下文在这里写的，但它也可以在Option 上稍作改动。

def one_choice_per_option
  errors.add(:options, "can't be more then one per selection") if (options.joins(:choices).group("options.id").having("COUNT(1) > 1") > 1)
end

我认为这应该可行，如果不是，它真的很接近您的需要。这个SQL是基本思想。有时，在这些查询上致电 .to_sql 会有所帮助，以查看确切的活动记录抽出什么。

---

编辑 3：

我正在考虑您在 Variant 上的原始代码，并想出了这个：

class Variant < ApplicationRecord
  has_many :selections
  has_many :choices, through: :selections

  validate :one_choice_per_option

  private
  def one_choice_per_option
    if (choices.joins(:option).group("options.id").having("COUNT(1) > 1") > 1)
      errors.add(:choices, "can't be more then one per selection") 
    end
  end
end