Django根据登录用户上传Beat数量切换模板报错排查

让我们一步步拆解并解决你遇到的三个问题：

1. 为什么会出现NameError: name 'self' is not defined？ #

你在类的顶层直接写了if get_queryset(self).count() <= 2:，这是错误的逻辑——类定义阶段还没有实例化对象，self根本不存在！类顶层的代码是服务器启动时就执行的，而不是每次处理请求时运行。

2. 为什么硬编码用户ID只有重启服务器才生效？ #

同样是因为类属性（比如template_name）在服务器启动时就被赋值了，不会随每次请求动态更新。必须把用户上传数量的判断放到每次请求都会执行的方法里。

3. 为什么Django找不到指定模板？ #

当你的template_name没有被正确设置时，CreateView会自动使用默认模板路径：<app_name>/<model_name>_form.html（也就是beatupload/beat_form.html），而你没有这个文件，所以报错。

修复后的完整代码 #

下面是修正后的uploadNew视图，完美解决所有问题：

from django.views.generic.edit import CreateView
from django.urls import reverse_lazy
from .models import beat

class uploadNew(CreateView):
    model = beat
    fields = ['title', 'beat']
    success_url = reverse_lazy('uploads')

    def get_template_names(self):
        # 每次请求时动态计算当前用户的上传数量
        user_beat_count = beat.objects.filter(producer=self.request.user).count()
        # 免费用户最多3个，所以<=2时还能上传
        if user_beat_count <= 2:
            return ['uploadNew.html']
        else:
            return ['becomeMember.html']

    def form_valid(self, form):
        form.instance.producer = self.request.user
        # Python3+ 推荐这种super写法
        return super().form_valid(form)

额外的优化建议 #

• 模型类名请遵循Python命名规范，改成Beat（大驼峰），避免和变量名混淆；视图类uploadNew也建议改成UploadNew。
• 可以把免费用户的上传限制放到settings.py里，比如FREE_USER_MAX_BEATS = 3，以后修改更方便：

  # settings.py
  FREE_USER_MAX_BEATS = 3
  
  # 视图中使用
  from django.conf import settings
  if user_beat_count < settings.FREE_USER_MAX_BEATS:
  

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