我已经“修复”了内存泄漏，但是..如何以更好的方式修复它？

Question

这是一个非常快速且临时的错误修复。 它有效，但我想找到更好的理解和解决方案。

这是生成泄漏的类构造

final transient  DataInputStream din;
final transient  DataOutputStream dout;
final transient  BufferedReader bin;
final transient  BufferedWriter bout;

NData(Socket sock0) throws IOException
    {
        sock=sock0;
        din= new DataInputStream(sock.getInputStream());
        dout = new DataOutputStream(sock.getOutputStream());
        bin = new BufferedReader(new InputStreamReader(din));
        bout = new BufferedWriter(new OutputStreamWriter(dout));
     //   .. 
    }

函数错误修复是更改它（删除最终），以便让我稍后分配 null

transient  DataInputStream din;
transient  DataOutputStream dout;
transient  BufferedReader bin;
transient  BufferedWriter bout;

NData(Socket sock0) throws IOException
    {
        sock=sock0;
        din= new DataInputStream(sock.getInputStream());
        dout = new DataOutputStream(sock.getOutputStream());
        bin = new BufferedReader(new InputStreamReader(din));
        bout = new BufferedWriter(new OutputStreamWriter(dout));
     //   .. 
    } 

 //And to add a "magic" destructor

 void nuller() {
        din=null;
        dout=null;
        bin=null;
        bout=null;
    }

有一个结束方法结束线程，关闭流，所以我在那里添加“ nuller”方法调用，内存泄漏就消失了。

为什么在完成线程并关闭流后，它继续在“byte[]”中分配内存？ 为什么GC不扔掉它？ （除了在脏的空赋值之后）

编辑：

正如卡萨布兰卡所说，也许 NData 对象仍然存在， 有一个

final static ConcurrentHashMap <String,NData>(); 

以 NData 作为值的 so，执行了删除（key）以从 Map 中清除对象，但是..这似乎还不够。

从 HashMap 中删除唯一的对象引用不足以删除该对象？

Answer 1

为什么线程结束并关闭流后，它继续在“byte[]”中分配内存？为什么GC不扔掉它？

仅当不再有对该对象的引用时，GC 才会“丢弃”该对象。在您的情况下，这意味着某些东西仍然持有对 NData 对象的引用。通过手动调用 nuller 方法，您只需释放对成员变量（din、dout 等）的引用，但 NData 对象可能仍然存在。您需要在其他地方查找谁在使用该对象，并确保该引用已清除。

更新：您到底是如何得出存在内存泄漏的结论的？ GC 仅定期运行，因此不能保证对象立即被释放。您可以尝试调用 System.gc() 来强制 GC 运行。此外，ConcurrentHashMap（我不熟悉）可能会缓存并发引用，并且在调用remove后这些引用可能不会立即释放。

Answer 2

清理 NData 对象后，您应该进行堆转储。然后在堆中找到NData对象（因为从你的描述来看，有人仍然持有对它的强引用！）。然后找出如何从根集访问该对象。

最近我不得不这样做，我发现使用这些工具的过程很方便：

VisualVM，用于获取堆转储。

Eclipse Memory Analyzer（我独立使用它）来分析堆，特别是查找来自根集。

目前，第二个比第一个先进得多，但第一个的 GUI 非常好用（如果它的功能也很完整的话！）。

Answer 3

在你的 nuller 中调用 bin.close() 和 bout.close() 就可以了。
只需确保 nuller 在某个地方的 finally 块中被调用，这样它总是被调用。