8

例如,我有一个将属性设置为 HttpServletRequest 的 servlet 代码:

request.setAttribute("someValue", someValue());
        RequestDispatcher rd = getServletContext().getRequestDispatcher("/SomeJsp.jsp");
        rd.forward(this.request, this.response);
        return;

如何确保上面的代码是线程安全的?

这是我得到的堆栈跟踪:

java.lang.NullPointerException
    at org.apache.catalina.connector.Request.notifyAttributeAssigned(Request.java:1552)
    at org.apache.catalina.connector.Request.access$000(Request.java:105)
    at org.apache.catalina.connector.Request$3.set(Request.java:3342)
    at org.apache.catalina.connector.Request.setAttribute(Request.java:1504)
    at org.apache.catalina.connector.RequestFacade.setAttribute(RequestFacade.java:541)
    at org.apache.catalina.core.ApplicationHttpRequest.setAttribute(ApplicationHttpRequest.java:281)
    at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:286)
    at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:210)
    at org.apache.catalina.core.ApplicationDispatcher.invoke(ApplicationDispatcher.java:684)
    at org.apache.catalina.core.ApplicationDispatcher.processRequest(ApplicationDispatcher.java:471)
    at org.apache.catalina.core.ApplicationDispatcher.doForward(ApplicationDispatcher.java:402)
    at org.apache.catalina.core.ApplicationDispatcher.forward(ApplicationDispatcher.java:329)
    at com.mycompany.myapp.servlet.SomeServlet.doRequest(SomeServlet.java:103)
    at com.mycompany.myapp.servlet.SomeServlet.doGet(SomeServlet.java:159)
4

5 回答 5

11
rd.forward(this.request, this.response);

这(双关语)表明您已将requestresponse作为类的实例变量。您的具体问题反过来表明该类本身的实例不是线程安全的。

假设它实际上是 servlet 本身,那么您就有了问题的原因。Servlet 根本不是线程安全的。在 webapp 启动期间只创建了一个实例,然后在应用程序范围内的所有请求中共享。

永远不应将请求或会话范围的数据分配为 servlet 的实例变量。它只会在同时发生另一个 HTTP 请求时被覆盖。正如您自己遇到的那样,这将使您的代码线程不安全。

这里有一些代码说明了这一点:

public class MyServlet extends HttpServlet {

    private Object thisIsNOTThreadSafe;

    protected void doGet(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
        Object thisIsThreadSafe;

        thisIsNOTThreadSafe = request.getParameter("foo"); // BAD!! Shared among all requests!
        thisIsThreadSafe = request.getParameter("foo"); // OK, this is thread safe.
    } 
}

将 HTTP 请求本身分配为 servlet 的实例变量实际上是一个巨大的错误。当用户 Y 在处理用户 X 的请求的同时触发另一个请求时,用户 X 将立即获得用户 Y 的requestresponse对象。这绝对是线程不安全的。导致 NPE 是因为request此时对用户 Y 的处理“完成”并因此被释放/销毁。

也可以看看:

于 2012-12-20T11:57:32.243 回答
2

根据定义,请求是线程安全的(与 Session 和 ServletContext 不同)。

关于例外:您使用的是什么 Tomcat 版本?这看起来像一个 Tomcat 错误。

作为方法返回类型的类是否someValue()实现HttpAttributeBindingListener?而且,someValue()方法可以返回null吗?如果两者都是,那么NullPointerException很明显。

于 2012-12-20T11:35:51.233 回答
1

不使用此运算符

     RequestDispatcher rd = getServletContext().getRequestDispatcher("/SomeJsp.jsp");
    rd.forward(request, response);
    return;
于 2012-12-21T09:32:19.623 回答
1

我不确定这与线程安全有什么关系。

您得到的异常是 NullPointerException,看起来 Tomcat 正在尝试调用空对象上的方法。

在 Java Web 应用程序中,每个请求都有自己的 HttpServletRequest 实例,因此您可以设置请求的属性并确信它仅适用于该用户。

于 2012-12-20T11:36:40.680 回答
0

如果你在 tomcat 中查看请求接口的实现,它看起来像下面的代码。

public void setAttribute(String name, Object value) {

    // Name cannot be null
    if (name == null)
        throw new IllegalArgumentException
            (sm.getString("coyoteRequest.setAttribute.namenull"));

    // Null value is the same as removeAttribute()
    if (value == null) {
        removeAttribute(name);
        return;
    }

    if (name.equals(Globals.DISPATCHER_TYPE_ATTR)) {
        internalDispatcherType = (DispatcherType)value;
        return;
    } else if (name.equals(Globals.DISPATCHER_REQUEST_PATH_ATTR)) {
        requestDispatcherPath = value;
        return;
    }

可以清楚地看到,如果 key 为 null,那么它会抛出IllegalArgumentException,但如果 value 为null,那么它只是从存储库中删除该 key 和旧的关联对象。但是如果它们都不是的,它会将对象与该键相关联并将它们添加到存储库中。

这似乎是一个临时或 tomcat 问题。

有关实现的更多信息,请参阅下面的链接 Source code of Request Implementation by tomcat

于 2012-12-21T06:20:04.807 回答