Django提交操作中变量未定义问题的排查与解决

我帮你梳理下这段视图代码里导致POST请求时出现变量未定义的核心问题，还有对应的修复方案：

1. 未先判断请求类型就直接操作POST数据 #

你现在的代码开头直接就去取request.POST里的userpost并查询数据库，但如果是GET请求（比如用户直接访问这个路由），request.POST是空的，不仅会导致ADMirror.objects.get()查询失败，还会让后续依赖owner的所有变量都无法定义。

修复方案：把所有POST相关的逻辑都包裹在请求类型判断的块里，开头就做请求校验：

def submitted(request):
    if request.method == 'POST':
        # 所有POST相关的业务逻辑都放在这个块内
        try:
            owner = ADMirror.objects.get(employeentname=request.POST.get('userpost'))
        except ADMirror.DoesNotExist:
            # 处理用户不存在的异常，比如返回错误页面
            return render(request, 'error.html', {'msg': '未找到对应用户'})
        
        firstname = owner.employeefirstname
        lastname = owner.employeelastname
        # ... 其余POST逻辑
    else:
        # 处理GET请求，比如重定向到表单页面
        return redirect('results')

2. 变量作用域缺失：部分分支未定义变量 #

像accesslevel、orglevelname、selectlist这些变量，你只在selectedaccesslevel等于3-7的分支里定义，但如果前端传错值（比如传了个8），这些变量就会变成未定义状态，后续使用时直接报错。

修复方案：先给这些变量设置默认值，再根据条件覆盖：

# 先初始化默认值，确保所有分支都能拿到变量
accesslevel = "未知权限级别"
orglevelname = []
selectlist = ""

selectedaccesslevel = request.POST.get('accesslevelid')
if selectedaccesslevel == '3':
    accesslevel = 'company-wide access'
    orglevelname = ''
    selectlist = "S00001"
elif selectedaccesslevel == '4':
    accesslevel = 'group level access'
    accesslevelname = request.POST.getlist('blevel')
    orglevelname = FacilityDimension.objects.filter(b_level__in=accesslevelname).values_list('group_name', flat=True).distinct()
    selectlist = request.POST.getlist('blevel')
elif selectedaccesslevel == '5':
    accesslevel = 'division level access'
    accesslevelname = request.POST.getlist('rlevel')
    orglevelname = FacilityDimension.objects.filter(r_level__in=accesslevelname).values_list('division_name', flat=True).distinct()
    selectlist = request.POST.getlist('rlevel')
# ... 其余权限级别的elif分支

3. 代码截断导致的语法/逻辑不完整 #

你提供的代码最后部分是截断的：previousdatare，这会直接导致语法错误，比如QVFormAccessRequest的实例化没有闭合，后续的save()操作也无法正常执行，进而引发变量未定义或逻辑中断的问题。

修复方案：补全截断的代码，确保模型实例化的参数完整、括号闭合：

# 补全后的示例代码
else:
    for j in selectlist:
        requestsave = QVFormAccessRequest(
            ntname='HCA\\'+owner.employeentname,
            first_name=owner.employeefirstname,
            last_name=owner.employeelastname,
            coid=owner.coid,
            title=owner.title,
            report_id=i,
            accesslevel_id=selectedaccesslevel,
            phi=selectedphi,
            access_beg_date=access_request_date,
            previousdatareducecode='',
            datareducecode=j,
            facility=owner.facilityname,
            requestid='0',
            requesttype='New'
        )
        requestsave.save()

4. 冗余的请求类型判断 #

你每个权限分支前都重复写了request.method == 'POST'，这完全没必要——既然已经把所有逻辑放在最外层的POST判断块里，内部就不需要再重复校验了，反而增加了维护成本。

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