如何将图库中的图片保存至其他文件夹？附尝试的copyFile代码

嘿，我看了你写的文件复制方法，思路是对的——用字节流来读写图片这类二进制文件完全没问题。不过咱们可以做一些优化，让代码更简洁、健壮，同时避免资源泄漏的风险：

优化建议 & 改进后的代码 #

1. 用try-with-resources自动管理流 #

Java 7及以上支持try-with-resources语法，它会自动帮你关闭实现了AutoCloseable接口的资源（比如InputStream和OutputStream），不用手动写close()，也能避免异常情况下资源没释放的问题。

2. 增加前置校验 #

复制文件前先检查：

• 输入文件是否存在且是真实文件（不是目录）
• 目标目录是否存在，不存在则创建（避免找不到目录的异常）

3. 更清晰的异常处理 #

可以打印更详细的异常堆栈信息，或者根据业务需求将异常向上抛出，方便上层处理。

改进后的代码如下：

private void copyFile(File inputPath, File outputPath) {
    // 前置检查：输入文件是否有效
    if (!inputPath.exists() || !inputPath.isFile()) {
        Log.e("CopyError", "输入文件不存在或不是有效文件：" + inputPath.getAbsolutePath());
        return;
    }

    // 确保目标目录存在
    File parentDir = outputPath.getParentFile();
    if (!parentDir.exists()) {
        if (!parentDir.mkdirs()) {
            Log.e("CopyError", "无法创建目标目录：" + parentDir.getAbsolutePath());
            return;
        }
    }

    try (InputStream in = new FileInputStream(inputPath);
         OutputStream out = new FileOutputStream(outputPath)) {

        byte[] buffer = new byte[1024];
        int read;
        while ((read = in.read(buffer)) != -1) {
            out.write(buffer, 0, read);
        }
        out.flush();
        Log.d("CopySuccess", "文件已复制到：" + outputPath.getAbsolutePath());

    } catch (FileNotFoundException fnfe) {
        Log.e("CopyError", "找不到文件：" + fnfe.getMessage());
        fnfe.printStackTrace();
    } catch (IOException e) {
        Log.e("CopyError", "文件复制过程中出错：" + e.getMessage());
        e.printStackTrace();
    }
}

代码说明 #

• try-with-resources的括号里声明流，代码块结束后会自动关闭它们，无需手动调用close()
• 前置检查提前拦截无效输入，避免后续抛出不必要的异常
• 用Log.e打印错误信息，更容易区分成功和失败日志，同时打印堆栈信息方便排查问题

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