PHP读取输入文件名并排序追加文件内容异常求助

我帮你梳理下当前代码里的几个核心问题，然后给出修复后的完整实现：

1. 读取逻辑错误：你用if($line == "\n" || $line == "") break;会在遇到空行时直接终止读取，这会导致只读取到第一个空行之前的内容，而不是整个文件的有效内容；
2. 缺少数据收集与排序逻辑：代码里的$values ...部分截断了，说明你还没完成收集所有行、排序的步骤；
3. 资源泄漏风险：操作完文件后没有关闭文件句柄，可能导致系统资源无法及时释放。

下面是修复后的完整代码，每一步都做了优化：

function main(){ 
    global $argc, $argv; 
    if($argc == 1){ 
        print("Please provide text file path"); 
        return; 
    } 
    print("START\n"); 
    run($argv[1]); 
    print("END\n"); 
}

function run($file_name){ 
    // 打开文件用于读取
    $br = fopen($file_name,"r"); 
    // 打开文件用于追加内容
    $bw = fopen($file_name,"a"); 

    if($br && $bw){ 
        $lines = [];
        // 循环读取所有行，跳过空行但不终止循环
        while(($line = fgets($br)) !== false){ 
            // 去除首尾空白（包括换行符），避免不同系统换行符的影响
            $trimmed_line = trim($line);
            // 仅保留非空行
            if(!empty($trimmed_line)){
                $lines[] = $trimmed_line;
            }
        }

        // 对收集到的内容进行排序（默认按字符串升序）
        sort($lines);

        // 将排序后的内容追加到原文件，添加分隔标识方便区分
        fwrite($bw, "\n--- 排序后的内容 ---\n");
        foreach($lines as $sorted_line){
            fwrite($bw, $sorted_line . "\n");
        }

        // 关闭文件句柄，释放系统资源
        fclose($br);
        fclose($bw);
    } else {
        // 添加错误提示，方便排查文件打开失败的问题
        print("Failed to open file: {$file_name}\n");
    }
}

// 执行主函数
main();

关键修改说明： #

• 新增$lines数组存储所有有效行，解决了空行终止读取的问题；
• 用trim()处理每行内容，统一判断空行的逻辑，避免因系统换行符差异导致的误判；
• 使用sort()完成排序，如果需要自定义排序规则（比如按数字大小、降序等），可以替换为usort()并传入自定义回调函数；
• 追加内容时添加了分隔线，方便区分原文件内容和新增的排序内容；
• 增加了文件打开失败的错误提示，同时关闭文件句柄避免资源泄漏。

额外注意事项： #

• 如果你的文件是UTF-8编码，建议在打开文件后添加stream_set_encoding($br, 'UTF-8')和stream_set_encoding($bw, 'UTF-8')，避免乱码；
• 如果处理超大文件，一次性读取所有行到内存可能会影响性能，此时可以考虑先将内容写入临时文件，排序后再追加到原文件，但对于普通大小的文件，当前方案足够简单高效。

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