我有以下模型和关联:
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
当我们想知道一个项目中有多少发现时,我们必须通过成功标准、需求和发现来统计它们。计数器缓存也会在这里保存大量查询。但是需要注意所有关联模型的大量创建/更新/删除内容以更新计数器缓存...
最佳答案
我认为最好的方法是创建另一个所有相关模型都可以有效查看的模型。您可以创建一个 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 提供整数,但如果您不想测试此类关系,则它们不必是实际相关的模型——除了高效之外,这就是它的优点查询 :).
你明白了。希望对您有所帮助!
关于ruby-on-rails - rails : Tight coupling between deeply nested models: how to uncouple them for tests and DB efficiency?,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/41502219/