如何用Python高效编写多分支判断逻辑（Django场景）

嘿，先帮你捋下原始代码里的几个明显问题：

• 语法错误：render语句末尾缺了个闭合括号，运行会直接报错
• 逻辑矛盾：内层else里返回的“POST but not logged in”完全说不通——外层已经判断用户已登录才会走到这个分支，不可能出现这种情况
• 表单初始化错误：处理POST请求时，你用了空表单forms.DocumentForm()，这样根本没法校验用户提交的数据

下面是重构后的代码，用提前返回的方式砍掉嵌套，逻辑清晰又符合Django的最佳实践：

from django.shortcuts import render
from django.contrib.auth.decorators import login_required

# 用Django内置装饰器自动处理登录校验，未登录直接跳转到指定登录页
@login_required(login_url='/accounts/login/')  # 替换成你的登录页面实际URL
def upload(request):
    # 非POST请求直接返回空白表单
    if request.method != 'POST':
        form = forms.DocumentForm()
        return render(request, 'upload.html', {'form': form})
    
    # 走到这里就说明：用户已登录 + 请求是POST，开始处理表单
    # 如果涉及文件上传，必须加上request.FILES参数
    form = forms.DocumentForm(request.POST, request.FILES)
    if form.is_valid():
        # 这里写表单验证通过后的业务逻辑，比如保存文件到数据库/服务器
        form.save()
        return HttpResponse('文件上传成功！')
    # 验证失败，返回带错误信息的表单给用户修正
    return render(request, 'upload.html', {'form': form})

优化点说明： #

• 用@login_required简化登录校验：Django内置的装饰器会自动检查用户是否登录，未登录则直接跳转到login_url指定的页面，不用自己写繁琐的if判断，代码更简洁且不易出错
• 提前返回减少嵌套：先处理非POST的情况，直接返回表单页面，剩下的代码自然就是POST请求的处理逻辑，避免了多层if-else嵌套，可读性拉满
• 修正表单处理逻辑：POST请求时必须用用户提交的request.POST（和request.FILES如果涉及文件）初始化表单，才能进行合法性校验；验证失败时返回带错误的表单，用户能直接看到哪里填错了
• 移除无效逻辑：删掉了原始代码里逻辑矛盾的错误提示，每个分支的作用都清晰明确

如果你的登录路由不是/accounts/login/，记得改成你项目里实际的登录页面URL。另外如果是处理文件上传，模板里的form标签一定要加上enctype="multipart/form-data"，否则文件会传不进来哦。

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