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)
怎么样
for obj in (queryset or []):
# Do your stuff
Run Code Online (Sandbox Code Playgroud)
它与JF Sebastians的建议相同,只是没有作为列表理解实现.