Java流关闭实践咨询：try/finally嵌套写法合规性及代码优化

嘿，作为Java新手就能有意识地关注资源关闭的编码规范，这真的是个非常好的开始！咱们先聊聊你最关心的流关闭问题，再一步步梳理代码里可以优化的地方。

一、流关闭的最佳实践：抛弃嵌套try-catch，用try-with-resources #

你现在在finally里嵌套try-catch来关闭流，这在Java 7之前是标准写法，但Java 7之后推出的try-with-resources语句彻底解决了这个繁琐的问题——它会自动关闭所有实现了AutoCloseable接口的资源（包括你用的FileInputStream、FileOutputStream、BufferedReader），不用手动写finally块，代码更简洁，还能避免资源泄漏。

核心思路就是把需要关闭的资源放在try后面的括号里，JVM会自动帮你处理关闭逻辑，哪怕中间抛出异常也不会漏掉：

try (BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
     FileInputStream input = new FileInputStream(file1);
     FileOutputStream output = new FileOutputStream(file2)) {
    // 你的业务逻辑代码
} catch (IOException e) {
    // 异常处理
}

二、你的代码里其他可以优化的点 #

咱们逐条拆解你的现有代码：

1. 用input.available()确定缓冲区大小不可靠 #

available()方法返回的是当前可从输入流中读取的字节数，不是文件的总大小。如果是大文件或者某些特殊流（比如网络流），这个值会远小于实际文件大小，直接用它定义缓冲区会导致读取不完整。

解决办法：可以一边读取字节一边存入列表，读完后再反转；或者用循环读取所有字节到数组（Java 9+可以直接用input.readAllBytes()）。

2. 控制台输入的BufferedReader没有关闭 #

虽然控制台的标准输入流一般不需要手动关闭，但遵循资源管理规范，把它也纳入try-with-resources里会更严谨。

3. 反转字节的写法可以更高效 #

你现在的for循环从后往前写入是可行的，但如果先把字节数组反转再一次性写入，代码会更清晰。注意：Collections.reverse()只能反转对象列表，所以需要把byte数组转成Byte类型的列表再反转；或者手动反转数组（效率更高）。

4. 异常处理可以更实用 #

你现在只打印异常消息，建议同时打印堆栈跟踪（e.printStackTrace()），这样出问题时能快速定位到代码行，方便调试。另外，可以针对特定异常做差异化提示，比如FileNotFoundException，直接提示用户检查文件路径。

5. main方法的throws IOException多余 #

你的copyArray()方法已经捕获了IOException，main方法不需要再抛出这个异常，直接去掉即可。

三、优化后的完整代码 #

结合上面的建议，优化后的代码如下：

import java.io.*;
import java.util.ArrayList;
import java.util.Collections;
import java.util.List;

public class Solution {
    private void copyArray() {
        // 使用try-with-resources自动管理所有可关闭资源
        try (BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
             FileInputStream input = new FileInputStream(getFileName(reader, "Enter a filename to be read: "));
             FileOutputStream output = new FileOutputStream(getFileName(reader, "Enter a filename to write to: "))) {

            // 读取所有字节到列表（兼容Java 8及以下）
            List<Byte> byteList = new ArrayList<>();
            int currentByte;
            while ((currentByte = input.read()) != -1) {
                byteList.add((byte) currentByte);
            }

            // 反转字节列表
            Collections.reverse(byteList);

            // 将反转后的字节写入输出流
            for (Byte b : byteList) {
                output.write(b);
            }

        } catch (FileNotFoundException e) {
            System.out.println("错误：找不到指定文件，请检查路径是否正确！");
            e.printStackTrace();
        } catch (IOException e) {
            System.out.println("发生I/O异常：" + e.getMessage());
            e.printStackTrace();
        }
    }

    // 抽取重复的读取文件名逻辑，让代码更简洁
    private String getFileName(BufferedReader reader, String prompt) throws IOException {
        System.out.print(prompt);
        return reader.readLine().trim();
    }

    public static void main(String[] args) {
        new Solution().copyArray();
    }
}

如果是Java 9及以上，读取和反转字节的逻辑可以更高效（避免装箱拆箱开销）：

// 直接读取所有字节到数组
byte[] buffer = input.readAllBytes();
// 手动反转数组
for (int i = 0; i < buffer.length / 2; i++) {
    byte temp = buffer[i];
    buffer[i] = buffer[buffer.length - 1 - i];
    buffer[buffer.length - 1 - i] = temp;
}
// 一次性写入反转后的数组
output.write(buffer);

四、总结一下 #

• 优先使用try-with-resources来管理资源，这是当前Java的最佳实践，告别嵌套try-catch。
• 不要依赖available()获取文件总大小，用循环读取或者readAllBytes()（Java 9+）更可靠。
• 尽量抽取重复代码，让代码结构更清晰、易维护。
• 异常处理要兼顾用户友好性和调试便利性。

希望这些建议能帮你建立良好的编码习惯，继续加油！

内容的提问来源于stack exchange，提问作者Pablo Aguirre de Souza