重构此Python代码以迭代容器

use*_*478 3 python django refactoring iterator

当然有更好的方法来做到这一点?

results = []
if not queryset is None:
    for obj in queryset:
        results.append((getattr(obj,field.attname),obj.pk))
Run Code Online (Sandbox Code Playgroud)

问题是有时queryset是None,当我尝试迭代它时会导致异常.在这种情况下,我只想将结果设置为空列表.这段代码来自Django视图,但我认为不重要 - 这似乎是一个更普遍的Python问题.

编辑:我发现我的代码是将一个空的查询集转换为"无"而不是返回一个空列表.能够假设查询集始终是可迭代的,通过允许删除"if"语句来简化代码.下面的答案可能对有相同问题的其他人有用,但无法修改其代码以保证查询集不是"无".

jfs*_*jfs 19

results = [(getattr(obj, field.attname), obj.pk) for obj in queryset or []]
Run Code Online (Sandbox Code Playgroud)


Han*_*rén 8

怎么样

for obj in (queryset or []):
    # Do your stuff
Run Code Online (Sandbox Code Playgroud)

它与JF Sebastians的建议相同,只是没有作为列表理解实现.