python内置函数all使用的坑

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

 

转载于:https://www.cnblogs.com/kylinfish/p/6289524.html

评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值