我应该关闭 FileChannel 吗？

Question

我今天遇到了我们的一个实用程序类的问题。它是文件的帮助程序，包含一些静态文件复制例程。以下是提取的相关方法以及测试方法。

问题是有时 setLastModified 调用会失败，返回 false。

在我的 PC（Windows 7，最新的 Java）上，我有时会收到“setLastModified failed”消息（大约 1000 次中有 25 次）。

我现在已经通过删除 FileChannel.close 调用解决了这个问题，但我更愿意了解为什么会发生这种情况，即使这是正确的解决方案。

还有其他人遇到同样的问题吗？

private void testCopy() throws FileNotFoundException, IOException {
  File src = new File("C:\\Public\\Test-Src.txt");
  File dst = new File("C:\\Public\\Test-Dst.txt");

  for (int i = 0; i < 1000; i++) {
    copyFile(src, dst);
  }
}

public static void copyFile(final File from, final File to) throws FileNotFoundException, IOException {
  final String tmpName = to.getAbsolutePath() + ".tmp";
  // Copy to a .tmp file.
  final File tmp = new File(tmpName);
  // Do the transfer.
  transfer(from, tmp);
  // Preserve time.
  if (!tmp.setLastModified(from.lastModified())) {
    System.err.println("setLastModified failed!");
  }
  // In case there's one there already.
  to.delete();
  // Rename it in.
  tmp.renameTo(to);
}

public static void transfer(final File from, final File to) throws IOException {
  FileInputStream in = null;
  FileOutputStream out = null;
  try {
    in = new FileInputStream(from);
    out = new FileOutputStream(to);
    transfer(in, out);
  } finally {
    if (null != in) {
      in.close();
    }
    if (null != out) {
      out.close();
    }
  }
}

public static void transfer(final FileInputStream from, final FileOutputStream to) throws IOException {
  FileChannel srcChannel = null;
  FileChannel dstChannel = null;
  //try {
    srcChannel = from.getChannel();
    dstChannel = to.getChannel();
    srcChannel.transferTo(0, srcChannel.size(), dstChannel);
  //} finally {
  //  if (null != dstChannel) {
  //    dstChannel.close();
  //  }
  //  if (null != srcChannel) {
  //    srcChannel.close();
  //  }
  }
}

编辑：我已将代码更改为仅关闭 Streams 而不是 FileChannel，因为研究建议关闭 FileChannel code> 还会关闭 Stream。

Answer 1

经过对保存 java 库源的各个站点进行一些研究后，看起来非常像 FileChannel.close 最终调用 FileInputStream.close 或 FileOutputStream.close > 其父对象。

这对我来说意味着您应该关闭 FileChannel 或 Stream，但不能同时关闭。

鉴于此，我正在更改原来的帖子以反映一种正确的方法，即关闭Stream而不是Channel。

Answer 2

如果您使用的是 Java 7，则可以使用 Files.copy(路径源、路径目标、CopyOption...选项)为此操作以避免编写、测试和调试您自己的实现。

或者，考虑使用外部库，例如 Apache Commons IO。具体来说，您会发现 FileUtils.copyFile(File srcFile, File destFile) 有趣：

/** 
 * Copies a file to a new location preserving the file date.
 * [...]
 * @param srcFile  an existing file to copy, must not be <code>null</code>
 * @param destFile  the new file, must not be <code>null</code>
 * [...]
 */
public static void copyFile(File srcFile, File destFile) throws IOException