升级至Java1.8+Tomcat8后ApplicationHttpRequest的getParameter行为异常

我之前升级Java 8+Tomcat 8的时候也踩过这个坑！当时用RequestDispatcher转发请求到另一个Servlet，突然发现之前判断参数是否存在的逻辑全出问题了——原来Tomcat 8在处理内部转发请求时，会把原始HttpServletRequest包装成它的内部类ApplicationHttpRequest，而这个类的getParameter方法和Tomcat 7里的标准实现有个关键差异：

当请求中完全不存在目标参数（比如你说的param1）时，Tomcat 7里的HttpServletRequest.getParameter("param1")会返回null，但ApplicationHttpRequest的同名方法会返回空字符串（""）。

这就导致原来依赖if (param == null)判断参数是否存在的代码，现在会误以为参数存在但值为空，进而引发各种奇怪的业务逻辑错误。

几个可行的解决方案： #

• 调整代码判断逻辑：把所有判断参数是否存在的地方，从if (param == null)改成if (param == null || param.isEmpty())。如果项目里用了Apache Commons Lang或者Spring的工具类，直接用StringUtils.isBlank(param)会更简洁可靠。
• 自定义请求包装类兼容旧行为：如果不想大面积修改代码，可以写一个HttpServletRequestWrapper的子类，覆盖getParameter方法，强制把空字符串转成null，保持和Tomcat 7一致的行为：

  public class Tomcat7CompatibleRequestWrapper extends HttpServletRequestWrapper {
      public Tomcat7CompatibleRequestWrapper(HttpServletRequest request) {
          super(request);
      }
  
      @Override
      public String getParameter(String name) {
          String value = super.getParameter(name);
          // 空字符串转为null，对齐Tomcat7逻辑
          return (value != null && value.trim().isEmpty()) ? null : value;
      }
  
      // 如果需要的话，也可以覆盖getParameterValues等方法，保持一致性
  }
  

  然后在转发请求的时候用这个包装类：

  RequestDispatcher dispatcher = request.getRequestDispatcher("/targetServlet");
  dispatcher.forward(new Tomcat7CompatibleRequestWrapper(request), response);
  

• 检查转发逻辑是否必要：如果是自己项目里的Servlet调用，能不能改成直接调用业务方法，而不是通过请求转发？这样可以绕开Tomcat的内部请求包装逻辑，从根源上避免这个问题。

这个问题本质上是Tomcat 8对Servlet请求转发的内部实现调整，ApplicationHttpRequest是Tomcat的私有内部类，官方文档不会特意说明这种细节差异，所以升级的时候很容易忽略。我当时排查了好半天，才发现是这个包装类搞的鬼😂

内容的提问来源于stack exchange，提问作者Dion Puggs Chapman