Question

我有一个守护程序，它读取文件的内容，然后压缩它并将其写入较小的.tar.gz文件。

出于某种原因，即使在我释放（或者我认为）所有使用过的内存之后，Java仍继续分配内存。我的代码/推理出了什么问题？

FileOutputStream fos    = null;
GZIPOutputStream gzipos = null;
OutputStreamWriter osw  = null;

BufferedWriter bw = null;
while (true) {
    if (f.length() != 0) {
        if (outputfile == null) {
            outputfile = outputfileroot + "_" + outputPart + ".tar.gz";

            fos = new FileOutputStream(outputfile);
            gzipos  = new GZIPOutputStream(fos);
            osw = new OutputStreamWriter(gzipos);
            bw  = new BufferedWriter(osw);
        }
        else if (new File(outputfile).length() > maxLengthOutputFile) {
            bw.flush();
            osw.flush();
            gzipos.flush();
            fos.flush();
            bw.close();
            osw.close();
            gzipos.close();
            fos.close();

            bw  = null;
            osw = null;
            gzipos  = null;
            fos = null;

            System.gc();

            System.out.println("Finished writing " + outputfileroot + "_" + outputPart + ".tar.gz");

            outputfile = outputfileroot + "_" + ++outputPart + ".tar.gz";
            fos     = new FileOutputStream(outputfile);
            gzipos  = new GZIPOutputStream(fos);
            osw     = new OutputStreamWriter(gzipos);
            bw      = new BufferedWriter(osw);
        }

        /**
         * Read the entire file
         */
        BufferedReader br = new BufferedReader(new FileReader(f));
        String line;
        while ((line = br.readLine()) != null) {
            // will send the content to another thread, so I need to read it line by line
            bw.write(line + "\r\n");
        }
        br.close();
        br = null;
        bw.flush();

        /**
         * Empty it
         */
        FileWriter fw = new FileWriter(f);
        fw.write("");
        fw.flush();
        fw.close();
        fw = null;
    }

    Thread.sleep(1000);
}

Answer 1

你正在调整这个。所有这些空值设置和gc() - 调用实际上并没有帮助，并且您有多次刷新和关闭，因为您真正需要。此外，您根本不需要使用读者和作家。所有这一切都可以简化为：

GZIPOutputStream gzipos = null;
while (true)
{
    if (f.length() != 0)
    {
        if (outputfile == null)
        {
            outputfile = outputfileroot + "_" + outputPart + ".tar.gz";
            gzipos = new GZIPOutputStream(new FileOutputStream(outputfile));
        }
        else
        {
            if (new File(outputfile).length() > maxLengthOutputFile)
            {
                gzipos.close();
                System.out.println("Finished writing " + outputfileroot + "_" + outputPart + ".tar.gz");
                outputfile = outputfileroot + "_" + ++outputPart + ".tar.gz";
                gzipos = new GZIPOutputStream(new FileOutputStream(outputfile));
            }
        }

        /**
         * Read the entire file
         */
        InputStream in = new FileInputStream(f);
        byte[] buffer = new byte[8192];
        int count;
        while ((count = in.read(buffer)) > 0)
        {
            gzipos.write(buffer, 0, count);
        }
        in.close();
        gzipos.flush();
        /**
         * Empty it
         */
        f.createNewFile();
    }
    Thread.sleep(1000);
}

我无法理解你的评论'会将内容发送到另一个帖子，所以我需要逐行阅读'。此代码中没有线程，并且您不需要逐行输入。

我也很好奇它是如何与产生输入文件的任何东西进行交互的。我认为您应该重命名输入文件，并在您决定复制它时立即创建一个新的空文件，而不是在复制步骤之后。

Answer 2

对象不会因为不再引用而从内存中释放。 JVM决定何时运行其“垃圾收集”，从而释放内存。但它通常不会运行垃圾收集，除非它需要。有关详细信息，请参阅this page about the topic。

您可以调用System.gc()来显式调用垃圾收集器（链接中的第7点），但不必运行。

Answer 3

System.gc()发送垃圾回收请求。运行时决定是执行它还是忽略所述请求。

Java没有释放内存

3 个答案: