何时删除列表理解和Pythonic方式？

Question

我创建了一个以下列方式将对象附加到列表的行

>>> foo = list()
>>> def sum(a, b):
...      c = a+b; return c
...
>>> bar_list = [9,8,7,6,5,4,3,2,1,0]
>>> [foo.append(sum(i,x)) for i, x in enumerate(bar_list)]
[None, None, None, None, None, None, None, None, None, None]
>>> foo
[9, 9, 9, 9, 9, 9, 9, 9, 9, 9]
>>>

这条线

[foo.append(sum(i,x)) for i, x in enumerate(bar_list)]

会给一个pylint W1060表达式什么都没有，但由于我已经在使用foo列表来附加值，所以我不需要将List Comprehension行赋予某些东西。

我的问题更多的是编程正确性

我应该删除列表理解并使用简单的表达式吗？

>>> for i, x in enumerate(bar_list):
...      foo.append(sum(i,x))

或者是否有正确的方法来使用列表理解和分配到什么？

回答

谢谢@ user2387370，@ kindall和@Martijn Pieters。 对于其余的注释，我使用append，因为我没有使用list（），我没有使用i + x，因为这只是一个简化的例子。

我把它留下如下：

histogramsCtr = hist_impl.HistogramsContainer()
for index, tupl in enumerate(local_ranges_per_histogram_list):
    histogramsCtr.append(doSubHistogramData(index, tupl))
return histogramsCtr

Answer 1

是的，这是不好的风格。 列表理解是建立一个列表。 您正在构建一个完整的清单None秒，然后把它扔了。 您实际需要的结果是这项工作的副作用。

为什么不首先使用列表理解来定义foo ？

foo = [sum(i,x) for i, x in enumerate(bar_list)]

如果它不是一个列表而是一些其他容器类，正如你在另一个答案的注释中提到的那样，写一个类来接受它的构造函数中的一个iterable（或者，如果它不是你的代码，则将它子类化为这样做），然后传递一个生成器表达式：

foo = MyContainer(sum(i, x) for i, x in enumerate(bar_list))

如果foo已经有一些价值并且您希望追加新项目：

foo.extend(sum(i,x) for i, x in enumerate(bar_list))

如果你真的想使用append()并且由于某些原因不想使用for循环，那么你可以使用这个结构; 生成器表达式至少可以避免在您不想要的列表上浪费内存和CPU周期：

any(foo.append(sum(i, x)) for i, x in enumerate(bar_list))

但是这比常规for循环更不清楚，还有一些额外的工作要做： any都在每次迭代时测试foo.append()的返回值。 您可以编写一个函数来使用迭代器并消除该检查; 最快的方法是使用零长度collections.deque ：

from collections import deque
do = deque([], maxlen=0).extend

do(foo.append(sum(i, x)) for i, x in enumerate(bar_list))

这实际上是相当可读的，但我相信它实际上并不比any()更快，并且需要额外的导入。 但是， do()或any()比for循环快一点，如果这是一个问题。

Answer 2

我认为通常不赞成使用列表推导仅用于副作用，所以我想说for循环在这种情况下更好。

但无论如何，你不能只做foo = [sum(i,x) for i, x in enumerate(bar_list)] ？

Answer 3

你绝对应该放弃列表理解。 结束。

• 您在阅读代码时感到困惑。 您正在构建副作用列表。
• 您正在支付CPU周期和内存来构建您要再次丢弃的列表。

在您的简化案例中，您忽略了您可以直接使用列表推导的事实：

[sum(i,x) for i, x in enumerate(bar_list)]