C#目录文件操作异常：删除目标文件后未按最后修改时间筛选，导致全部文件被复制

我来帮你分析代码里的问题，一步步解决这个异常：

核心问题分析 #

你的代码主要有两个关键问题，导致了不符合预期的行为：

1. 删除目标文件的逻辑位置错误 #

你把删除目标目录文件的代码放在了遍历源文件的循环内部，这会导致每处理一个源文件，就重复删除一遍目标目录的初始文件——后续复制到目标目录的新文件不在最初的dele数组里，不会被删除，但这种重复操作不仅效率极低，还可能在文件已被删除后抛出FileNotFoundException异常。

2. 时间筛选逻辑的潜在误解（或验证缺失） #

当前的时间条件fi.LastWriteTime < DateTime.Now.AddDays(-7)表示文件最后修改时间早于7天前的此刻（比如现在是5月20日10点，就是修改时间在5月13日10点之前）。如果你的所有源文件都满足这个条件，自然会被全部复制。另外，你可能对“早于7天”的逻辑理解有误，比如实际想要的是“最近7天内的文件”，那条件就写反了。

3. 额外隐患：文件复制未处理覆盖 #

默认的File.Copy不允许覆盖同名文件，如果目标目录已有同名文件，会直接抛出IOException。

修复后的完整代码 #

string path = @"C:\Users\peixo\OneDrive\Ambiente de Trabalho\APTIV\moving files\folder";
string destpath = @"C:\Users\peixo\OneDrive\Ambiente de Trabalho\APTIV\moving files\folder1";

public Form1()
{
    InitializeComponent(); // 不要忘记窗体默认的初始化方法

    // 1. 一次性删除目标目录所有文件，只执行一次
    string[] destFiles = Directory.GetFiles(destpath);
    foreach (string fileToDelete in destFiles)
    {
        try
        {
            File.Delete(fileToDelete);
        }
        catch (Exception ex)
        {
            // 处理删除异常（比如文件被占用）
            Console.WriteLine($"无法删除文件 {fileToDelete}：{ex.Message}");
        }
    }

    // 2. 提前计算7天前的时间，避免循环内重复计算
    DateTime sevenDaysAgo = DateTime.Now.AddDays(-7);
    string[] sourceFiles = Directory.GetFiles(path);

    foreach (string sourceFile in sourceFiles)
    {
        FileInfo fi = new FileInfo(sourceFile);
        
        // 👉 根据你的实际需求调整条件：
        // - 复制7天前及更早的文件：保留 <
        // - 复制最近7天内的文件：改成 >
        if (fi.LastWriteTime < sevenDaysAgo)
        {
            // 用Path.Combine安全拼接路径，避免分隔符问题
            string destFilePath = Path.Combine(destpath, Path.GetFileName(sourceFile));
            try
            {
                // 第三个参数true允许覆盖同名文件
                File.Copy(sourceFile, destFilePath, true);
            }
            catch (Exception ex)
            {
                Console.WriteLine($"无法复制文件 {sourceFile}：{ex.Message}");
            }
        }
    }

    // ... 其他代码
}

关键修复点说明 #

1. 优化删除逻辑：把删除目标文件的代码移到源文件遍历循环外，只执行一次，同时添加异常处理避免程序崩溃。
2. 明确时间筛选逻辑：提前计算sevenDaysAgo提升效率，并且注释了条件的含义，你可以根据需求灵活调整<或>。如果还是复制全部文件，建议添加调试代码查看每个文件的时间：

   Console.WriteLine($"{sourceFile} 最后修改时间：{fi.LastWriteTime} | 7天前时间：{sevenDaysAgo}");
   

3. 安全处理文件复制：使用Path.Combine拼接路径，添加true参数允许覆盖同名文件，避免异常。

额外建议 #

尽量不要在窗体构造函数中执行耗时的文件操作，建议把这段逻辑移到Form_Load事件或者后台线程中，避免窗体启动卡顿。

内容的提问来源于stack exchange，提问作者Gabriel Peixoto