使用pop方法删除列表中的重复元素

Question

我已尝试使用以下代码删除以下代码以删除列表中的重复项，但它给了我一个错误：列表索引超出范围。

l1=[8,32,33,21,98,3,21,32,89,45,34,33,90,33,21,34,33]
print('Original List:',l1)
i=0
while i <len(l1):
    no=l1[i]
    for x in range(i+1,len(l1)):
        val=l1[x]
        if val==no:
            l1.pop(x)
    i+=1
print('List after removing duplicate items is: ',l1)

请帮我解决错误，我该如何纠正错误

Answer 1

它抛出错误的原因是你的内部循环只设置了一次长度。 因此，一旦您开始删除项目，它的长度就会比它将尝试迭代的长度短。 您可以尝试将内部循环更改为 while 循环。 但更 Pythonic 的方法是使用 for 循环。

但是考虑到您只对元素使用简单的相等比较，为什么不试试这个。

l1 = list(dict.fromkeys(l1))

dict.fromkeys 创建一个字典，其键设置为第一个参数的值，然后默认将值设置为 None 。 所以基本上它输出： {8: None, 32: None, 33: None, 21: None, 98: None, 3: None, 89: None, 45: None, 34: None, 90: None}

由于字典不能有重复的键，重复的就消失了。

然后在其上调用list()通过使用新字典的键作为值将其转换为列表。

如果你必须用循环来做，你可以尝试建设性地做：

l1=[8,32,33,21,98,3,21,32,89,45,34,33,90,33,21,34,33]
print('Original List:',l1)
new_list = []
for x in l1:
    if x not in new_list:
        new_list.append(x)
print('List after removing duplicate items is: ', new_list)

但是如果你真的必须使用pop()和循环。 你可以这样做：

for i, x in reversed(list(enumerate(l1))): #loop through the list from the end (with the index of that item)
    if l1.count(x) > 1: # if there is more than one of that element in the list, pop the last element you tested.
        l1.pop(i)

如果顺序无关紧要，我的做法是使用集合： l1 = list(set(l1))