重构“查看”代码

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

有没有必要的首先是其他所有。将最终的渲染移回缩进，并且会捕获其他情况。请注意，这是文档中明确描述的模式。另外，使用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_to_response与render它做同样的事情 - 你只需要输入少...尝试偷懒;）

我想，这有效。没有经过测试。如果请求没有在POST中完成，那么is_valid()返回False，将form返回为None，这是有道理的。并在POST请求和无效值的情况下，然后else踢和返回错误消息的形式...我认为，这个工程..请测试