如何在Bash脚本的命令中嵌入if语句？脚本问题求助

我看你在写Bash脚本的case分支时，在处理Cassandra状态检查的逻辑里犯了几个语法和逻辑上的小错误，尤其是if语句的写法和管道内变量处理的问题，我来帮你修正并解释清楚：

你代码里的核心问题 #

1. 管道子shell的变量传递问题：你在管道里直接写DN="$(grep DN)"，这会导致DN变量被创建在管道的子shell中，父shell根本拿不到这个变量的值，后续的判断自然失效。
2. if语句语法完全错误：[[if [[!DN]]; echo "All Good" else "Node(s) "$DN" is Down" ;fi]] 这种写法不符合Bash的语法规范，正确的if结构需要then关键字，变量引用要加$，条件判断也要用正确的格式。
3. 变量判断逻辑错误：判断变量是否为空不能直接写!DN，应该用-z "$DN"来检查变量是否为空字符串。

修正后的脚本片段 #

select opt in "${options[@]}"; do
    case $opt in
        "1) Check Cassandra Status.")
            # 先把Cassandra状态命令的输出保存到变量，避免子shell变量传递问题
            cassandra_output=$(ssh skyusr@"$IP" "export JAVA_HOME=/opt/mesosphere && /var/lib/mesos/slave/slaves/*/frameworks/*/executors/*/runs/latest/apache-cassandra-3.0.10/bin/nodetool -p 7199 status" | sed -n '6,10p' | awk '{print $1,$2}')
            # 从输出中提取包含DN（Down节点）的行
            down_nodes=$(echo "$cassandra_output" | grep DN)
            
            # 正确的if条件判断逻辑
            if [[ -z "$down_nodes" ]]; then
                echo "All Good"
            else
                echo "Node(s) $down_nodes is Down"
            fi
            ;;
        "2) Run Repair on all nodes.")
            echo "You chose to run repair on all nodes."
            # 这里可以添加实际的修复命令，比如循环所有节点执行nodetool repair
            # 示例（需要你提前定义节点列表变量node_list）：
            # for node_ip in "${node_list[@]}"; do
            #     ssh skyusr@"$node_ip" "export JAVA_HOME=/opt/mesosphere && /var/lib/mesos/slave/slaves/*/frameworks/*/executors/*/runs/latest/apache-cassandra-3.0.10/bin/nodetool repair"
            # done
            echo "Repair initiated successfully."
            ;;
        # 可以添加退出选项
        "Quit")
            break
            ;;
        *)
            echo "Invalid option, please try again."
            ;;
    esac
done

关键调整说明 #

• 把ssh命令的输出先保存到cassandra_output变量，这样所有后续处理都在父shell中进行，变量不会丢失。
• 用down_nodes变量存储故障节点的信息，更直观易懂。
• 使用[[ -z "$down_nodes" ]]判断是否存在故障节点：如果变量为空，说明没有DN行，所有节点正常；否则输出故障节点。
• 给第二个分支补充了基础的提示和示例代码，方便你后续完善修复逻辑。

额外注意事项 #

• 确保$IP变量已经正确赋值，否则ssh连接会失败。
• 你的Cassandra路径用了通配符*，如果匹配到多个目录，可能会导致nodetool命令执行出错，建议用更精确的路径或者用find命令定位正确的nodetool位置。
• 要保证skyusr用户能免密登录目标节点，否则脚本会卡住等待密码输入。

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