リストからアイテムを削除するときに、リストをループするのは非常に危険です。ほとんどの場合、いくつかの要素をスキップすることになります。
>>> L = [1, 1, 2, 2, 3, 3]
>>> for x in L:
... print x
... if x == 2:
... L.remove(2)
...
1
1
2
3
3
また、それぞれ.remove
が O(n) の複雑さであるため、非効率的です
新しいリストを作成してバインドし直す方が良いlist1
import os
list1 = ['myfile.v', 'myfile2.sv', 'myfile3.vhd', 'etcfile.v', 'randfile.sv']
list2 = ['myfile', 'myfile2', 'myfile3']
set2 = set(list2) # Use a set for O(1) lookups
list1 = [x for x in list1 if os.path.splitext(x)[0] not in set2]
または「インプレース」バージョンの場合
list1[:] = [x for x in list1 if os.path.splitext(x)[0] not in set2]
コメントで説明されているように、真にインプレース バージョンの場合 - 余分な O(n) メモリを使用しません。O(n)時間で実行
>>> list1 = ['myfile.v', 'myfile2.sv', 'myfile3.vhd', 'etcfile.v', 'randfile.sv']
>>> p = 0
>>> for x in list1:
... if os.path.splitext(x)[0] not in set2:
... list1[p] = x
... p += 1
...
>>> del(list1[p:])
>>> list1
['etcfile.v', 'randfile.sv']