近期开发项目,由于后台需要为移动端的提供数据接口,我得意在项目中开发了一组数据接口,json格式的。因为所有的接口,都是我自己写的,编码风格都比较统一,所以我就使用了filter过滤器,进行所所有的接口进行登陆验证过滤。但是自己太马虎了,也有可能是自己很久没有写java代码了。过滤器的业务处理些的很潦草。如下是我写的过滤器:
@Override
public void doFilter(ServletRequest servletRequest, ServletResponse servletResponse, FilterChain filterChain)
throws IOException, ServletException {
HttpServletRequest request = (HttpServletRequest) servletRequest;
HttpServletResponse response = (HttpServletResponse) servletResponse;
String requestURI = request.getRequestURI();
if(!requestURI.contains("/json/jsonOrderFood") && !requestURI.contains("/json/jsonLogin") && !requestURI.contains("/json/jsonRegister")){
HttpSession session = request.getSession();
SUser sUser = (SUser) session.getAttribute("userInfo");
if (sUser == null) {
response.setCharacterEncoding("UTF-8");
response.setContentType("application/json; charset=utf-8");
JSONObject res = new JSONObject();
res.put("result","error");
PrintWriter writer = response.getWriter();
writer.append(res.toString());
return;
} else {
filterChain.doFilter(request, response);
}
}
filterChain.doFilter(request, response);
}
看到了吧,上面的代码就会出现问题,会出现我请求路径对应的方法被调用两次。原因是filterChain.doFilter(request, response); 会执行两次,就形成了两个分支。所以就调用了两次。说实话,我印象中,记得filter只要处理完毕放行就可以了,就是没有想到放行的次数还会有影响。下面就改后的代码。
@Override
public void doFilter(ServletRequest servletRequest, ServletResponse servletResponse, FilterChain filterChain)
throws IOException, ServletException {
HttpServletRequest request = (HttpServletRequest) servletRequest;
HttpServletResponse response = (HttpServletResponse) servletResponse;
String requestURI = request.getRequestURI();
if(!requestURI.contains("/json/jsonOrderFood") && !requestURI.contains("/json/jsonLogin") && !requestURI.contains("/json/jsonRegister")){
HttpSession session = request.getSession();
SUser sUser = (SUser) session.getAttribute("userInfo");
if (sUser == null) {
response.setCharacterEncoding("UTF-8");
response.setContentType("application/json; charset=utf-8");
JSONObject res = new JSONObject();
res.put("result","error");
PrintWriter writer = response.getWriter();
writer.append(res.toString());
return;
}
}
filterChain.doFilter(request, response);
}
看来有时间真的复习下基础的东西了。