Kiedy należy używać user.get_profile w django?

Question

Widziałem inną odpowiedź here i inne miejsca w sieci, które zalecają używanie user.get_profile podczas rozszerzania wbudowanego użytkownika django. Nie zrobiłem tego w poniższym przykładzie. Wydaje się, że funkcjonalność działa dobrze, ale czy jest jakiś minus, jeśli nie używasz user.get_profile()?

modelu

class UserProfile(models.Model): 
    user = models.ForeignKey(User, primary_key=True) 
    quote = models.CharField('Favorite quote', max_length = 200, null=True, blank=True) 
    website = models.URLField('Personal website/blog', null=True, blank=True) 

class UserProfileForm(ModelForm): 
    class Meta: 
     model = UserProfile 
     fields = ('quote', 'website') 

widok

@login_required 
def user_profile(request): 
    user = User.objects.get(pk=request.user.id) 
    if request.method == 'POST': 
     upform = UserProfileForm(request.POST) 
     if upform.is_valid(): 
      up = upform.save(commit=False) 
      up.user = request.user 
      up.save() 
      return HttpResponseRedirect('/accounts/profile') 
    else: 
     upform = UserProfileForm() 
    return render_to_response('reserve/templates/edit_profile.html', locals(), context_instance=RequestContext(request)) 

Answer 1

Kod działa jak napisałeś, ale dlatego, że nie przechodzą instancję do modelu to trochę nietypowa, więc to może poproś innego programistę Django o trochę dłużej, aby dowiedzieć się, co się dzieje.

Widok, który można utworzyć, aby utworzyć instancję modelu z instancją, aby istniejące wartości profilu były wyświetlane w formularzu. W twoim przypadku dostaniesz puste pola.

upform = UserProfileForm(instance=user.get_profile()) 

Ponieważ nie przewidują wystąpienie, oszczędzając spróbuje utworzyć nowy user_profile, których nie chcemy. Tak się nie stanie w twoim przypadku, ponieważ klucz podstawowy został user, ale to też trochę niezwykłe.

Główną zaletą pisania user.get_profile() jest to, że nie trzeba wiedzieć, który model jest używany dla profilu użytkownika. Jeśli jesteś zadowolony z kodu twardego UserProfile w swoim kodzie, możesz zamiast tego wstawić instance=UserProfile.objects.get(user=user).