修改pager-taglib的两个bug

BUG1：在网页中输入一些中文的过滤信息，然后查询，一开始的结果是正确的，但是当点击分页链接想跳到别的页面的时候会出现乱码。
原因：在com.jsptags.navigation.pager包下面的PagerTag类里面有个addParam方法，他在进行编码处理的时候使用了已经被废除的java.net.URLEncoder.encode方法，该方法直接使用系统默认的编码来进行编码，系统默认的编码是gbk，而我们页面的编码是utf-8，这样就造成的乱码。
修复方法：

final void addParam(String name, String value) {
		//获取页面编码
		String encoding=pageContext.getResponse().getCharacterEncoding();
		try {
			if (value != null) {
//				name = java.net.URLEncoder.encode(name);
//				value = java.net.URLEncoder.encode(value);这两个方法会直接使用系统默认的编码来进行编码，系统默认的编码是gbk，而我们页面的编码是utf-8
				name = java.net.URLEncoder.encode(name,encoding);
				value = java.net.URLEncoder.encode(value,encoding);

				uri.append(params == 0 ? '?' : '&')
				   .append(name).append('=').append(value);

			   params++;

			} else {
				String[] values = pageContext.getRequest().getParameterValues(name);

				if (values != null) {
					name = java.net.URLEncoder.encode(name,encoding);
					for (int i = 0, l = values.length; i < l; i++) {
						value = java.net.URLEncoder.encode(values[i],encoding);
						uri.append(params == 0 ? '?' : '&')
						   .append(name).append('=').append(value);

						params++;
					}
				}
			}
		} catch (UnsupportedEncodingException e) {
			e.printStackTrace();
		}
	}

BUG2:当过滤结果为0的时候，lastpager的链接为负数。
原因：

package com.jsptags.navigation.pager;

public final class LastTag extends JumpTagSupport {

	protected int getJumpPage() {
		//这边没有作判断
		return (pagerTag.getPageCount() - 1);
	}
}

修复方法

package com.jsptags.navigation.pager;

public final class LastTag extends JumpTagSupport {

	protected int getJumpPage() {
		if (pagerTag.getPageCount()==0) {
			return 0;
		}
		return (pagerTag.getPageCount() - 1);
	}
}

评论

1 楼 lxh2002 2013-06-13  

很不错，一直在用这个分页库，当场是强制改成utf-8编码的，没有考虑可以获取页面编码。学习了，谢谢！