如何破坏SelectionKey附件? attach(null)不起作用

时间:2013-07-15 10:33:48

标签: java tcp memory-leaks nio

我的java应用程序有内存泄漏 - 当我的资源清除代码正在执行时,任务管理器显示内存使用情况没有改变。我的代码

while (isRunning) {
      try 
      {

             selector.select(); 

             long sum=0;

             Set keys =  selector.selectedKeys();
             Iterator it = keys.iterator(); 
             while(it.hasNext())
             {
                 SelectionKey key = (SelectionKey)it.next();
                 if (key.isReadable())
                 { 
                      SocketChannel sc = (SocketChannel) key.channel();

                      ByteBuffer bb;
                      if(key.attachment()==null)
                      {
                          bb = ByteBuffer.allocate(1024*1024);
                          key.attach(bb);
                      }
                      else
                      {
                          bb = (ByteBuffer)key.attachment();
                          bb.clear();
                      }

                      int x =  sc.read(bb); 

                       System.out.println(x +" bytes were read");
                       if(x==-1)
                       {
                           key.attach(null); //doesn't work
                           sc.close();
                           //bb = null; // also doesn't work
                       }
                 } 
             }
           keys.clear();
      } 
      catch (Exception ex) 
      {     
          ex.printStackTrace(new PrintStream(System.out));
      }
      finally
      { 
            //stopServer();
      }
    }

测试逻辑 - 我编写了简单的TCP客户端java程序,向服务器发送100条消息。我故意为每个连接分配大缓冲区 - 1MB。当客户端完成其作业int x = sc.read(bb);返回-1并执行以下代码时:

if(x==-1)
                           {
                               key.attach(null); //doesn't work
                               sc.close();
                               //bb = null; // also doesn't work
                           }

我用调试输出检查了它,这段代码真的被执行但是任务管理器仍然显示大量的内存使用量。问题出在哪里?

1 个答案:

答案 0 :(得分:1)

当然key.attach(null)有效。如果没有,附加非null对象也不起作用。相同的代码。

但是,在任何情况下,关闭SocketChannel会取消该密钥,这会将其从已注册的所有Selectors的所有密钥集中删除,因此您无论如何都不会再次看到该密钥,因此无论您是否拨打key.attach(null),它都符合GC的条件,附件也是如此,因此这是多余的。要么您在其他地方有另一个对附件的引用,要么您的内存使用问题在其他地方。