如何实现逐个删除文件直至目录为空？现有脚本优化求助

我来帮你搞定这个需求！你的核心目标是循环处理results/todel/目录里的所有文件——只要目录里有文件就执行「删注册行+删文件」的操作，直到目录清空，对吧？咱们先梳理原脚本的小问题，再给出完整的优化方案：

原脚本的几个潜在问题 #

• scandir()[2]只能取第一个文件，没法循环处理，而且如果目录为空会直接报错
• sed命令的引号写法有问题，变量$firstFile没法正确插入正则，另外如果文件名含特殊字符（比如空格、&、*），会导致shell命令执行失败甚至出现安全风险

优化后的完整代码 #

我推荐用PHP原生函数来实现，更安全可控，还能自然实现循环：

<?php
$registerFile = "results/register.php";
$todelDir = "results/todel/";

// 循环处理，直到目录中没有文件
while (true) {
    // 获取目录中的所有文件（自动排除.和..）
    $files = glob($todelDir . "*");
    
    // 没有文件就退出循环
    if (empty($files)) {
        break;
    }
    
    // 取当前第一个文件
    $currentFile = basename($files[0]);
    $fullFilePath = $files[0];
    
    // 1. 从注册文件中删除包含该文件名的行
    $registerContent = file_get_contents($registerFile);
    // 转义文件名中的正则特殊字符（比如.、*这些），避免匹配出错
    $safeFileName = preg_quote($currentFile, '/');
    // 匹配整行包含该文件名的内容并删除（m修饰符让^$匹配每行开头结尾）
    $updatedContent = preg_replace("/^.*{$safeFileName}.*$/m", '', $registerContent);
    file_put_contents($registerFile, $updatedContent);
    
    // 2. 删除目录中的目标文件
    unlink($fullFilePath);
    
    // 可选：如果需要控制处理速度，避免资源占用过高，可以加个延迟
    // sleep(1);
}
?>

关键细节说明 #

• 用glob()替代scandir()：glob($todelDir . "*")会直接返回目录下所有文件的完整路径，自动过滤掉.和..，比scandir更省心
• 循环逻辑：while(true)配合空文件检查，确保只要目录里有文件就一直处理，直到$files为空时退出循环
• 安全处理文件名：preg_quote()会转义文件名中的正则元字符，比如如果文件名是my.file.txt，不会被正则当成「任意字符」来匹配
• 避免shell命令风险：用PHP原生的file_get_contents和preg_replace处理注册文件，比调用sed更安全，不用担心文件名含特殊字符导致shell命令出错

如果坚持用sed命令的修正方案 #

如果你一定要保留shell_exec调用sed的方式，记得要正确转义变量，避免安全问题：

// 替换原sed相关代码为：
$safeFileName = escapeshellarg($currentFile);
$safeRegisterFile = escapeshellarg($registerFile);
$cmd = "sed -i '/{$safeFileName}/d' {$safeRegisterFile}";
shell_exec($cmd);

但还是更推荐用PHP原生处理的方案，可控性更强。

内容的提问来源于stack exchange，提问作者Andrés Chandía