重写Django模型管理器last()方法后未过滤问题排查

首先，你的代码里有一个关键错误直接导致了过滤失效：在返回语句中，你把已经过滤好的record_queryset当成函数调用了（record_queryset()）。

Django的filter()返回的是一个QuerySet对象，不是可调用的函数。当你对QuerySet加括号时，实际上会触发管理器的all()方法，相当于重新获取了所有记录——这就导致之前的filter(*args, **kwargs)完全没起作用，这也是你得到60条全量记录的核心原因。

另外还有两个小问题需要修正：

• 日志里的name变量没有定义，你应该从传入的kwargs里获取，或者直接打印查询条件
• 用[0]取元素如果QuerySet为空会抛IndexError，虽然你加了try-except，但可以用更优雅的方式处理

import logging
from django.db import models

logger = logging.getLogger(__name__)

class RecordManager(models.Manager):
    def last(self, *args, **kwargs):
        '''返回符合条件的最后一条记录，若存在多条则记录日志'''
        # 获取过滤后的QuerySet
        record_queryset = self.filter(*args, **kwargs)
        record_count = record_queryset.count()
        
        if record_count > 1:
            # 修复name未定义的问题，打印查询参数和匹配的ID列表
            ids = ", ".join(str(obj.id) for obj in record_queryset)
            logger.debug(f"{record_count} duplicates found for query: {kwargs}. IDs: {ids}")
        elif record_count == 0:
            return None
        
        # 直接对过滤后的QuerySet使用原生last()，按id倒序取最后一条
        # 显式指定排序，避免受模型默认排序影响
        return record_queryset.order_by('-id').last()

1. 移除QuerySet的括号调用：直接使用record_queryset.order_by('-id').last()，完整保留了之前的过滤条件
2. 修复日志变量问题：通过遍历QuerySet获取ID列表，并用kwargs打印查询条件，避免未定义变量报错
3. 更安全的取值方式：使用Django QuerySet原生的last()方法，它会在QuerySet为空时自动返回None，不需要额外的try-except（当然保留也没问题，但这样更简洁）
4. 明确排序逻辑：显式指定order_by('-id')，确保是按ID倒序取最新的记录，不会受模型Meta.ordering的默认设置影响

现在调用Record.objects.last(name="jim")就会正确返回符合name="jim"的最后一条记录，同时如果有多个匹配项，会在日志里打印数量和对应的ID列表。

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