Bash变量sent无法保留拼接值，重复发送邮件问题排查

你的问题核心在于管道创建了独立的子Shell进程，导致sent变量的赋值无法传递回父Shell，所以每次循环时父Shell里的sent始终是空值，自然会重复处理同一个IP。

为什么会这样？ #

在Bash里，管道（|）的每一段命令都会在单独的子Shell中运行。你脚本里的if语句是跟在grep -c "$ip" |后面的，所以这个if是在子Shell里执行的——你在子Shell里给sent赋值，这个变量只存在于那个子Shell的内存空间里，当子Shell执行完毕退出后，父Shell（也就是外层for循环所在的Shell）完全看不到这个赋值，sent还是初始的空字符串。这就是为什么你在if里echo能看到值，出了fi就没了。

怎么解决？ #

有两种可靠的方法，推荐第一种（兼容性更好，逻辑更清晰）：

方法1：用命令替换替代管道，避免子Shell

把统计次数的逻辑改成用命令替换（$()）获取结果，这样整个判断逻辑都在父Shell里执行，变量赋值自然能保留：

# 先初始化sent变量
sent=""
# 遍历数组时用"${ARRAY[@]}"，避免带空格的元素被拆分
for ip in "${ARRAY[@]}"; do
    # 用Here String把数组内容传给grep，统计出现次数
    count=$(grep -c "$ip" <<< "${ARRAY[*]}")
    # 判断条件：次数小于10，且当前ip未被处理过
    if [[ $count -lt 10 && $sent != *"$ip"* ]]; then
        # 赋值时加双引号，防止特殊字符破坏变量
        sent="$sent$ip"
        # 执行邮件发送逻辑
        sed -n "/$ip/p" "$1" | mail -s "subject" "mail@mail.mail"
    fi
done

方法2：开启lastpipe选项（Bash 4.2+可用）

如果一定要保留管道写法，可以在脚本开头加上shopt -s lastpipe，这个选项会让管道的最后一个命令在父Shell中执行，而不是子Shell：

shopt -s lastpipe
sent=""
for ip in "${ARRAY[@]}"; do 
    for e in "${ARRAY[@]}"; do 
        echo "$e" 
    done | grep -c "$ip" | {
        read count
        if [[ $count -lt 10 && $sent != *"$ip"* ]]; then 
            sent="$sent$ip"
            sed -n "/$ip/p" "$1" | mail -s "subject" "mail@mail.mail" 
        fi
    }
done

不过这个方法依赖Bash版本，兼容性不如第一种，所以更推荐方法1。

额外优化建议 #

1. 遍历数组时用"${ARRAY[@]}"而非${ARRAY[*]}：前者会把每个数组元素作为单独的项，避免元素里有空格时被错误拆分。
2. 变量赋值和引用时尽量加双引号：防止特殊字符（比如空格、*）导致意外的Shell扩展。
3. 统计数组元素出现次数的逻辑可以更高效：比如先用printf "%s\n" "${ARRAY[@]}" | sort | uniq -c一次性统计所有元素的次数，再遍历结果，这样比嵌套循环更高效（尤其是数组很大的时候）。

内容的提问来源于stack exchange，提问作者Miguel.G