目录移动不起作用（文件已存在）

Question

我有一个主文件夹：

c:\test

我这里有两个文件夹：Movies 和 Photos。

Photos 有三个包含相同结构文件的文件夹：People、Animals 和 Buildings。我正在尝试使用以下代码：

Directory.Move(@"c:\test\Movies", @"c:\test\Test");

我得到了一个异常:

File already exists

- user278618

Answer 1

这个方法将递归地移动文件夹的内容并覆盖现有文件。
你应该添加一些异常处理。
编辑：
该方法使用 while 循环和堆栈实现，而不是递归。

public static void MoveDirectory(string source, string target)
{
    var stack = new Stack<Folders>();
    stack.Push(new Folders(source, target));

    while (stack.Count > 0)
    {
        var folders = stack.Pop();
        Directory.CreateDirectory(folders.Target);
        foreach (var file in Directory.GetFiles(folders.Source, "*.*"))
        {
             string targetFile = Path.Combine(folders.Target, Path.GetFileName(file));
             if (File.Exists(targetFile)) File.Delete(targetFile);
             File.Move(file, targetFile);
        }

        foreach (var folder in Directory.GetDirectories(folders.Source))
        {
            stack.Push(new Folders(folder, Path.Combine(folders.Target, Path.GetFileName(folder))));
        }
    }
    Directory.Delete(source, true);
}
public class Folders
{
    public string Source { get; private set; }
    public string Target { get; private set; }

    public Folders(string source, string target)
    {
        Source = source;
        Target = target;
    }
}

更新：
使用Directory.EnumerateFiles递归地替代使用堆栈的版本，这是一个更简单的版本。
这仅适用于.NET 4或更高版本，如果要在较早版本的.NET中使用，请将Directory.EnumerateFiles更改为Directory.GetFiles。

public static void MoveDirectory(string source, string target)
{
    var sourcePath = source.TrimEnd('\\', ' ');
    var targetPath = target.TrimEnd('\\', ' ');
    var files = Directory.EnumerateFiles(sourcePath, "*", SearchOption.AllDirectories)
                         .GroupBy(s=> Path.GetDirectoryName(s));
    foreach (var folder in files)
    {
        var targetFolder = folder.Key.Replace(sourcePath, targetPath);
        Directory.CreateDirectory(targetFolder);
        foreach (var file in folder)
        {
            var targetFile = Path.Combine(targetFolder, Path.GetFileName(file));
            if (File.Exists(targetFile)) File.Delete(targetFile);
            File.Move(file, targetFile);
        }
    }
    Directory.Delete(source, true);
}

- Jens Granlund

---

一个非常好的解决方案，大大改善了原本相当糟糕的内置方案。 - Steve Owen

为什么这种方法比递归更好？ - Daniel

我看到的问题是，文件在复制过程中被锁定，并且只有在最后才变得可用。Directory.Delete(source, true)将删除它。 - Daniel

将 Directory.GetFiles(folders.Source, "*.*") 更改为 Directory.GetFiles(folders.Source, "*") 以获取所有带或不带扩展名的文件。 - Parag Meshram

@JensGranlund，刚刚验证了一下...你是正确的 :) * 或 . 也可以 :) - Parag Meshram

Answer 2

目标目录不应该已经存在 - Directory.Move 方法会为你创建目标目录。

- Fiona - myaccessible.website

Answer 3

ProcessStartInfo p = new ProcessStartInfo("cmd", "/c move \"c:\\test\\Movies\" \"c:\\test\Test\\"");
p.WindowStyle = ProcessWindowStyle.Hidden; //hide mode
Process.Start(p);

- Suat Erenler

Answer 4

Directory.Move 失败的最常见两个原因是：

• 它在不同的卷中（需要复制/删除）
• 它已经存在（默认情况下不支持覆盖）

这里是我针对第二个问题（覆盖）的简单解决方案：

public bool MoveDirectory(string sourceDirName, string destDirName, bool overwrite)
{
    if (overwrite && Directory.Exists(destDirName))
    {
        var needRestore = false;
        var tmpDir = Path.Combine(Path.GetTempPath(), Path.GetRandomFileName());
        try
        {
            Directory.Move(destDirName, tmpDir);
            needRestore = true; // only if fails
            Directory.Move(sourceDirName, destDirName);
            return true;
        }
        catch (Exception)
        {
            if (needRestore)
            {
                Directory.Move(tmpDir, destDirName);
            }
        }
        finally
        {
            Directory.Delete(tmpDir, true);
        }
    }
    else
    {
        Directory.Move(sourceDirName, destDirName); // Can throw an Exception
        return true;
    }
    return false;
}

- Jaider

---

这段代码存在问题，如果Move(destDirName,tmpDir)失败，则在finally Delete(tmpDir)处会抛出异常。 - Peer Sommerlund

C# 中没有内置的方法来执行这个常见的 Windows 操作吗？ - Kyle Delaney

Answer 5

在复制新内容到目标文件夹之前，删除目标文件夹是否安全？

    Directory.Delete(@"c:\test\test");
    Directory.Move(@"c:\test\movies",@"c:\test\test");

- cptScarlet

Answer 6

你可以直接使用移动方法。

Directory.Move(@"c:\test\Movies\", @"c:\test\Test\");

该文件夹将被删除并复制到测试文件夹中。

- Gagan Burde