请求协助编写并修复终止所有Citrix进程的Bash脚本

原脚本的问题 #

• 仅在脚本启动时计算一次Citrix进程数量，循环内不会重新统计，导致要么无限执行kill操作，要么直接跳过所有终止逻辑
• 未明确处理进程数为0的场景，逻辑不够严谨

修复方案1：优化原脚本逻辑 #

保留原脚本的思路，修正循环内的进程检查逻辑：

#!/bin/bash
# 循环检查并终止Citrix进程
while true; do
    # 每次循环重新统计Citrix进程数量（忽略大小写匹配）
    check=$(ps -ef | grep -i citrix | grep -v grep | wc -l)
    
    if [ $check -eq 0 ]; then
        echo "未找到Citrix进程，终止操作"
        break
    elif [ $check -gt 0 ]; then
        # 通过xargs传递PID执行终止命令，比生成kill脚本更安全
        ps -ef | grep -i citrix | grep -v grep | awk '{print $2}' | xargs kill -TERM
        # 短暂等待进程完成终止，避免重复触发
        sleep 1
    fi
done

修复方案2：使用更简洁的工具（推荐） #

利用pgrep和pkill工具，简化进程检查与终止逻辑，更可靠高效：

#!/bin/bash
# 检查是否存在Citrix进程（忽略大小写）
if pgrep -i citrix > /dev/null; then
    echo "正在终止所有Citrix进程..."
    pkill -i citrix
    
    # 等待所有Citrix进程完全终止
    while pgrep -i citrix > /dev/null; do
        sleep 0.5
    done
    echo "所有Citrix进程已终止"
else
    echo "未找到Citrix进程，无需执行终止操作"
fi

方案2的优势 #

• pgrep直接匹配进程名，比ps | grep更简洁，避免误统计grep自身进程的问题
• pkill直接终止匹配进程，无需手动提取PID，减少出错可能
• 循环等待逻辑确保所有相关进程都被终止，避免遗漏

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