Bash脚本IF/ELIF语句异常：输入'n'仍触发条件执行求助

问题根源 #

你的脚本出现逻辑异常有两个核心原因：

1. 变量赋值语法错误
   nmap_opt = " " 不符合Bash语法规则——变量名与等号之间不能有空格。这行代码会被解析为执行名为nmap_opt的命令（而非变量赋值），导致nmap_opt变量初始值未定义（为空）。

2. 条件判断运算符误用
   你用整数比较运算符-eq来比较字符串（用户输入的y/n），这是错误的。-eq仅适用于整数比较，用它处理字符串时，Bash会尝试将字符串转成整数，转换失败会导致条件判断逻辑混乱，最终意外触发then分支，无论输入y还是n都会添加对应的Nmap选项。

解决方法 #

针对上述问题，给出以下修复步骤：

1. 修正变量赋值 #

将nmap_opt = " "改为nmap_opt=""，确保变量正确初始化为空字符串（避免后续拼接时出现多余空格）。

2. 替换条件判断逻辑 #

放弃if/elif的字符串比较写法，改用case语句处理y/n输入——这种方式更简洁，还天然支持大小写兼容，同时避免运算符误用问题。

3. 可选：添加输入验证循环 #

添加循环确保用户只能输入y/Y或n/N，直到输入有效为止，避免无效输入进入逻辑分支。

修正后的完整脚本 #

#!/bin/bash
#requires root or sudo priv

#check for a successful ping

target_ip=$1
nmap_opt=""  # 修正赋值语法，初始为空
#nmap options variables --
nmap_os="-O"
nmap_version="-sV"
nmap_udp="-sU"
nmap_stealth="-sS"
nmap_aggr="-A"
nmap_option_input_temp=""

ping "$target_ip" -c 5
if [ $? -eq 0 ]; then
        echo "successful ping on $target_ip" 
else
        echo "ping unsuccessful, check VPN connection or host may be down"
        exit 1
fi

read -p "enter pathway for nmap output: " nmap_file

# 检测系统选项，添加输入验证循环
while true; do
    read -p "detect os? (y/n) " nmap_option_input_temp
    case "$nmap_option_input_temp" in
        y|Y)
            nmap_opt="${nmap_opt} ${nmap_os}"
            break
            ;;
        n|N)
            break
            ;;
        *)
            echo "invalid option, please enter y or n"
            ;;
    esac
done

# 检测版本选项，添加输入验证循环
while true; do
    read -p "detect version? (y/n) " nmap_option_input_temp
    case "$nmap_option_input_temp" in
        y|Y)
            nmap_opt="${nmap_opt} ${nmap_version}"
            break
            ;;
        n|N)
            break
            ;;
        *)
            echo "invalid option, please enter y or n"
            ;;
    esac
done

echo "nmap selected option/s: $nmap_opt"
#starting nmap..
sudo nmap $nmap_opt -oN "$nmap_file" "$target_ip" 

额外优化说明 #

• 对$target_ip、$nmap_file等变量添加双引号包裹，避免目标IP或路径包含特殊字符、空格时出现解析错误。
• case语句替代if/elif后，代码逻辑更清晰，无需额外处理大小写转换。

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