重构“视图”代码

Question

我的“视图”代码在其代码中有明显的重复。有没有办法重构“返回”代码？

def form_contractor_view(request):
    if request.method == 'POST':
        form = ContractorForm(request.POST)
        if form.is_valid():
            form.save()
            return redirect('index_view')
        else:
            return render_to_response(
                'form_contractor.html',
                {'form': form},
                context_instance=RequestContext(request),
            )
    else:
        form = ContractorForm()
        return render_to_response(
           'form_contractor.html',
           {'form': form},
           context_instance=RequestContext(request),
        )

Answer 1

根本不需要第一个else。将最终渲染移回缩进，这将捕获 else 情况。请注意，这是文档中明确描述的模式。

另外，使用render 代替render_to_response。

def form_contractor_view(request):
    if request.method == 'POST':
        form = ContractorForm(request.POST)
        if form.is_valid():
            form.save()
            return redirect('index_view')
    else:
        form = ContractorForm()
    return render(
       request,
       'form_contractor.html',
       {'form': form},
    )

Answer 2

我会努力的：

def form_contractor_view(request):
    form = ContractorForm(request.POST or None)
    if form.is_valid():
        form.save()
        return redirect(reverse('index_view')) # <-- you forgot reverse() here
    return render(
        request,
        'form_contractor.html',
        {'form': form}
    )

我用render 替换了render_to_response，它做同样的事情——你只需少打字……尽量偷懒；）

我认为，这行得通。虽然没有测试。如果请求未在 POST 中完成，则 is_valid() 返回 False 将 form 作为 None 再次返回，这是有道理的。对于请求在 POST 中完成且其中包含无效值的情况，然后else 启动并返回带有错误消息的表单...我认为，这可行..请测试