Django按当前用户过滤：自定义Manager放模型层还是视图层？

核心结论：按当前用户过滤的逻辑应该放在视图层，模型层不适合处理依赖请求上下文的过滤逻辑，原因和具体实现方式如下： #

一、为什么模型层的Manager行不通？

你之前尝试的FilterManager写法会直接报错，因为模型层的Manager无法访问请求对象（request）。Manager是绑定在模型上的全局对象，生命周期和单个请求无关，它不知道当前发起请求的用户是谁，所以根本拿不到self.request.user。模型层的Manager只适合处理静态的、与上下文无关的过滤逻辑（比如你举例的active=True、city='kolkata'这类固定值过滤）。

二、视图层的最优实现：用Mixin复用过滤逻辑

既然你在多个视图里都需要重复写这个过滤逻辑，最简洁的方式是封装一个自定义Mixin，把过滤逻辑抽出来，让需要的ViewSet继承它即可：

1. 编写UserFilterMixin：

# 可以放在views.py或单独的mixins.py文件中
class UserFilterMixin:
    def get_queryset(self):
        # 先获取基础查询集
        queryset = super().get_queryset()
        # 过滤当前用户的Event数据
        return queryset.filter(author=self.request.user)

2. 在ViewSet中使用Mixin：

class EventViewSet(UserFilterMixin, viewsets.ModelViewSet):
    serializer_class = serializers.EventSerializer
    queryset = Event.objects.all()  # 使用默认Manager即可

这样所有需要按当前用户过滤的ViewSet，只需要继承UserFilterMixin，不用重复编写过滤代码，完美解决重复问题。

三、模型层可以做什么辅助？

如果想在模型层封装用户相关的查询逻辑（但还是需要视图层传入用户参数），可以给Event模型添加类方法：

class Event(models.Model):
    author = models.ForeignKey(User, on_delete=models.CASCADE)
    # 其他字段...

    @classmethod
    def get_for_user(cls, user):
        return cls.objects.filter(author=user)

然后在视图层的get_queryset中调用：

def get_queryset(self):
    return Event.get_for_user(self.request.user)

这种方式既保持了模型层封装数据逻辑的职责，又能复用查询规则，但核心的用户参数还是得从视图层的request获取。

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