ruby-on-rails - rails : Tight coupling between deeply nested models: how to uncouple them for tests and DB efficiency?

Question

我有以下模型和关联:

SuccessCriterion
  has_many :requirements
  has_many :findings, through: :requirements

Requirement
  belongs_to :success_criterion
  has_many :findings

Finding
  belongs_to :requirement
  has_one :success_criterion, through: :requirement

• 每个发现的状态都可以是critical
• 如果一项需求至少有一个关键发现，则其状态也为关键
• 如果成功标准至少有一个关键要求，则其状态也为关键

所以临界状态是继承的。

在显示成功标准列表时，我想显示其状态是否关键。为此，我通过遍历所有需求来计算状态，然后再次遍历所有发现并搜索至少一个关键发现。

这需要大量的数据库查询，所以我正在考虑在成功标准中缓存状态，每次我添加/修改/删除发现时，状态都应该在成功标准中更新。

最好的方法是什么？我想到了像这样的 after_save 过滤器:

model Finding
  belongs_to :requirement
  has_one :success_criterion, through: :requirement

  after_save :update_success_criterion_status

  private
    def update_success_criterion_status
      if status.critical?
        success_criterion.update_attribute :status, :critical
      else
        success_criterion.calculate_status! # Iterate through all findings and look for at least one critical
      end
    end
end

在开发中，我想这会很有效。

但是测试呢？对于成功标准的单元测试，我必须为每个测试提供必要的关联，否则 after_save 过滤器会崩溃。这意味着每次测试的开销很大。

当然，我可以做一些像 hacky 这样的事情

after_save :update_success_criterion_status, unless: -> { Rails.env.test? }

但我认为这不是一个好方法。

有更好的方法吗？还是我走的路线完全错误？也许甚至有 gem 可以处理这样的事情(修改关联资源的属性)？

附言

类似的要求是可以对深度关联的元素进行反缓存。比方说，我们有另一个模型，例如

Project
  has_many :success_criteria

当我们想知道一个项目中有多少发现时，我们必须通过成功标准、需求和发现来统计它们。计数器缓存也会在这里保存大量查询。但是需要注意所有关联模型的大量创建/更新/删除内容以更新计数器缓存...

Answer 1

我认为最好的方法是创建另一个所有相关模型都可以有效查看的模型。您可以创建一个 Status具有不同状态类型的 bool 列的模型——至少其中一个是 critical列，显然——您在其中存储计数。

您将能够使用 <model_instance>.status 检索任何模型的状态这样，您就可以确定它是否重要。例如，finding.status.critical? , 或 a_success_criterion.status.critical?甚至 a_success_criterion.critical? .

model Finding
  belongs_to :requirement
  has_one :success_criterion, through: :requirement
  has_one :status

  def critical?
    status.critical? # Note: Rails should define this method for you on the status because it's a boolean column
  end
end

类似地，检索 Requirements的状态，您只需使用 requirement.status 即可和 requirement.critical?以确定需求是否关键。

model Requirements
  belongs_to :success_criterion
  has_many :findings
  has_many :statuses

  def critical?
    return true if statuses.where(critical: true).any?
  end
end

并检索 SuccessCriterion的状态，您只需使用 success_criterion.status 即可因为:

model SuccessCriterion
  has_many :requirements
  has_many :findings, through: :requirements
  has_many :statuses

  def critical?
    return true if statuses.where(critical: true).any?
  end
end

关键部分(没有双关语意——最初:)是当您创建一个发现并因此创建一个状态时:您必须或应该为该状态提供该发现的 ID、要求和成功它所属的标准，因此您可能希望为它们的存在添加验证。此外，您可能希望在创建状态时添加一个 before_validation，在 Status 中或在 Finding ，看起来像这样(在 Status 中):

model Status
  belongs_to :finding, :requirement, success_criterion
  before_validation :populate_ids
  validates_presence_of :finding_id, :requirement_id, :success_criterion_id

  def populate_ids
    self.finding_id = finding.id
    self.requirement_id = finding.requirement.try(:id)
    self.success_criterion_id = finding.requirement.try(:success_criterion).try(:id)
  end
end

在您的测试中，您只需为 id 提供整数，但如果您不想测试此类关系，则它们不必是实际相关的模型——除了高效之外，这就是它的优点查询 :).

你明白了。希望对您有所帮助!