遍历子目录批量替换TXT文件名空格为下划线的脚本故障排查

我拥有约10000个包含文件的目录，希望编写循环遍历每个目录，筛选出其中的.txt文件，并将文件名中的空格替换为下划线。我编写了如下Shell脚本：

for f in *FOLDERS*
do
    cd "$f" && echo "Entering into $f" || { echo "Error: could not enter into $f"; continue; }
    for y in $(ls *.txt | mv "$y" "${y// /_}")
    do
        echo ${y}
    done
done 

但该脚本无法在每个目录正常运行，请问我哪里出错了？

核心错误点 #

• ls与mv的管道用法完全错误：你写的$(ls *.txt | mv "$y" "${y// /_}")逻辑混乱，ls输出的文件名会传到mv的标准输入，但mv根本不读标准输入来处理文件；而且此时$y还没被赋值，mv执行时会报错，整个子命令输出为空，内层循环根本跑不起来。
• 未返回父目录：进入子目录cd "$f"后，没有执行cd ..回到上层目录，导致第二次循环时会在当前子目录下找*FOLDERS*，完全偏离预期的遍历路径。
• 可能遍历非目录项：*FOLDERS*会匹配所有名称含FOLDERS的文件或目录，如果碰到同名文件，cd命令会直接报错。
• 内层循环逻辑颠倒：不需要先通过ls获取文件名再循环，直接遍历.txt文件逐个处理才是正确逻辑。

修正后的脚本 #

# 遍历所有名称包含FOLDERS的目录
for dir in *FOLDERS*/
do
    # 跳过非目录项
    if [ ! -d "$dir" ]; then
        echo "跳过非目录: $dir"
        continue
    fi

    # 进入目录，失败则跳过后续操作
    cd "$dir" || { echo "无法进入目录: $dir"; continue; }
    echo "进入目录: $dir"

    # 遍历当前目录下所有.txt文件
    for txt_file in *.txt
    do
        # 处理目录下无txt文件的情况（避免匹配到字面量*.txt）
        if [ ! -f "$txt_file" ]; then
            continue
        fi

        # 替换文件名中的空格为下划线
        new_name="${txt_file// /_}"
        # 文件名无空格时跳过重命名，避免无效操作
        if [ "$txt_file" != "$new_name" ]; then
            mv -v "$txt_file" "$new_name"
            echo "重命名完成: $txt_file -> $new_name"
        fi
    done

    # 返回上层目录，保证下一次循环路径正确
    cd ..
done

修正说明 #

• 精准遍历目录：在*FOLDERS*后加/，让shell只匹配目录；同时增加[ ! -d "$dir" ]判断，双重避免处理非目录项。
• 安全遍历文件：直接用for txt_file in *.txt遍历文件，避免使用ls（ls输出在处理特殊字符时容易出现异常）。
• 处理边界情况：当目录下没有.txt文件时，*.txt会保留字面量，通过文件存在性判断跳过。
• 保证路径正确：每次处理完子目录后执行cd ..，回到父目录，确保后续循环遍历正确的目录集合。
• 避免无效操作：判断原文件名与新文件名是否一致，跳过无空格的文件，减少不必要的mv执行。

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