如何通过Django外键关联更新ExtendedUser模型的level字段

原有代码问题分析 #

你的代码存在3个核心问题导致功能不生效：

• 统计完成订单数的时机错误：你是在修改当前订单的is_complete为True后、还未将修改写入数据库时就统计数量，当前这笔刚完成的订单不会被计入统计结果，计数会少1
• 修改用户等级后未保存对应的ExtendedUser对象：你只修改了内存中extendeduser的level属性，仅调用了checkout.save()，ExtendedUser的修改根本没同步到数据库
• 重复调用checkout.save()，属于冗余操作
• 异常捕获使用裸except不够规范，可能掩盖其他未知错误

修正后的代码 #

def sellerSubmitView(request, pk):
    form = SellerSubmitForm()

    if request.method == "POST":
        file_field = request.FILES.get("file_field")
        try:
            checkout = Checkout.objects.get(id=pk)
        except Checkout.DoesNotExist:
            return redirect("manage-order")
        else:
            # 创建提交记录
            SellerSubmit.objects.create(checkout=checkout, file_field=file_field)
            # 更新订单状态并保存到数据库
            checkout.order_status = "DELIVERED"
            checkout.is_complete = True
            checkout.save()
            # 统计当前卖家已完成的订单数（此时当前订单已经计入统计）
            completed_count = Checkout.objects.filter(seller=request.user, is_complete=True).count()
            print("已完成订单数：" + str(completed_count))
            # 满足条件则更新卖家等级
            if completed_count > 5:
                extended_user = request.user.extendeduser
                extended_user.level = "LEVEL1"
                extended_user.save() # 必须保存ExtendedUser对象修改才能生效
                print("更新后等级：" + extended_user.level)
            
            return redirect("manage-order")

    args = {
        "form": form,
        "checkout_id": pk,
    }
    return render(request, "wasekPart/sellerSubmit.html", args)

可选优化点 #

如果需要保证逻辑的可靠性，还可以做以下优化：

• 把等级更新的逻辑放到Django信号（Signal）中实现，只要Checkout的is_complete被修改为True就自动触发等级校验，避免在多个视图中写重复逻辑
• 统计订单数的时候增加order_status="DELIVERED"的过滤条件，避免状态和完成标记不匹配的脏数据干扰统计结果
• 增加等级判断逻辑，如果当前等级已经高于LEVEL1就不用重复更新，减少不必要的数据库操作

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