使用 FileInfo 类移动配对文件后，需要优雅的方式从文件夹中移动孤立文件

Question

我有一个文件夹，从中移动成对的相关文件（xml 与 pdf 配对）。可以随时将其他文件存入此文件夹，但该实用程序每 10 分钟左右运行一次。我们可以使用 FileSystemWatcher 类，但由于内部原因，我们不使用此实用程序。

我使用 System.IO.FileInfo 类在每次运行期间读取文件夹中的所有文件（仅是 xml 和 pdf）。一旦我将文件放入 FileInfo 对象中，我就会迭代这些文件，将匹配项移动到工作文件夹。完成后，我想将所有未配对但位于 FileInfo 对象中的文件移动到失败文件夹。

由于我似乎无法从 FileInfo 对象中删除项目（或者我丢失了某些内容），因此（1）使用 Directory 类 .GetFiles 中的字符串数组，（2）从 FileInfo 对象创建一个 Dictionary 并在迭代期间从中删除值，或者 (3) 是否有使用 LINQ 或其他方法的更优雅的方法？

这是到目前为止的代码：

internal static bool CompareXMLandPDFFileNames(FileInfo[] xmlFiles, FileInfo[] pdfFiles, string xmlFilePath)
    {
        string workingFilePath = xmlFilePath + @"\WORKING";            

        if (xmlFiles.Length > 0)
        {
            foreach (var xmlFile in xmlFiles)
            {
                string xfn = xmlFile.Name; //xml file name
                string pdfName = xfn.Substring(0,xfn.IndexOf('_')) + ".pdf"; //parsed pdf file name contained in xml file name

                foreach (var pdfFile in pdfFiles)
                {
                    string pfn = pdfFile.Name; //pdf file name
                    if (pfn == pdfName)
                    {
                        //move xml and pdf files to working folder...
                        FileInfo xmlInfo = new FileInfo(xmlFilePath + xfn);
                        FileInfo pdfInfo = new FileInfo(xmlFilePath + pfn);
                        if (!File.Exists(workingFilePath + xfn))
                        {
                            xmlInfo.MoveTo(workingFilePath + xfn);                                
                        }

                        if (!File.Exists(workingFilePath + pfn))
                        {
                            pdfInfo.MoveTo(workingFilePath + pfn);
                        }                            
                    }
                }
            }

            //all files in the file objects should now be moved to working folder, if not, fix orphans...
        }

        return true;
    }

Answer 1

说实话我觉得这个问题问得有点差劲。这个问题是以一种非常复杂的方式表述的。我认为工作流程应该设计得更加稳健和确定性。 （例如，为什么不首先上传压缩集中的文件对？）

（并且没有“某人”很可能“以前一定没有来过这里”）

以下是一些随机改进：

using System;
using System.Linq;
using System.IO;
using System.Text;
using System.Text.RegularExpressions;
using System.Collections.Generic;

namespace O
{
    static class X
    {
        private static readonly Regex _xml2pdf = new Regex("(_.*).xml$", RegexOptions.Compiled | RegexOptions.IgnoreCase);

        internal static void MoveFileGroups(string uploadFolder)
        {
            string workingFilePath = Path.Combine(uploadFolder, "PROGRESS");

            var groups = new DirectoryInfo(uploadFolder)
                .GetFiles()
                .GroupBy(fi => _xml2pdf.Replace(fi.Name, ".pdf"), StringComparer.InvariantCultureIgnoreCase)
                .Where(group => group.Count() >1);

            foreach (var group in groups)
            {
                if (!group.Any(fi => File.Exists(Path.Combine(workingFilePath, fi.Name))))
                    foreach (var file in group)
                        file.MoveTo(Path.Combine(workingFilePath, file.Name));
            }
        }

        public static void Main(string[]args)
        {
        }
    }
}

1. 使用可读名称（说出你的意思）
2. 如果文件名不包含“_”，则 IndexOf 返回 -1；随机上传文件名可能会使程序失败
3. 在 Windows 上处理文件名不区分
4. 大小写 不要手动执行路径连接（您可能会意外地制造 UNC 路径，并且您的代码的可移植性较差）
5. 不要假设一个 xml 将映射到一个 pdf：命名方案意味着许多 xml 映射到相同的 pdf 名称。此实现允许（或者您可以通过拒绝 groups.Where(g => g.Count()>2) 来检测情况）
6. 仅以原子方式移动组 (!)：如果满足以下任一条件组中的文件存在于目标目录中，请勿移动任何文件（否则您将遇到竞争条件，其中组的一部分在最后一个文件（完全）上传之前被移动，并且永远不会被移动，因为该组是不再检测到

其他项目 (todo)

• 不要传递冗余参数 如果您想要进行过滤，您可以传递 FI[] 而不是原始 GetFiles() 调用
• ，特别是：
  • 处理IO异常
  • 上传过程中可能会出现锁定错误（进行测试或最终导致文件损坏）；您需要原子处理这些（即不要移动组中的任何文件，除非所有文件都可以移动；这会有点棘手）
    • 测试您的代码（我的示例均未经过测试；它只是在 Linux 上使用 Mono 进行编译）