Bash脚本问题：UID范围校验有效但已占用校验失效，求优化

你的脚本存在几个关键问题导致功能异常，以下是具体问题和优化后的实现：

原脚本的核心问题 #

• cut命令语法错误：cut -d "":" -f3 里的引号位置错误，应该是-d ":"，错误写法会导致cut解析参数失败，输出不符合预期内容。
• UID存在性校验逻辑错误：[[ "$num" -eq "$hpass" ]] 无法正确判断，因为hpass是多行UID列表，-eq只能做单个数值比较，没法匹配列表内容。
• 冗余处理步骤：printf '%d\n' "$dbpasswd" 完全没必要，cut提取后已经是每行一个UID，sort -n之后直接就是格式化好的多行数值。

优化后的脚本 #

#!/bin/bash

# 正确提取/etc/passwd中的UID并排序，去掉冗余处理
used_uids=$(cut -d ":" -f3 /etc/passwd | sort -n)

while true; do
    read -p "Enter a number between 1000 and 59999: " num
    echo ""

    # 先校验输入是否为整数
    if ! [[ "$num" =~ ^[0-9]+$ ]]; then
        echo "Error: Please enter a valid integer."
        continue
    fi

    # 校验UID范围，修正边界判断包含1000
    if [[ "$num" -gt 999 && "$num" -lt 60000 ]]; then
        # 用grep精确匹配输入的UID是否存在于已使用列表中
        if echo "$used_uids" | grep -q "^$num$"; then
            echo "$num is already in use"
        else
            echo "$num is in range and not yet used!"
            break
        fi
    else
        echo "$num is out of range. Must be between 1000 and 59999."
    fi
done

优化说明 #

• 修正cut命令参数错误，确保正确提取UID。
• 添加输入合法性校验，避免非数字输入导致脚本报错。
• 用grep -q "^$num$"精确匹配UID，-q让grep静默执行，只返回退出状态，效率更高。
• 调整范围判断边界：原脚本-gt 1000会排除1000本身，改成-gt 999才能包含合法起始值1000。

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