Django Admin列表视图单次API调用优化方案咨询

重写get_queryset()完全可行，这是实现单次API调用的合适时机——因为这个方法在列表视图加载时只会执行一次，刚好能满足你减少API调用次数的需求。

你的初步实现思路没问题，但要注意几个细节： #

1. 字典键的匹配问题：不要直接用obj实例作为字典键（Python中实例作为键需要正确实现__hash__方法，容易出问题），建议用模型的唯一标识（比如obj.id）作为键，确保取值准确。
2. API调用的针对性：如果查询集是分页的，可以先提取当前页对象的ID，只请求这些ID对应的数据，避免拉取不必要的内容，进一步优化性能。
3. 异常处理：添加API调用的异常捕获，避免请求失败导致页面崩溃，同时可以给个默认提示。

优化后的完整示例代码： #

from django.contrib import admin
from .models import YourModel

class YourModelAdmin(admin.ModelAdmin):
    list_display = ['id', 'name', 'some_action']

    def get_queryset(self, request):
        qs = super().get_queryset(request)
        # 提取当前查询集所有对象的ID
        obj_ids = list(qs.values_list('id', flat=True))
        self.status = {}
        
        if obj_ids:
            try:
                # 单次调用API，传入需要的ID列表
                api_results = some_expensive_list_call(ids=obj_ids)
                # 转换成以ID为键的字典，方便快速取值
                self.status = {item['id']: item['status'] for item in api_results}
            except Exception:
                # API调用失败时，保持空字典，后续返回默认值
                pass
        
        return qs

    @admin.display(description="foo")
    def some_action(self, obj):
        # 从预加载的字典中取值，无数据时返回默认提示
        return self.status.get(obj.id, "未获取数据")

admin.site.register(YourModel, YourModelAdmin)

其他可行方法： #

也可以重写changelist_view方法，在页面渲染前执行API调用，逻辑和get_queryset类似：

def changelist_view(self, request, extra_context=None):
    qs = self.get_queryset(request)
    obj_ids = list(qs.values_list('id', flat=True))
    if obj_ids:
        self.status = some_expensive_list_call(ids=obj_ids)
    else:
        self.status = {}
    return super().changelist_view(request, extra_context)

不过get_queryset更贴合Django Admin的生命周期，逻辑上更连贯，优先推荐这种方式。

内容的提问来源于stack exchange，提问作者xtlc