Java: 使用nio Files.copy 移动目录

Question

我是nio类的新手，移动文件夹到新创建的文件夹时遇到了麻烦。首先我使用以下代码创建了两个目录：

File sourceDir = new File(sourceDirStr); //this directory already exists
File destDir = new File(destDirectoryStr); //this is a new directory

我接着尝试将现有的文件复制到新目录中，使用以下命令：

Path destPath = destDir.toPath();
for (int i = 0; i < sourceSize; i++) {
    Path sourcePath = sourceDir.listFiles()[i].toPath();
    Files.copy(sourcePath, destPath.resolve(sourcePath.getFileName()));
}

这会抛出以下错误：

Exception in thread "main" java.nio.file.FileSystemException: destDir/Experiment.log: Not a directory

我知道destDir/Experiment.log不是一个现有的目录，它应该是Files.copy操作的结果产生的新文件。有人能指出我的操作出了什么问题吗？谢谢！

- Adam_G

---

destDir 在磁盘上存在吗？如果不存在，您可能需要先使用 File#mkdirs() 创建它。 - millimoose

我已经运行了 destDir.exists()，它返回了 True。这似乎几乎认为 destDir/Experiment.log 应该是一个目录。不过，这不是这种情况吗？ - Adam_G

Answer 1

你需要使用walkFileTree来复制目录。如果你在一个目录上使用Files.copy，只会创建一个空目录。

以下代码摘自/改编自http://codingjunkie.net/java-7-copy-move/

File src = new File("c:\\temp\\srctest");
File dest = new File("c:\\temp\\desttest");
Path srcPath = src.toPath();
Path destPath = dest.toPath();

Files.walkFileTree(srcPath, new CopyDirVisitor(srcPath, destPath, StandardCopyOption.REPLACE_EXISTING));

public static class CopyDirVisitor extends SimpleFileVisitor<Path>
{
    private final Path fromPath;
    private final Path toPath;
    private final CopyOption copyOption;

    public CopyDirVisitor(Path fromPath, Path toPath, CopyOption copyOption)
    {
        this.fromPath = fromPath;
        this.toPath = toPath;
        this.copyOption = copyOption;
    }

    @Override
    public FileVisitResult preVisitDirectory(Path dir, BasicFileAttributes attrs) throws IOException
    {
        Path targetPath = toPath.resolve(fromPath.relativize(dir));
        if( !Files.exists(targetPath) )
        {
            Files.createDirectory(targetPath);
        }
        return FileVisitResult.CONTINUE;
    }

    @Override
    public FileVisitResult visitFile(Path file, BasicFileAttributes attrs) throws IOException
    {
        Files.copy(file, toPath.resolve(fromPath.relativize(file)), copyOption);
        return FileVisitResult.CONTINUE;
    }
}

- AaronHolland

Answer 2

如果目标文件夹不存在，只需创建它即可。

File sourceDir = new File(source); //this directory already exists
File destDir = new File(dest); //this is a new directory
destDir.mkdirs(); // make sure that the dest directory exists

Path destPath = destDir.toPath();
for (File sourceFile : sourceDir.listFiles()) {
    Path sourcePath = sourceFile.toPath();
    Files.copy(sourcePath, destPath.resolve(sourcePath.getFileName()));
}

请注意，sourceDir.listFiles()会返回目录，您需要递归进入或忽略它们...

- RudolphEst

---

谢谢，但不幸的是这仍然会抛出相同的错误。我甚至运行了destDir.exists()，它返回True。它几乎像是认为destDir/Experiment.log应该是一个目录。不过，这不是这种情况吗？ - Adam_G

我在我的电脑上运行了这段代码，只要目录中没有子目录需要复制，它就可以正常工作。 - RudolphEst

我能想到的唯一区别是，我需要使用 for (int i = 0; i < trainingFilesSize; i++) { Path vectorPath = vectorDir.listFiles()[i].toPath(); 因为我将复制某些索引。不过这应该不会有什么影响，对吧？ - Adam_G

你是对的...它不应该有任何影响。我感到困惑。 - RudolphEst

Answer 3

这是我递归移动目录从源到目标的解决方案。它像魔法一样运行良好。

public static void move(Path source, Path target) throws IOException {

    class FileMover extends SimpleFileVisitor<Path> {
        private Path source;
        private Path target;

        private FileMover(Path source, Path target) {
            this.source = source;
            this.target = target;
        }

        @Override
        public FileVisitResult visitFile(final Path file, final BasicFileAttributes attrs) throws IOException {
            Files.move(file, target.resolve(source.relativize(file)),
                StandardCopyOption.REPLACE_EXISTING);
            return FileVisitResult.CONTINUE;
        }

        @Override
        public FileVisitResult preVisitDirectory(final Path dir, final BasicFileAttributes attrs) throws IOException {
            Path newDir = target.resolve(source.relativize(dir));
            try {
                Files.copy(dir, newDir,
                    StandardCopyOption.COPY_ATTRIBUTES,
                    StandardCopyOption.REPLACE_EXISTING);
            } catch (DirectoryNotEmptyException e) {
                // ignore and skip
            }
            return FileVisitResult.CONTINUE;
        }

        @Override
        public FileVisitResult postVisitDirectory(final Path dir, final IOException exc) throws IOException {
            Path newDir = target.resolve(source.relativize(dir));
            FileTime time = Files.getLastModifiedTime(dir);
            Files.setLastModifiedTime(newDir, time);
            Files.delete(dir);
            return FileVisitResult.CONTINUE;
        }
    }

    FileMover fm = new FileMover(source, target);
    EnumSet<FileVisitOption> opts = EnumSet.of(FileVisitOption.FOLLOW_LINKS);

    Files.walkFileTree(source, opts, Integer.MAX_VALUE, fm);
}

- Downhillski

Answer 4

for (int i = 0; i < sourceSize; i++) {
    Path sourcePath = sourceDir.listFiles()[i].toPath();
    Files.copy(sourcePath, destPath.resolve(sourcePath.getFileName()));
}

这是非常奇怪的代码。你已经从某个地方得到了一个文件计数，存储在sourceSize中，但是你却在每次迭代中调用listFiles()。我本来期望看到更像这样的代码：

for (File file : sourceDir.listFiles()) {
    Path sourcePath = file.toPath();
    Files.copy(sourcePath, destPath.resolve(sourcePath.getFileName()));
}

- user207421

---

由于本问题范围之外的原因，我需要使用“int i = 0”格式。这与选择随机数并传递索引号有关。 - Adam_G

你完全错了，这不是在每次迭代时都继续调用listFiles()的理由。你正在冒着目录在迭代之间发生变化，并重复处理同一文件的风险。 - user207421