Bash脚本中函数参数$2正常，赋值变量后无输出的问题求助

我来帮你分析一下问题所在，以及怎么修复：

问题根源 #

你的脚本里有两个关键问题导致了这个奇怪的现象：

• 用ls遍历文件是错误的实践：ls的输出会被Shell按空白符（空格、换行、制表符）分割，如果文件名里包含这些特殊字符，循环变量fileelement会被拆成多个不完整的片段，传递给deleteFiles的路径就会出错。
• 参数传递和变量引用未加引号：拼接路径调用deleteFiles时没加引号包裹参数，函数内部赋值和输出变量时也没加引号，这会让Shell对变量内容进行分词、通配符扩展等操作，导致变量值意外丢失或被篡改。

修复后的代码 #

function deleteFiles {
    # 用local声明局部变量，避免污染全局环境
    local expireTime="$1"
    local deleteFolder="$2"
    # 加引号输出，确保完整显示变量内容
    echo "$deleteFolder"
}

function cleanexpiredfiles {
    local expireTime="$1"
    local targetDir="$2"
    # 用glob模式直接遍历目录下的所有条目，避开ls的缺陷
    for fileelement in "$targetDir"/*
    do
        # 处理目录为空的情况（此时glob会保留原字符串，需要跳过）
        [ -e "$fileelement" ] || continue
        # 用引号包裹参数，确保完整传递路径
        deleteFiles "$expireTime" "$fileelement"
    done
}

function main() {
    cleanexpiredfiles $((16*60*60)) /opt/gdsmigration/logs
}

main "$@"

关键修复点解释 #

• 替换ls遍历为glob模式："$targetDir"/*会直接获取目录下的所有文件/目录路径，不管文件名里有什么特殊字符，都能完整保留路径信息。
• 全程用引号包裹变量：在参数传递、变量赋值、输出时都用双引号包裹，防止Shell对变量内容进行不必要的处理，确保路径完整传递和赋值。
• 使用local声明局部变量：避免函数内的变量污染全局变量，让脚本逻辑更清晰健壮。

这样修改后，echo $2和echo $deleteFolder的输出就会完全一致了。

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