Функция просмотра Джанго


У меня есть функция просмотра, что позволяет пользователю добавлять / редактировать / удалять объект. Как выполняет следующие функции посмотрите, и какими способами я могу улучшить его? Я думал о разделении аспектов функционирования в разных частях, но так как нет 'компонент' больше, чем четыре-пять строк кода, я думал, что будет немного перебор.

@login_required
def edit_education(request, edit=0):
    profile = request.user.get_profile()
    education = profile.education_set.order_by('-class_year')
    form = EducationForm(data=request.POST or None, request=request)

    if request.method == 'POST':

        ##### to add a new school entry ######
        if 'add_school' in request.POST:
            if form.is_valid():
                new_education = form.save(commit=False)
                new_education.user = profile
                new_education.save()
                return redirect('edit_education')

        ##### to delete a school entry #####
        for education_id in [key[7:] for key, value in request.POST.iteritems() if key.startswith('delete')]:
            Education.objects.get(id=education_id).delete()
            return redirect('edit_education')

        ###### to edit a school entry -- essentially, re-renders the page passing an "edit" variable #####
        for education_id in [key[5:] for key, value in request.POST.iteritems() if key.startswith('edit')]:
            edit = 1
            school_object = Education.objects.get(id = education_id)
            form = EducationForm(instance = school_object, request=request)
            return render(request, 'userprofile/edit_education.html', 
                        {'form': form, 
                         'education':education, 
                         'edit': 1, 
                         'education_id': education_id}
                    )
        ##### to save changes after you edit a previously-added school entry ######
        if 'save_changes' in request.POST:
            instance = Education.objects.get(id=request.POST['education_id']) 
            form = EducationForm(data = request.POST, instance=instance, request=request, edit=1)
            if form.is_valid():
                form.save()
                return redirect('edit_education')

return render(request, 'userprofile/edit_education.html', {'form': form, 'education': education})

И в моем шаблоне, если это поможет что-нибудь прояснить:

{% for education in education %}
    <p><b>{{ education.school }}</b> {% if education.class_year %}{{ education.class_year|shorten_year}}, {% endif %} {{ education.degree}}
        <input type="submit" name="edit_{{education.id}}" value='Edit' />
        <input type="submit" name="delete_{{education.id}}" value="Delete" />
    </p>
{% endfor %}


274
2
задан 19 июля 2011 в 02:07 Источник Поделиться
Комментарии
2 ответа

Если условия могут быть упрощены.

fpv_list = [ item.partition("_") for item in request.POST.iteritems() ]
deletes = [ v for f,p,v in fpv_list if f == 'delete' ]
for education_id in deletes:
whatever

edits = [ v for f,p,v in fpv_list if f == 'edit' ]
for education_id in edits:
whatever

Это устраняет зависимость от зная длину 'удалить' и длиной 'редактирование' префиксы на кнопках.

4
ответ дан 19 июля 2011 в 01:07 Источник Поделиться

Эта процедура слишком длинная. Я никогда не поверю, ты перестанешь с такой логикой. Рано или поздно вы решите добавить что-то еще здесь, а потом этот кусок кода станет еще больше. У вас есть 4 абсолютно отличные блоки есть, так почему бы вам просто не разделить их на 4 отдельных процедур, оставляя только более высокого уровня логики в оригинальной edit_education?

3
ответ дан 19 июля 2011 в 07:07 Источник Поделиться