解析Tomcat 6、7在EL表达式解析时存在的一个Bug

时间:2021-09-15 00:15:18

今天在做数据分页显示的时候遇到了一个问题,经过测试,证实是Tomcat 6的一个bug,我所用的版本为:apache-tomcat-6.0.36,和7.0.30均能复现。下面详细描述一下这个bug:

该bug是在JSTL<c:forEach>标签中发现的,后来分析是EL表达式实现时产生的问题。jsp页面中有一个list需要遍历,这个list的类型为ArrayList<String>,我在其中放置的数据为(为方便我写成数组的形式):["1","...","4","5","6","7","8","...","10"],这是一个很常见的带页码缩略的分页导航。在展示这些数据的时候我使用了下面的代码:

复制代码代码如下:

<c:forEach var="looper" items="${pageHelper.pageList}">
 <c:choose>
  <c:when test="${looper eq pageHelper.pageDot}">
  <p>分页游标的 点点点</p>
  </c:when>
  <c:when test="${looper eq pageHelper.pageNo}">
  <p>当前页为第${looper}页面</p>
  </c:when>
  <c:otherwise>
  <p>分页游标:${looper}</p>
  </c:otherwise>
 </c:choose>
</c:forEach>


这里pageHelper就是分页组件,其中预设了pageDot为"...",pageNo为当前的页码(假设为6),其他情况直接显示分页游标。在循环遍历中只不过使用了最基本的条件判断语句,由于pageList在定义中已经明确指出是List<String>,按逻辑应该eq是按照字符串判断的,但是居然出异常了: 

复制代码代码如下:

javax.el.ELException: Cannot convert ... of type class java.lang.String to class java.lang.Long


为什么会出现“类型转换错误”呢?通过分析代码走向,当进入循环后,list中的第一条数据是“1”,而pageHelper.pageNo为long型,此时tomcat的EL表达式解析器会把looper类型转换为Long型而不是把pageHelper.pageNo类型转换为String进行比较,当遍历到下一元素时,looper="...",这时looper的类型已经确定,比较的时候tomcat还要试图将looper转换为Long类型,于是就出错了。

 

为此我专门写了一个实例代码:

复制代码代码如下:

<c:forEach var="looper" items="${pageHelper.pageList}">
 <c:choose>
  <c:when test="${looper eq fn:trim(pageHelper.pageDot)}">
  <p>分页游标的 点点点</p>
  </c:when>
  <c:when test="${looper eq fn:trim(pageHelper.pageNo)}">
  <p>当前页为第${looper}页面</p>
  </c:when>
  <c:otherwise>
  <p>分页游标:${looper}</p>
  </c:otherwise>
 </c:choose>
</c:forEach>


很简单,每次比较的时候都把后者用fn:trim方法进行去除左右非可见字符。相当于强制转换为String类型,此时tomcat又可以正常解析代码,并未报错。

 

同样的一套代码,我将其部署到resin中发现无论是修改前还是修改后都能正常运行,可见,应该是tomcat的bug。

示例代码:点击下载

让tomcat报错的演示地址:/bug/show.do

避免此bug的方法演示地址:/bug/avoid.do

以上地址前可能需要加上项目名称(具体取决于你如何部署该项目)