2

我们有一个用 Java 编写的相当大而复杂的应用程序，它运行在 Gridgain 包之上。我遇到的问题是，在每个请求开始之前，此应用程序将在那里处理请求大约一天，从而导致 java.nio.channels.ClosedByInterruptException 类型的异常。

我的假设是应用程序没有释放文件句柄，并且在连续使用一天后它耗尽并且不能再继续处理请求（每个请求都需要从每个网格节点读取多个文件）。我们已经将大部分文件 IO 操作包装在诸如此类的类中

package com.vlc.edge;

import com.vlc.common.VlcRuntimeException;
import java.io.File;
import java.io.FileNotFoundException;
import java.io.FileReader;
import java.io.IOException;
import java.io.Reader;

public final class BufferedReaderImpl implements BufferedReader {
    private java.io.BufferedReader reader;

    public BufferedReaderImpl(final String source) {
        this(new File(source));
    }

    public BufferedReaderImpl(final File source) {
        try {
            reader = new java.io.BufferedReader(new FileReader(source));
        } catch (FileNotFoundException e) {
            throw new VlcRuntimeException(e);
        }
    }

    public BufferedReaderImpl(final Reader reader) {
        this.reader = new java.io.BufferedReader(reader);
    }

    public String readLine() {
        try {
            return reader.readLine();
        } catch (IOException e) {
            throw new VlcRuntimeException(e);
        }
    }

    public void close() {
        try {
            reader.close();
        } catch (IOException e) {
            throw new VlcRuntimeException(e);
        }
    }
}

我认为问题在于这种设计没有明确释放文件句柄，我提出的解决方案是添加这样的 finalize 方法

    protected void finalize() throws Throwable
    {
        reader.close();
        super.finalize();   
    }

这将明确地做到这一点。问题（最后）是这是否可能产生任何影响。诸如 java.io.BufferedReader 之类的类是否已经具有处理此类问题的某种机制？

编辑：这里也非常感谢检查这是否真的是问题的方法......即有没有办法查询正在运行的JVM并询问它的文件句柄分配？

4

3 回答 3

13

不能依赖终结器来调用。这不是资源管理的好方法。Java中的标准结构是：

InputStream in = null;
try {
  in = ...;
  // do stuff
} catch (IOException e) {
  // error
} finally {
  if (in != null) { try { in.close(); } catch (Exception e) { } }
  in = null;
}

您可能希望将这些句柄包装在一个类中，但这不是一种可靠的方法。

于 2009-10-22T04:02:38.913 回答
6

覆盖没有什么意义finalize()。如果句柄正在被垃圾收集和完成，那么实例也是如此java.io.BufferedReader，它将被关闭。

有可能（根据规范）句柄正在被垃圾收集但尚未最终确定，但这不太可能。

您可以尝试使用PhantomReferences 清理未使用的文件句柄，但我的猜测是您的实例BufferedReaderImpl仍然从某个地方引用（例如，Map从文件名到打开句柄的值），这就是阻止它们被关闭的原因（在这种情况下终结器无济于事。）

于 2009-10-22T04:18:24.653 回答
1

Java 规范说不保证 ' finalize()' 会被执行。您的代码必须明确地关闭FileReader自己。

于 2009-10-22T04:04:54.193 回答