您的位置:首页 > 编程语言 > Python开发

python内置函数all使用的坑

2017-01-16 14:31 246 查看
  在代码的改造过程中,因为忽略了一个问题导致数据异常,在改造的过程中以及后续的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))
内容来自用户分享和网络整理,不保证内容的准确性,如有侵权内容,可联系管理员处理 点击这里给我发消息
标签: