这个删除不在数组中的所有文件的 C# 例程看起来没问题吗？

Question

我有一个包含文件路径列表的数据库。我想构建一个例程来清理文件夹，如果没有数据库记录，则删除目录中的文件（用于临时ajax文件上传，在用户未完成表单的情况下，等等...）。

我在想这样的事情：

var dbFiles = db.allPaths();
var allFiles = Directory.EnumerateFiles(path);

foreach (var f in allFiles) {
  if (!dbFiles.Contains(f) {
    File.Delete(f);
  }
}

有什么“陷阱”在等着我吗？该例程最初将设置为每周运行一次，如果临时文件出现问题，则运行频率会更高。它将在几乎没有用户的情况下运行，因此性能虽然很重要，但并不是最重要的。

更新

哇，有很多很棒的答案。这段代码正在变成值得“分享”的东西。 ;D 我上面的代码只是一个简单、快速的占位符位...但它已转换为可靠的代码。谢谢你！

Answer 1

看起来不错，但您可以使其更简单：

foreach (var file in allFiles.Except(dbFiles))
{
    File.Delete(file);
}

您必须确保路径的格式完全相同。如果一个列表包含相对文件而另一个列表包含绝对文件，或者如果一个列表使用“/”而另一个列表使用“\”，那么您最终将删除您不希望删除的内容。

理想情况下，您首先要明确地规范化文件，但我看不到在 .NET 中获取规范文件名的好方法...

编辑：请注意 Path.GetFullPath 确实不 规范化。它修复了斜杠并使其成为绝对，但它没有解决大小写问题：“c:/users”变为“c:\users”，但“c:/Users”变为“c:\Users”。

这可以通过在调用 Except 中使用字符串比较器来解决：

var dbFiles = db.AllPaths().Select(Path.GetFullPath));
var allFiles = Directory.EnumerateFiles(path).Select(Path.GetFullPath));

foreach (var file in allFiles.Except(dbFiles, StringComparer.OrdinalIgnoreCase))
{
    File.Delete(file);
}

现在忽略大小写 - 但以“顺序”方式。我不知道 Windows 文件系统在区分大小写方面真正做什么。

Answer 2

我觉得不错；但是我从来没有在 C# 中删除过文件，只是在 VB 中删除过文件。
但是，您可能希望将其放入 Try/Catch 循环中，就好像该文件无法删除（只读、当前正在使用、不再存在等）一样，它将引发异常。

编辑：路径如何存储？请记住，在 C# 中，您需要转义路径“//”而不是使用“\”IIRC。

编辑2：把最后的编辑内容去掉，哈哈。

Answer 3

将所有建议合并为一个：

// canonicalize paths
var dbFiles = db.allPaths().Select(Path.GetFullPath);
var allFiles = Directory.EnumerateFiles(Path.GetFullPath(path))

foreach (var file in allFiles.Except(dbFiles, StringComparer.OrdinalIgnoreCase))
{
    try {
        File.Delete(file);
    } catch (IOException) {
        // handle exception here
    }
}

Answer 4

我认为它在精神上是好的，尽管它会更接近：

List<string> dbFiles = db.allPaths();
string[] allFiles = Directory.GetFiles(path);

foreach (string f in allFiles)
    if (!dbFiles.Contains(f))
        File.Delete(f);