首先,search()视图对于空字符串的处理相当薄弱——仅显示一条”Please submit a search term.”的提示信息。 若用户要重新填写表单必须自行点击“后退”按钮, 这种做法既糟糕又不专业。如果在现实的案例中,我们这样子编写,那么Django的优势将荡然无存。
在检测到空字符串时更好的解决方法是重新显示表单,并在表单上面给出错误提示以便用户立刻重新填写。 最简单的实现方法既是添加else分句重新显示表单,代码如下:
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
|
from django.http import HttpResponse
from django.shortcuts import render_to_response
from mysite.books.models import Book
def search_form(request):
return render_to_response( 'search_form.html' )
def search(request):
if 'q' in request.GET and request.GET[ 'q' ]:
q = request.GET[ 'q' ]
books = Book.objects. filter (title__icontains = q)
return render_to_response( 'search_results.html' ,
{ 'books' : books, 'query' : q})
else :
* * return render_to_response( 'search_form.html' , { 'error' : True }) * *
|
(注意,将search_form()视图也包含进来以便查看)
这段代码里,我们改进来search()视图:在字符串为空时重新显示search_form.html。 并且给这个模板传递了一个变量error,记录着错误提示信息。 现在我们编辑一下search_form.html,检测变量error:
1
2
3
4
5
6
7
8
9
10
11
12
13
14
|
<html>
<head>
<title>Search< / title>
< / head>
<body>
* * { % if error % } * *
* * <p style = "color: red;" >Please submit a search term.< / p> * *
* * { % endif % } * *
<form action = "/search/" method = "get" >
< input type = "text" name = "q" >
< input type = "submit" value = "Search" >
< / form>
< / body>
< / html>
|
我们修改了search_form()视图所使用的模板,因为search_form()视图没有传递error变量,所以在条用search_form视图时不会显示错误信息。
通过上面的一些修改,现在程序变的好多了,但是现在出现一个问题: 是否有必要专门编写search_form()来显示表单? 按实际情况来说,当一个请求发送至/search/(未包含GET的数据)后将会显示一个空的表单(带有错误信息)。 所以,只要我们改变search()视图:当用户访问/search/并未提交任何数据时就隐藏错误信息,这样就移去search_form()视图以及对应的URLpattern。
1
2
3
4
5
6
7
8
9
10
11
12
|
def search(request):
error = False
if 'q' in request.GET:
q = request.GET[ 'q' ]
if not q:
error = True
else :
books = Book.objects. filter (title__icontains = q)
return render_to_response( 'search_results.html' ,
{ 'books' : books, 'query' : q})
return render_to_response( 'search_form.html' ,
{ 'error' : error})
|
在改进后的视图中,若用户访问/search/并且没有带有GET数据,那么他将看到一个没有错误信息的表单; 如果用户提交了一个空表单,那么它将看到错误提示信息,还有表单; 最后,若用户提交了一个非空的值,那么他将看到搜索结果。
最后,我们再稍微改进一下这个表单,去掉冗余的部分。 既然已经将两个视图与URLs合并起来,/search/视图管理着表单的显示以及结果的显示,那么在search_form.html里表单的action值就没有必要硬编码的指定URL。 原先的代码是这样:
1
|
< form action = "/search/" method = "get" >
|
现在改成这样:
1
|
< form action = "" method = "get" >
|
action=”“意味着表单将提交给与当前页面相同的URL。 这样修改之后,如果search()视图不指向其它页面的话,你将不必再修改action。