为什么这个列表理解不符合我的预期？

Question

project_keys = sorted(projects.keys())的原始列表是[101, 102, 103, 104, 105, 106, 107, 108, 109, 110]，其中以下项目今年被视为invalid：108、109、110。

因此：

for project in projects.itervalues():
# The projects dictionary is mapped to the Project class
    if project.invalid:
    # Where invalid is a Bool parameter in the Project class
     project_keys.remove(project.proj_id)  

print project_keys

这将返回一个整数列表（即项目 ID）：

[101, 102, 103, 104, 105, 106, 107]

甜。

现在，我希望它使用列表推导式尝试同样的事情。

project_keys = [project_keys.remove(project.proj_id) for project in projects.itervalues() if project.invalid  

print project_keys

这会返回：

[None, None, None]

所以我要填充一个与已删除元素相同编号的列表，但它们是Nones？

谁能指出我做错了什么？

此外，我为什么要在顶部的 for-if 块上使用列表推导？简明？好看吗？

Answer 1

您的列表理解使用副作用。只需执行它应该更新 project_keys 以提供您想要的结果。

[project_keys.remove(project.proj_id)
 for project in projects.itervalues()
 if project.invalid]

remove 的返回值为None。将列表理解的结果分配给 project_keys 是您出错的地方。

一个简单的循环在这里可能更清晰。使用副作用的列表推导可能会造成混淆。

但是，您可以通过稍微不同的方式解决您的问题：

project_keys = sorted(project.proj_id
                      for project in projects.itervalues()
                      if not project.invalid)

这会保留您感兴趣的项目，而不是删除您不感兴趣的项目。我上面给出的示例使用生成器表达式而不是列表推导式，但两者都适用。

Answer 2

先生，您误解了列表理解。

你可能想要什么（用文字）

我想删除所有无效的项目 ID。

你写了什么

project_keys = [project_keys.remove(project.proj_id)
                for project in projects.itervalues() if project.invalid]

实际发生了什么

dummy = []
for project in projects.itervalues():
  if project.invalid:
    dummy.append(project_keys.remove(project.proj_id)) #what are you
project_keys = dummy                                   #removing items from?
del dummy                                            

实际发生了什么（现在有了更多“功能”）

mapped-fun = lambda project: project_keys.remove(project.proj_id)
filtering-fun = lambda project: project.invalid
project_keys = map(mapped-fun, filter(filtering-fun, projects.itervalues()))

---

如您所见，列表推导式不是围绕for 循环的语法糖。相反，列表推导是 map() 和 filter() 周围的语法糖：将函数应用于 sequence 中匹配条件的所有项目并获得 列表 个结果作为回报。

这里，函数实际上是指输入到输出的无副作用转换。这意味着您“不能”使用更改输入本身的方法，例如list.sort()；您必须使用它们的等效功能，例如 sorted()。

但是，“不能”并不是说您会收到错误消息或nasal demons；我的意思是你在滥用语言。在您的情况下，在将列表理解分配给变量时对其进行评估确实会产生预期的副作用 - 但它是否会在预期的变量上产生它们？

看，这可以在没有错误的情况下执行的唯一原因是在此列表理解之前，有 另一个 列表称为 project_keys 并且它是 那个 您所在的列表真的变了！

列表推导是函数式编程的结果，它拒绝副作用。使用列表推导时请记住这一点。

---

因此，您可以使用以下思考过程来实际获得所需的列表理解。

你真正想要的（用文字）

我想要所有有效的项目 ID（= 不是无效的。）

你真正想要的

dummy = []
for project in projects.itervalues():
  if not project.invalid:
    dummy.append(project.proj_id)
project_keys = dummy
del dummy

你真正想要的（现在有更多功能）

mapped-fun = lambda project: project.proj_id
filtering-fun = lambda project: not project.invalid
project_keys = map(mapped-fun, filter(filtering-fun, projects.itervalues()))

你真正想要的（现在作为列表理解）

project_keys = [project.proj_id for project in projects.itervalues()
                if not project.invalid]