Java空指针引用与垃圾回收问题求助：文件复制方法异常排查

嘿，我看你这个文件复制方法里的空指针问题挺典型的，Fortify扫出来的Null Dereference根源在于没有提前校验输入参数合法性，加上流的初始化逻辑存在漏洞，直接导致后续调用is.read()时触发空指针异常。我给你拆解下问题，再给出完整的修复方案：

原代码的核心问题 #

1. 当inPUT为null时，is会保持null状态，但后续直接进入while循环调用is.read()，这必然触发NullPointerException；同理outPUT为null时，os也是null，后续写入操作也会出问题。
2. 没有使用资源自动管理机制，流资源在异常情况下无法保证关闭，会导致资源泄漏。
3. 读写逻辑有小瑕疵：原代码写入时没有指定实际读取的字节数，最后一次缓冲区未填满时会写入无效数据。

修复后的完整代码 #

import java.io.FileInputStream;
import java.io.FileOutputStream;
import java.io.IOException;
import java.io.InputStream;
import java.io.OutputStream;

public class FileCopier {
    // 推荐用4096作为缓冲区大小，更适配系统IO的块大小
    private static final int BLKSIZ = 4096;

    public void copyFileBuffered(String inPUT, String outPUT) throws IllegalArgumentException, IOException {
        // 第一步：提前校验参数，直接拦截非法输入
        if (inPUT == null || inPUT.isBlank()) {
            throw new IllegalArgumentException("输入文件路径不能为null或空白字符串");
        }
        if (outPUT == null || outPUT.isBlank()) {
            throw new IllegalArgumentException("输出文件路径不能为null或空白字符串");
        }

        // 第二步：用try-with-resources自动管理流资源，无需手动关闭
        try (InputStream is = new FileInputStream(inPUT);
             OutputStream os = new FileOutputStream(outPUT)) {

            byte[] buffer = new byte[BLKSIZ];
            int bytesRead;
            // 读取直到流结束（read返回-1），写入时指定实际读取的字节数
            while ((bytesRead = is.read(buffer)) != -1) {
                os.write(buffer, 0, bytesRead);
            }
            // 可选：刷新输出流，确保所有数据落地到磁盘
            os.flush();
        }
        // try-with-resources会自动关闭is和os，无论是否发生异常
    }
}

关键修复点说明 #

• 参数前置校验：在方法开头就检查输入输出路径的合法性，直接抛出IllegalArgumentException，既避免了后续的空指针，还能给出清晰的错误信息，方便调试。
• try-with-resources资源管理：这是Java 7+的特性，所有实现AutoCloseable接口的资源（比如IO流）都可以放在try的括号中，代码块执行完毕后会自动关闭资源，即使发生异常也能保证资源释放，比手动在finally里关闭更可靠简洁。
• 读写逻辑优化：把count改成bytesRead更语义化，写入时指定0, bytesRead是因为最后一次读取可能没填满缓冲区，直接写整个缓冲区会写入无效的垃圾数据。

这样修改后，不仅解决了Fortify扫描出的空指针引用问题，还优化了资源管理和读写逻辑，让代码更健壮。

内容的提问来源于stack exchange，提问作者Yehonaton