Shell脚本循环处理文件封装函数失效及find结果存数组问题咨询

1. 函数封装后无待处理文件输出的排查 #

两个最常见的出错原因：

1. 变量名冲突：你原来的代码里用了source作为存储目录的变量名，而source是Shell内置命令，封装进函数后作用域变化很容易引发执行异常，建议重命名为sourceDir这类不会冲突的名称。
2. 变量作用域/调用顺序问题：Shell函数必须先定义再调用，同时如果函数内部要用到外部变量，要么明确作为参数传入，要么确保是全局变量，强烈推荐通过参数传递避免全局变量污染。
   正确的封装示例：

source ./logger.sh

# 先定义函数，后调用
moveFiles() {
    # 接收传入的参数，避免依赖全局变量
    local sourceDir="$1"
    local older="$2"
    local scriptName="$3"
    local logFile="$4"

    while IFS= read -r -d $'\0'; do
        printf "\nIn while: Currently processing file: ${REPLY}\n"
        local file="${REPLY}"
        log 1 "${scriptName}" "moveFiles" "Currently processing file: ${file}" "${logFile}"
    done < <(find "${sourceDir}" -maxdepth 1 -type f -mtime +"${older}" ! -name '*.FLAG' ! -name '*.FAIL' -print0)
}

# 全局变量定义
now=$(date +"%Y.%m.%d_%H.%M.%S")
logFile="/path/to/the/directory/${now}.log"
touch "${logFile}"
scriptName="test.sh"
sourceDir="/path/to/the/directory"
older=7

# 调用函数并传入参数
moveFiles "${sourceDir}" "${older}" "${scriptName}" "${logFile}"

2. find结果存入数组的正确实现 #

你之前的写法错误原因是：list=$(find ...)会把所有输出存为单个字符串，哪怕用了-print0，直接赋值为数组也不会按null分隔拆分，只会得到一个元素。
正确实现分两种场景：

适用Bash 4.4及以上版本（最简洁写法） #

# 直接用mapfile按null分隔把find结果存入数组
mapfile -d $'\0' fileList < <(find "${sourceDir}" -maxdepth 1 -type f -mtime +"${older}" ! -name '*.FLAG' ! -name '*.FAIL' -print0)

兼容低版本Bash写法 #

fileList=()
while IFS= read -r -d $'\0' file; do
    fileList+=("${file}")
done < <(find "${sourceDir}" -maxdepth 1 -type f -mtime +"${older}" ! -name '*.FLAG' ! -name '*.FAIL' -print0)

拿到数组后，就可以直接遍历或者传给其他函数：

# 直接遍历处理
for file in "${fileList[@]}"; do
    log 1 "${scriptName}" "main" "Currently processing file: ${file}" "${logFile}"
done

# 传给其他函数处理
processInLoop "${fileList[@]}"

processInLoop() {
    local fileList=("${@}")
    for file in "${fileList[@]}"; do
        # 你的业务处理逻辑
        echo "Processing: ${file}"
    done
}

注意事项 #

• 所有变量引用建议包裹双引号，避免文件名包含空格、换行等特殊字符时出现解析错误
• 不要使用Shell内置命令、关键字作为变量名，避免不可预期的执行异常
• 处理文件名相关的输出时，统一使用null作为分隔符，可100%兼容所有合法Linux文件名的特殊字符

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