Bash数值校验函数shellcheck无错却报then附近语法错误如何解决？

你代码的核心问题是if条件段和then关键字之间缺少必要的分隔符：如果then和条件判断放在同一行，必须在条件结束后加;，否则需要把then单独放在新行。你写的(( "$PICK" <= "$LIMIT" )) then没有分隔符，bash无法识别then的起始位置，就会触发syntax error near unexpected token 'then'报错。
另外你的校验函数还存在一个隐藏问题：你调用时传入了$PICK和$LIMIT两个参数，但函数内部直接读取全局变量，没有使用传入的参数，后续如果全局变量名变更，函数会直接失效。

#!/bin/bash
validateRange () {
    # 读取传入的参数，优先使用入参，兼容原有全局变量调用方式
    local pick=${1:-$PICK}
    local limit=${2:-$LIMIT}
    if (( pick > 0 )) && (( pick <= limit )); then 
        echo "valid number"
        return 0
    fi
    return 1
}
# 调用示例
(( LIMIT=4 ))
(( PICK=2 ))
validateRange "$PICK" "$LIMIT"

• 合并算术判断写法，简化单条件逻辑：

  # 只需一组算术扩展括号
  if (( pick > 0 && pick <= 4 )); then
    # 校验通过后的业务逻辑
  fi
  

• POSIX兼容写法，可在非bash的shell环境中运行：

  if [ "$pick" -gt 0 ] && [ "$pick" -le 4 ]; then
    # 校验通过后的业务逻辑
  fi
  

• 带输入格式校验的完整写法，提前过滤非数字输入：

  validateRange() {
    local pick=$1
    # 先判断输入是否为纯正整数，避免非数字输入触发算术报错
    if [[ ! $pick =~ ^[1-9][0-9]*$ ]]; then
      echo "输入不是合法正整数"
      return 1
    fi
    if (( pick > 0 && pick <=4 )); then
      echo "valid number"
      return 0
    fi
    echo "数字超出1-4的可选范围"
    return 1
  }
  

• 固定小范围场景下的case语句写法：

  case $pick in
    [1-4]) echo "valid number" ;;
    *) echo "非法输入" ;;
  esac
  

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