重构此python代码以在容器上迭代

results = [(getattr(obj, field.attname), obj.pk) for obj in queryset or []]

怎么样

for obj in (queryset or []):
    # Do your stuff

这与J.F塞巴斯蒂安的建议相同,只是没有作为清单理解来实现。

对于它的价值,Django管理者有一个“无”查询集,您可以使用它来避免不必要的无检查。使用它来确保没有空查询集可能会简化代码。

if queryset is None:
    queryset = MyModel.objects.none()

参考文献:

• Django documentation on none()
• Null Object Pattern

你可以使用列表理解,但除此之外,我看不出你能改进什么。

result = []
 if queryset:
     result = [(getattr(obj, field.attname), obj.pk) for obj in queryset]