在代码的改造过程中,因为忽略了一个问题导致数据异常,在改造的过程中以及后续的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))