Обработка formset | Forum | Django на русском

18 June 2013, 16:26

stol_n

Posts: 30

18 June 2013, 16:26

stol_n

Posts: 30

Правильно ли я обрабатываю formset или есть более легкие(дзен) способы?

if formset.is_valid() and formset.has_changed(): # проверяем была ли изменена форма
                        for i in range(0, formset.total_form_count()):
                            form_sportsmen = formset.forms[i]
                            cleaned_data2 = form_sportsmen.cleaned_data
                            try:
                                Participant_sportsmen.objects.sportsmen_add(now_delegation,
                                    cleaned_data2['id_sportsmen']
                                )
                            except:
                                pass

Еще напрягает, то что под метод cleaned_data попадают пустые формы и их приходится прогонять через конструкцию try...except

Answer | 0

18 June 2013, 16:36

alerion

Posts: 5750
Location: Львов

18 June 2013, 16:36

alerion

Posts: 5750
Location: Львов

try...except уже указывает что что-то не так. Вообще никогда не используйте его выполняя pass для обработки ошибки. Вот как-то так:

    form = self.get_form(form_class)

    if form.is_valid():
        form_is_valid = True
        self.object = form.save(commit=False)
    else:
        form_is_valid = False

    formset = self.formset_class(request.POST, instance=self.object or self.model())

    if formset.is_valid() and form_is_valid:
        self.object.save()
        formset.save()
        return HttpResponseRedirect(self.get_success_url())
    else:
        context = self.get_context_data(form=form)
        context['formset'] = formset
        return self.render_to_response(context)

Фиксю баги по трейсбеку.

Answer | 0

18 June 2013, 16:37 alerion Posts: 5750 Location: Львов	18 June 2013, 16:37 alerion Posts: 5750 Location: Львов formset типа самом может сохранять сразу данные. Там же есть фабрика формсетов для ModelForm. Смотрел в исходниках Django, понятнее чем в доке. Фиксю баги по трейсбеку.
	Answer \| 0

18 June 2013, 17:23 stol_n Posts: 30	18 June 2013, 17:23 stol_n Posts: 30 т.е. как я понял form_class это произвольный class из forms.py? например в моем случае - sportsmenAdd. а formset_class в моем случае это - `SportsmenFormSet = formset_factory(sportsmenAdd)`
	Answer \| 0

18 June 2013, 18:00 stol_n Posts: 30	18 June 2013, 18:00 stol_n Posts: 30 не могу разобрать какой контекст несет в себе self
	Answer \| 0

18 June 2013, 18:14 stol_n Posts: 30	18 June 2013, 18:14 stol_n Posts: 30 Поискал по сайту нарыл инфу, разобрался, пойду код переписывать. Глупости человеческой нет предела(с).
	Answer \| 0

18 June 2013, 18:43

alerion

Posts: 5750
Location: Львов

18 June 2013, 18:43

alerion

Posts: 5750
Location: Львов

На self не смотрите. Это я юзал CBV, просто логика как работать с формсет в представлении. Ничего там сохранять самостоятельно не нужно, это кривой подход. Оно само умеет.

Фиксю баги по трейсбеку.

Answer | 0

18 June 2013, 19:31 stol_n Posts: 30	18 June 2013, 19:31 stol_n Posts: 30 почитал топик отсюда http://djbook.ru/forum/topic/1052/#post-6622 , разобрался, понял что надо использовать modelformset_factory, а не formset_factory, теперь буду почаще заглядывать в учебник по python и исходники django
	Answer \| 0

19 June 2013, 0:54 Andrew_Lvov Posts: 102	19 June 2013, 0:54 Andrew_Lvov Posts: 102 stol_n, хватит себя критиковать. Все мы постоянно учимся. formset_factory тоже подходит, если есть форма.
	Answer \| 0