为何ShellCheck报SC2128警告？我的Bash脚本问题咨询

我编写了一段Bash脚本，定义is_option函数检查输入选项是否有效。调用该函数时使用"$options"传递数组，触发ShellCheck的SC2128警告：未带索引展开数组仅返回第一个元素。我在函数内使用了"${options[@]}"，且参考了ShellCheck的正确示例，仍不理解为何报错。

我的脚本代码 #

#!/bin/bash

is_option(){
  for option in "${options[@]}"
  do
    if [[ "${option}" == "$choice" ]]
      then
          return 0
      fi
  done
  return 1
}

options=("1" "2" "3" "4" "5")
echo "Enter choice"
read -r choice
is_option "$options" "$choice"

if is_option "$options" "$choice"
then
  echo "Valid choice"
else
  echo "Invalid choice"
fi

ShellCheck输出 #

$ shellcheck myscript
 
Line 17:
is_option "$options" "$choice"
           ^-- SC2128 (warning): Expanding an array without an index only gives the first element.
 
Line 19:
if is_option "$options" "$choice"
              ^-- SC2128 (warning): Expanding an array without an index only gives the first element.

$ 

参考的ShellCheck示例 #

myarray=(foo bar)
for f in "${myarray[@]}"
do
  cat "$f"
done

问题原因 #

1. 调用函数时的"$options"确实只会返回数组的第一个元素，这是警告的直接来源。
2. 你的函数依赖全局变量options和choice，调用时传递的"$options"和"$choice"根本没被函数使用，属于无效传递，反而因为错误的数组展开方式触发了警告。

解决方案 #

推荐使用第二种方案（避免全局变量依赖，更符合Shell脚本规范）：

方案1：保留全局变量，移除无效参数传递

修改函数，只接收choice参数：

is_option(){
  local choice="$1"
  for option in "${options[@]}"
  do
    if [[ "${option}" == "$choice" ]]
    then
        return 0
    fi
  done
  return 1
}

调用时不再传递数组：

is_option "$choice"

if is_option "$choice"
then
  echo "Valid choice"
else
  echo "Invalid choice"
fi

方案2：通过函数参数传递数组和选项（更规范）

修改函数，让它通过参数接收目标选项和所有数组元素，完全脱离全局变量：

is_option(){
  local choice="$1"
  shift # 移除第一个参数，剩余参数即为选项数组元素
  for option in "$@"
  do
    if [[ "${option}" == "$choice" ]]
    then
        return 0
    fi
  done
  return 1
}

调用时正确展开数组所有元素：

is_option "$choice" "${options[@]}"

if is_option "$choice" "${options[@]}"
then
  echo "Valid choice"
else
  echo "Invalid choice"
fi

这样既解决了SC2128警告，又提升了函数的复用性，避免了全局变量带来的潜在问题。

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