Django：从表格中保存外键

Question

这是this question的延续，其中我想弄清楚如何构建由经纬度FloatFields组成的PointField。我已@西蒙的建议和重组我的模型看起来像这样：

class Point(models.Model): 
    lat = models.FloatField() 
    lon = models.FloatField() 

class Thing(models.Model): 
    point = models.ForeignKey(Point) 

我的形式有相应的从谷歌地图经度和纬度值两个字段坐标：

class StepThreeForm(forms.Form): 
    lat = forms.FloatField() 
    lon = forms.FloatField() 
    ... 

然而，这并不明显的原因工作，但我不知道如何解决它。为了澄清，我试图有两个对应于外键值lat和lon的表单字段。这里是补充信息（我使用的是FormWizard和forms.Form）：

---

url(r'^mapform/$', login_required(MyWizard.as_view([StepOneForm, StepTwoForm, StepThreeForm])), name='create'), 

class MyWizard(SessionWizardView): ## this view also serves to edit existing objects and provide their instances 
    def done(self, form_list, **kwargs): 
     id = form_list[0].cleaned_data['id'] 
     try: 
      thing = Thing.objects.get(pk=id) 
      instance = thing 
     except: 
      thing = None 
      instance = None 
     if thing and thing.user != self.request.user: 
      raise HttpResponseForbidden() 
     if not thing: 
      instance = Thing() 
      for form in form_list: 
       for field, value in form.cleaned_data.iteritems(): 
        setattr(instance, field, value) 
      instance.user = self.request.user 
      instance.save() 
     return render_to_response('wizard-done.html', { 
       'form_data': [form.cleaned_data for form in form_list],}) 

我明白任何和所有的建议和帮助！

---

编辑：基于Yuji Tomita的输入更新。它大部分都很有意义（谢谢！），但我不确定它为什么会导致ValueError。

class MyWizard(SessionWizardView): 
    .... 
      for form in form_list: 
       form.save(instance) 
    ... 

class StepOneForm(forms.Form): 
    ... 
    def save(self, thing): 
     for field, value in self.cleaned_data.items(): 
      setattr(thing, field, value) 

class StepTwoForm(forms.Form): 
    ... 
    def save(self, thing): 
     for field, value in self.cleaned_data.items(): 
      setattr(thing, field, value) 

我相信，我应该保持表单字段为纬度和经度，因为我用我的形式的谷歌地图，并采取纬度和经度从选定的输入，然后从这些值建构一个点领域：

class StepThreeForm(forms.Form): 
    lat = forms.FloatField() 
    lon = forms.FloatField() 

    def save(self, thing): 
     thing.point = Point.objects.get_or_create(lat=self.cleaned_data.get('lat'), lon=self.cleaned_data.get('lon')) 

这将产生ValueError: Cannot assign "(<Point: Point object>, False)": "Thing.point" must be a "Point" instance.

Traceback: 
File "/lib/python2.7/django/core/handlers/base.py" in get_response 
    111.       response = callback(request, *callback_args, **callback_kwargs) 
File "/lib/python2.7/django/contrib/auth/decorators.py" in _wrapped_view 
    20.     return view_func(request, *args, **kwargs) 
File "/lib/python2.7/django/views/generic/base.py" in view 
    48.    return self.dispatch(request, *args, **kwargs) 
File "/lib/python2.7/django/contrib/formtools/wizard/views.py" in dispatch 
    223.   response = super(WizardView, self).dispatch(request, *args, **kwargs) 
File "/lib/python2.7/django/views/generic/base.py" in dispatch 
    69.   return handler(request, *args, **kwargs) 
File "/lib/python2.7/django/contrib/formtools/wizard/views.py" in post 
    286.     return self.render_done(form, **kwargs) 
File "/lib/python2.7/django/contrib/formtools/wizard/views.py" in render_done 
    328.   done_response = self.done(final_form_list, **kwargs) 
File "/myproject/myapp/forms.py" in done 
    93.  form.save(instance) 
File "/myproject/myapp/forms.py" in save 
    67.   thing.point = Thing.objects.get_or_create(lat=self.cleaned_data.get('lat'), lon=self.cleaned_data.get('lon')) 
File "/lib/python2.7/django/db/models/fields/related.py" in __set__ 
    366.         self.field.name, self.field.rel.to._meta.object_name)) 

Answer 1

我建议建立在你的每一个形成一个save方法，知道如何将自己保存到数据库。它遵循一种通用模式，即“表单通过form.save（）执行其操作”，因此应该直观地遵循。

底线是，现在你有一个毯子：“对于所有形式的每个领域，将Thing属性设置为这些领域”。

由于实际上你有每个表单的保存行为，我认为有必要将实例传递给每个表单，以便每个表单都有机会以适合其字段的方式保存数据。然后

class Form1(...): 
    def save(self, thing): 
     for field, value in self.cleaned_data.items(): 
      setattr(thing, field, value) 

class Form2(...): 
    def save(self, thing): 
     thing.point = Point.objects.get_or_create(lat=self.cleaned_data.get('lat'), long=...) 
     # note, you may not want get_or_create if you don't want to share points. 

你的观点将成为：

for form in form_list: 
    form.save(instance) 

只是一个想法。

如果你想更加干燥它和喜欢你的其他形式的自动化，我会建立具有已定义的保存方法的基本形式：

class BaseSaveBehaviorForm(forms.Form): 
    def save(self, thing): 
     for field, value in self.cleaned_data.items(): 
      setattr(thing, field, value) 

class NormalBehaviorForm(BaseSaveBehaviorForm): 
    # your forms as usual 


class SpecialSaveBehaviorForm(forms.Form): 
    def save(self, instance): 
     # do something unusual