视图内条件执行问题：用户专属变量声明报错及优化咨询

1. 修正「local variable referenced before assignment」错误 #

这个错误的根源很明确：当用户不是supervisor时，deliverer变量根本没被初始化，但你在context里依然尝试引用它。要解决这个问题，我们只需要提前给变量设置一个合理的默认值，确保无论条件是否成立，变量都存在。

修正后的代码如下：

def order_detail(request, order_id):
    order = get_object_or_404(Order, id=order_id)
    # 初始化默认值：空的QuerySet，和filter返回的类型保持一致
    deliverer = User.objects.none()
    if request.user.is_supervisor:
        user_location = request.user.supervisor.town
        deliverer = User.objects.filter(deliverer__town=user_location)
    context={
        'order': order,
        'all_delivery_guys_in_town': deliverer
    }
    template = 'orders/order_mgt/detail.html'
    return render(request, template, context)

这样，即使当前用户不是supervisor，deliverer也会是一个空的QuerySet，前端模板可以正常处理（比如判断是否为空再渲染），不会出现变量未定义的报错。

2. 同一视图内条件声明的更优方案 #

视图层应该尽量简洁，专注于请求/响应的处理，复杂的业务逻辑可以考虑以下两种优化方式：

方案一：使用条件表达式简化代码 #

如果逻辑比较简单，直接用Python的条件表达式把变量声明压缩成一行，让代码更紧凑：

def order_detail(request, order_id):
    order = get_object_or_404(Order, id=order_id)
    deliverer = (
        User.objects.filter(deliverer__town=request.user.supervisor.town)
        if request.user.is_supervisor
        else User.objects.none()
    )
    context={
        'order': order,
        'all_delivery_guys_in_town': deliverer
    }
    template = 'orders/order_mgt/detail.html'
    return render(request, template, context)

方案二：将业务逻辑封装到模型方法（推荐） #

把和用户相关的逻辑封装到User模型中，不仅让视图更干净，还能实现代码复用（其他视图也可以调用这个方法）：

首先在User模型中添加方法：

from django.contrib.auth.models import AbstractUser

class User(AbstractUser):
    # 你的现有字段...

    def get_local_deliverers(self):
        # 先检查用户是否关联了supervisor对象，避免DoesNotExist错误
        if hasattr(self, 'supervisor'):
            return User.objects.filter(deliverer__town=self.supervisor.town)
        # 默认返回空QuerySet
        return User.objects.none()

然后视图可以简化成：

def order_detail(request, order_id):
    order = get_object_or_404(Order, id=order_id)
    deliverer = request.user.get_local_deliverers()
    context={
        'order': order,
        'all_delivery_guys_in_town': deliverer
    }
    template = 'orders/order_mgt/detail.html'
    return render(request, template, context)

这种方式符合DRY（Don't Repeat Yourself）原则，也让业务逻辑更集中，后续维护和测试会更方便。

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