应该如何关闭InputStream和OutputStream？

Question

我使用以下代码来关闭与服务器的连接的输入流和输出流：

try {
        if (mInputStream != null) {
            mInputStream.close();
            mInputStream = null;
        }

        if (mOutputStream != null) {
            mOutputStream.close();
            mOutputStream = null;
        }
    } catch (IOException e) {
        e.printStackTrace();
    }

但是，流并未关闭，它们仍然存在。如果我再次连接，则会有两个不同的输入流。 catch 部分没有捕获任何异常。

我做错了什么？

Answer 1

编辑：在底部添加了 Java 8 try-with-resources 示例，因为该语言自最初发布以来已经不断发展。

如果您使用的是 Java 7（或更低版本），则您发布的代码存在两个问题：

1. .close() 调用应在 finally 块中处理。这样它们将始终被关闭，即使它在途中的某个地方落入了 catch 块。
2. 您需要在其自己的 try/catch 块中处理每个 .close() 调用，否则您可能会使其中一个处于打开状态。如果您尝试关闭输入流失败，您将跳过关闭输出流的尝试。

您想要的更像是这样的：

    InputStream mInputStream = null;
    OutputStream mOutputStream = null;
    try {
        mInputStream = new FileInputStream("\\Path\\MyFileName1.txt");
        mOutputStream = new FileOutputStream("\\Path\\MyFileName2.txt");
        //... do stuff to your streams
    }
    catch(FileNotFoundException fnex) {
        //Handle the error... but the streams are still open!
    }
    finally {
        //close input
        if (mInputStream != null) {
            try {
                mInputStream.close();
            }
            catch(IOException ioex) {
                //Very bad things just happened... handle it
            }
        }
        //Close output
        if (mOutputStream != null) {
            try {
                mOutputStream.close();
            }
            catch(IOException ioex) {
                //Very bad things just happened... handle it
            }
        }
    }

如果您使用的是 Java 8+，则不需要任何 catch/finally 噪音。您可以使用 try-with-resources 语法，无论您何时以何种方式离开该块，Java 都会为您关闭资源：

    try(InputStream mInputStream = new FileInputStream("\\Path\\MyFileName1.txt")) {
        try(OutputStream mOutputStream = new FileOutputStream("\\Path\\MyFileName2.txt")) {
        //... do stuff to your streams
    }
}