技术团队中经典缺陷分享

见解分享

2023-10-07 17:10:34

经典缺陷案例：揭开共同借款人配偶唯一性校验背后的故事

缺陷背景

软件开发过程中，缺陷发现是司空见惯的。然而，有些缺陷因其启发性和趣味性，在技术团队中留下深刻印象，成为经典案例。本文将带你深入探讨一个经典缺陷的发现过程，从中汲取经验教训，提升测试质量，促进团队合作。

准备阶段：精雕细琢的测试点

在本例中，缺陷发生在金融科技项目的测试阶段。测试点是检验共同借款人配偶的唯一性。测试人员为准备测试可谓煞费苦心，耗时三天整理了三条数据：一条配偶信息，另外两条是相同关系的数据。

预期结果：环环相扣的校验规则

根据业务需求，共同借款人配偶的唯一性校验规则清晰明了：同一笔贷款下，配偶信息不能重复；不同笔贷款下，配偶信息可以重复。基于此规则，测试人员预期如下结果：

三条测试数据中，只有一条配偶信息通过唯一性校验。
系统提示错误信息，表明配偶信息已存在。

实际结果：出乎意料的漏洞

然而，实际测试结果却令人大跌眼镜：三条测试数据全部通过唯一性校验，系统也没有任何错误提示。这无疑是令人困惑的，测试人员随即展开深入调查。

缺陷分析：抽丝剥茧的逻辑追踪

通过细致分析，测试人员发现缺陷根源在于代码逻辑错误。具体而言，代码中缺少了对不同笔贷款配偶信息重复性的校验，导致系统错误地允许了重复的配偶信息。

代码示例：

if (isLoanDifferent(loan1, loan2)) {
  // 不同笔贷款，允许配偶信息重复
} else {
  // 同一笔贷款，校验配偶信息唯一性
  if (isSpouseDuplicate(spouse1, spouse2)) {
    // 配偶信息重复，提示错误信息
  }
}

经过修改，代码逻辑完善如下：

if (isLoanDifferent(loan1, loan2)) {
  // 不同笔贷款，允许配偶信息重复
} else {
  // 同一笔贷款，校验配偶信息唯一性
  if (isSpouseDuplicate(spouse1, spouse2)) {
    // 配偶信息重复，提示错误信息
  }
} else if (isLoanDifferent(loan1, loan2)) {
  // 不同笔贷款，校验配偶信息重复性
  if (isSpouseDuplicate(spouse1, spouse2)) {
    // 配偶信息重复，提示错误信息
  }
}