Python - 效率 - 在调用filter()之前检查列表是否为空？

Question

我有一个文件列表，我需要过滤可接受的文件类型(例如，仅保留“.txt”文件)。然后，当且仅当最终列表不为空时，我才对最终列表进行“操作”。我的原始代码(带有玩具列表)是这样的:

files = ["file1.txt", "file2.txt", "file3.jpg"]
if files:
    files[:] = filter(check_filetype, files)
    if files:
         # do something with remaining .txt files

但是，我很好奇是否需要第一个“if”语句，以在过滤之前检查此列表是否为空。我测试过，filter() 将接受(并返回)一个空列表。

那么，哪个更有效，是在第一个“if”语句中添加一个额外的分支，还是过滤一个空列表？我倾向于删除第一个“if”语句。

Answer 1

在我看来，您可以删除第一个 if 语句。另请注意，您可以只使用列表理解，这通常被认为更Pythonic。至少，与 filter + 自定义函数相比，它对于较大的列表应该更有效:

files = ['file1.txt', 'file2.txt', 'file3.txt']
files = [fn for fn in files if check_filetype(fn)]

if files:
    # do something