在代码的改造过程中，因为忽略了一个问题导致数据异常，在改造的过程中以及后续的review中都没注意到这个问题，单元测试也没有覆盖到，记录如下。这个坑在于all的使用上，如果参数为空元组或空列表时，返回值为True，这是要特别注意的地方。改造时忽略了这个地方，应该这样写就对了：
if not codexes or codex in codexes
原代码：
result = set()
for student_id, codex in rs:
conditions = []
if codexes:
conditions.extend([codex in codexes])
if all(conditions):
result.add(student_id)
return result.intersection(set(student_ids))
新代码改为如下引出问题：
result = set()
for student_id, codex in rs:
if (codexes and codex in codexes):
result.add(student_id)
return result.intersection(set(student_ids))
修补后正确应该为：
result = set()
for student_id, codex in rs:
if not codexes or codex in codexes:
result.add(student_id)
return result.intersection(set(student_ids))
此内容由惯性聚合(RSS阅读器)自动聚合整理，仅供阅读参考。 原文来自 — 版权归原作者所有。