6

我InheritableThreadLocal在我的Servlet课堂上使用。这样它就可以从它的子线程中获得。InheritableThreadLocal在线程池执行程序中使用是邪恶的吗？. 比如servlet线程池。

我的问题。

1) 为什么我们要避免InheritableThreadLocals在 servlet 中使用？

2）这种内存泄漏是否可能发生在InheritableThreadLocal？

3) 有什么替代方法InheritableThreadLocal吗？

4）如果线程被重用，存储的值threadlocal不会被清除会怎样？

我的实时场景

public class UserAccessFilter implements javax.servlet.Filter {

      static final InheritableThreadLocal<String> currentRequestURI = new InheritableThreadLocal<String>();

      public void  doFilter(ServletRequest req, ServletResponse resp , FilterChain fc) throws IOException, ServletException{
              String uri = request.getRequestURI();
              fc.doFilter(request, response);         
      }
}


public class MailServlet extends HttpServlet{

      @Override
protected void doGet(HttpServletRequest req, HttpServletResponse resp) throws ServletException, IOException {
    String mailCount = req.getParameter("mailCount");

    if(mailCount != null && !"".equals(mailCount) && mailCount.matches("[0-9]+")){
        MailThread mailThread = new MailThread("xxx@gmail.com", generateToAddress(Integer.parseInt(mailCount))); //NO I18N
        Thread t = new Thread(mailThread);
        t.start();
    }

    resp.getWriter().println("Mail Servlet.............."); //NO I18N

}              

}

class MailThread implements Runnable{

private String from;
private String to;

public MailThread(String from , String to){
    this.from = from;
    this.to = to;
}


@Override
public void run() {
    sendMail();
}

    public void sendMail(){
        //I want this uri from child threads. I can't pass this value to this constructor.
        String uri = currentRequestURI.get();
        //Do Mail Operations
    }


}

过滤器 --> Servlet A --> 子线程 ---> 邮件线程（这里我得到了过滤器中设置的值）。

4

2 回答 2

15

为什么我们应该避免在 servlet 中使用 InheritableThreadLocals？

它们代表了将信息从一个请求泄漏到另一个请求的潜在路径。“问题”是请求由线程池处理。当请求完成时，线程处理的下一个请求很可能是针对不同用户的。但是，如果您忘记在完成第一个请求时清除线程本地状态，则它可能会被第二个请求使用。

InheritableThreadLocal 中是否可能存在内存泄漏？

是的……有点。如果我们假设工作池是有界的，那么任何线程的线程本地状态都有可能被覆盖，从而清除内存泄漏。在最坏的情况下，问题是有限的内存泄漏......受池中线程数的限制。

信息泄露问题更令人担忧。

InheritableThreadLocal 有什么替代方案吗？

在请求或响应对象中设置属性会更好。

如果线程被重用会发生什么，threadlocal中存储的值不会被清除。

它不会被清除。那就是问题所在！

于 2013-01-24T10:20:10.357 回答
0

您的示例有效，MailThread 继承currentRequestURI了创建时的值。

但是UserAccessFilter和java.lang.InheritableThreadLocal只是用来混淆代码试图做什么，这是邪恶的部分。

更改构造函数有什么问题，MailThread以便您可以MailServlet像这样传递请求 URI：

MailThread mailThread = new MailThread("xxx@gmail.com", generateToAddress(Integer.parseInt(mailCount)), req.getRequestURI());

那么就不需要本地线程，不需要过滤器，代码更清晰。

于 2013-01-24T10:19:14.850 回答