编写Python列表去重函数的代码存在问题,求错误排查
代码存在的问题
remove方法参数使用错误:list.remove()接收的参数是要删除的元素值,而非索引i。示例中首次循环i=0,0并不在测试列表[1,2,3,4]中,会直接抛出ValueError: list.remove(x): x not in list报错。- 遍历过程中修改原列表导致索引越界:循环的
range是基于列表初始长度生成的,删除列表元素后列表实际长度会缩短,后续循环访问list1[i]时会出现IndexError索引越界报错。 - 去重逻辑完全不成立:
- 只要首次命中重复元素就会直接
return,只会删除第一个匹配的重复元素就终止函数,无法完成整个列表的去重。 - 如果输入列表没有重复元素,不会进入
if分支,函数没有任何返回值,最终返回None,不符合返回去重后列表的要求。
- 只要首次命中重复元素就会直接
- 存在副作用:函数直接修改传入的原列表对象,会导致函数外部的原始列表被意外篡改,不符合常规工具函数的设计预期。
正确实现参考
如果不需要保留元素原始顺序,最简单的实现是利用集合去重:
def func(list1): return list(set(list1))
如果需要保留元素首次出现的顺序(Python3.7+支持):
def func(list1): return list(dict.fromkeys(list1))
兼容所有Python版本且保序的写法:
def func(list1): seen = set() res = [] for item in list1: if item not in seen: seen.add(item) res.append(item) return res
内容的提问来源于stack exchange,提问作者angeloskar
相关产品推荐
相关产品推荐

