django 的 Form 类是否保持状态？

Question

我正在使用 django 构建我的第一个表单，并且我看到了一些我真的没想到的行为。我定义了一个表单类：

class AssignmentFilterForm(forms.Form):
filters = []
filter = forms.ChoiceField()

def __init__(self, *args, **kwargs):
    super(forms.Form, self).__init__(*args, **kwargs)
    self.filters.append(PatientFilter('All'))
    self.filters.append(PatientFilter('Assigned', 'service__isnull', False))
    self.filters.append(PatientFilter('Unassigned', 'service__isnull', True))

    for i, f in enumerate(self.filters):
        self.fields["filter"].choices.append((i, f.name))

当我使用以下方法将此表单输出到模板时：

{{ form.as_p }}

我看到了正确的选择。但是，刷新页面后，我在选择框中看到了 3 次列表。再次点击刷新会导致列表在选择框中显示 10 次！

这是我的看法：

@login_required
def assign_test(request):
pg = PhysicianGroup.objects.get(pk=physician_group)

if request.method == 'POST':
    form = AssignmentFilterForm(request.POST)
    if form.is_valid():
        yes = False
else:
    form = AssignmentFilterForm()
    patients = pg.allPatients().order_by('bed__room__unit', 'bed__room__order', 'bed__order' )

return render_to_response('hospitalists/assign_test.html', RequestContext(request,  {'patients': patients, 'form': form,}))

我做错了什么？

谢谢，皮特

Answer 1

这其实是 Python 的一个吸引很多人的特性。

当您像使用filters = [] 一样在类上定义变量时，表达式的右半部分在最初定义类时被计算。因此，当您的代码首次运行时，它将在内存中创建一个新列表并返回对该列表的引用。因此，每个AssignmentFilterForm 实例都有自己的过滤器变量，但它们都将指向内存中的同一个列表。要解决这个问题，只需将 self.filters 的初始化移动到您的 __init__ 方法中。

大多数情况下，您不会遇到此问题，因为您使用的类型未存储为参考。数字、布尔值等被存储为它们的值。字符串是通过引用存储的，但字符串是不可变的，这意味着每次更改字符串并返回新的引用时都必须在内存中创建一个新字符串。

指针在脚本语言中并不经常出现，因此一开始往往会让人感到困惑。

这是一个简单的 IDLE 会话示例，用于显示正在发生的事情

>>> class Test():
    myList = []
    def __init__( self ):
        self.myList.append( "a" )


>>> Test.myList
[]
>>> test1 = Test()
>>> Test.myList
['a']
>>> test1.myList
['a']
>>> test2 = Test()
>>> test2.myList
['a', 'a']
>>> test1.myList
['a', 'a']
>>> Test.myList
['a', 'a']

Answer 2

我拿起了 Pro Django 这本书来回答这个问题。顺便说一句，这是一本很棒的书，我强烈推荐它！

解决方案是让选择字段和我的助手 var 都成为实例变量：

class AssignmentFilterForm(forms.Form):
def __init__(self, pg, request = None):
    super(forms.Form, self).__init__(request)
    self.filters = []

    self.filters.append(PatientFilter('All'))
    self.filters.append(PatientFilter('Assigned', 'service__isnull', False))
    self.filters.append(PatientFilter('Unassigned', 'service__isnull', True))
    self.addPhysicians(pg)

    self.fields['filter'] = forms.ChoiceField()
    for i, f in enumerate(self.filters):
        self.fields['filter'].choices.append((i, f.name))

清除选项是可行的，但肯定会导致线程问题。

Answer 3

您将附加到 PER-CLASS 变量 self.filters。通过在 __init__ 的开头执行 self.filters = [] 将其变为 PER-INSTANCE 变量。

Answer 4

从其他一些答案中澄清一下：

字段是并且必须是类变量。他们通过元类来完成各种事情，这是定义它们的正确方法。

但是，您的 filters 变量不必是类 var。它可以很容易地成为一个实例变量——只需从类中删除定义并将其放入__init__。或者，也许更好的是，根本不让它成为一个属性——只是__init__ 中的一个本地变量。然后，不要附加到filters.choices，而是重新分配它。

def __init__(self, *args, **kwargs):
        super(forms.Form, self).__init__(*args, **kwargs)
        filters = []
        filters.append(PatientFilter('All'))
        filters.append(PatientFilter('Assigned', 'service__isnull', False))
        filters.append(PatientFilter('Unassigned', 'service__isnull', True))

        self.fields["filter"].choices = [(i, f.name) for i, f in enumerate(filters)]    

Answer 5

如上所述，您需要将过滤器初始化为实例变量：

def __init__(...):
    self.filters = []
    self.filters.append(...)
    # ...

如果你想了解更多关于 Form 类的工作原理，你应该阅读 Django wiki 中的这个页面：

• Model Creation and Initialization

它讨论了 Model 类的内部结构，但您会发现字段的一般设置与 Form 有点相似（减去数据库的内容）。这有点过时（2006 年），但我认为基本原则仍然适用。如果你是新手，元类的东西可能会有点混乱。