如何以线程安全的方式为新用户分配递增的用户ID?

时间:2012-09-25 15:59:13

标签: java servlets

我刚刚开始学习Java,我在创建Java服务器时遇到了问题。

在HTML上,应该有一个表单,JavaScript会每秒提交一次表单(按POST方法)。表单应该有一个隐藏字段,其中包含用户ID的值。服务器应在请求中查找用户标识。它应该为新用户提供新ID,并向访问过的用户显示他们的ID。

每个表单提交后,题字更改为:null,2,null,2,null,2 ....这里是代码:

public class ServerConnect extends AbstractHandler{
         private AtomicInteger ids = new AtomicInteger(0);    
         private String userId;

         public void handle(String target,
         Request baseRequest,
         HttpServletRequest request,
         HttpServletResponse response)    throws IOException, ServletException
         {

             response.setContentType("text/html;charset=utf-8");
             response.setStatus(HttpServletResponse.SC_OK);
             baseRequest.setHandled(true);
             response.getWriter().println(PageGenerator.generateForm(userId));
             userId = request.getParameter("userId");

             if (userId == null){
                 ids.getAndIncrement();
                 userId = ids.toString();
             }        
          }

         public static void main(String[] args) throws Exception {

               Server server = new Server(8080);
               server.setHandler(new ServerConnect());
               server.start();
               server.join();
        }
}

public class PageGenerator {

    public static String generateForm(String val){
        String htmlCode = null;

        htmlCode = "...
             <script>
                function reload(){ 
                   document.forms['MainForm'].submit();
                }
                setTimeout("reload()", 1000); 
             </script>    
             .... + val + ...
             <form id='MainForm' method = 'POST'>
                 <input type = "hidden" name = "userId" value = \""+ val + \""> 
             <form>
                             ...       
    return htmlCode;
    }    

}

我无法理解为什么用户ID等于2,以及为什么在用户获得他的id之后,它的值变为null,然后再次变为2

1 个答案:

答案 0 :(得分:3)

您似乎在解析userId之前渲染了您的网页。在您检查PageGenerator.generateForm(userId)

之前,您已if(userId == null)

此外,您应该理解AtomicLong.getAndIncrement()是一个复合操作,类似于i++(但是是一个原子操作)。

所以你需要有如下代码:

if (userId == null){
     userId = ids.getAndIncrement();
} 

最后,我不确定AbstractHandler的生命周期,但假设您的类的新实例每隔一个请求由Jetty实例化是合理的。由于ids的范围是“实例”字段,因此它会被新请求丢弃(从而有效地重置)。

关于Jetty线程模型的

编辑答案:Jetty Architecture

要解决此问题,您需要将ids声明为static final(从而有效地使其成为“Singleton”ID生成器)以使其在多个请求中存活。玩具应用程序的正确方法,但不是生产代码中必须做的事情。