Django视图字典定义时FilterProductForm的__init__自动执行问题

首先你的认知存在一个最核心的误解：Python 字典构造时，所有值位置的表达式都会立即执行，和后续是否访问对应键没有任何关系，不是第22个FilterProduct键特殊，你之前21个Product表单的实例化逻辑也在字典定义时全部执行了，只是它们的构造过程没有报错，你没有感知到而已。

报错的具体根因 #

你进入else分支时，obj是和当前product_name对应的模型实例（比如测试时是Product1的实例），但你在构造form_dict时，会直接执行FilterProductForm的实例化代码，把非FilterProduct模型的obj作为instance参数传给了FilterProductForm，同时还传入了user参数：

1. 最早的TypeError报错，是因为你当时的FilterProductForm的__init__方法还没有加user = kwargs.pop('user',None)这行逻辑，接收了额外的user参数就直接抛错了。
2. 加上pop user的逻辑后依旧存在问题：FilterProductForm是绑定FilterProduct模型的ModelForm，当你传入其他模型的实例作为instance时，__init__里的elif self.instance.pk:分支会尝试访问self.instance.product.filter_set，而其他模型的实例要么没有product属性，要么product关联的对象没有filter_set属性，自然就会报错。
3. 你后续修改了__str__和判断条件，相当于加了一层兼容，即使传入其他模型的实例也不会走到会报错的逻辑分支，所以代码暂时能跑，但本质是绕过了问题，没有解决根因。

正确的修复方案 #

完全没有必要预实例化所有表单，既浪费服务器资源，还会触发无关代码的执行，你可以把表单类存在字典里，只实例化当前需要的那一个表单即可：

# 先定义表单类映射，不会触发任何实例化逻辑
FORM_CLASS_MAP = {
    'FilterProduct': FilterProductForm,
    'Product1': Product1Form,
    'Product2': Product2Form,
    # 其余表单类...
    'Product21': Product21Form,
}

def add_to_quote(request,*args,**kwargs):
    pk = kwargs['pk']
    product_name = kwargs.get('product_name') or request.POST['product_name']
    # 直接取需要的表单类
    target_form_class = FORM_CLASS_MAP[product_name]

    if kwargs.get('line_item') is None:
        product_template_form = 'quote/product_template_form.html'
        # 仅实例化需要的表单
        if product_name == 'FilterProduct':
            form = target_form_class(request.POST, initial={'quote':pk}, user=request.user)
        else:
            form = target_form_class(request.POST, initial={'quote':pk})
    else:
        obj = get_object_or_404(model_dict[product_name], pk=kwargs['line_item'])
        product_template_form = 'quote/edit_product_form.html'
        # 仅实例化需要的表单
        if product_name == 'FilterProduct':
            form = target_form_class(request.POST or None, initial={'quote':pk}, instance=obj, user=request.user)
        else:
            form = target_form_class(request.POST or None, initial={'quote':pk}, instance=obj)
    # 后续原有逻辑
    return render(request, product_template_form, {'form':form})

这样修改后，只会执行当前请求对应表单的实例化逻辑，不会触发其他无关表单的构造代码，从根源上避免了这类问题。

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