Python - Efficiency -在调用filter()之前检查list是否为空？

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