简单的 Bash 脚本不起作用

Question

我正在处理的 shell 脚本有问题，无论出于何种原因，它都没有按预期工作。脚本的大部分内容以及出错的地方如下：

for username in $list
do
   sleep $sleep
   VAL=`curl -H 'Client-ID: $clientid' -s $host$username | jq -r .message | grep -E "unavailable|not found"`
   if [[ $VAL == *"not found"* ]]; then
     echo "$username is available"
     echo "$username" >> available.names
   else
     echo -e "$username is reserved"
   fi
done

我知道有一些变量，例如 sleep、host、clientid 和 username，但为了让您了解我正在使用的内容，curl 命令的结果在VAL 行（减去 grep）会是这样的：

User "username" was not found
or
User "username" is unavailable

因此，包括到grep -E 的管道，VAL 的结果将类似于：

not found

现在根据这个StackOverflow answer，我应该能够使用通配符，以便在使用我的脚本时，如果VAL 包含“未找到”，那么它应该回显用户名可用。正如我在这里所做的那样：

[[ $VAL == *"not found"* ]]

但是，它正在做的是 else 语句，用于检查所有正在检查的用户名。因此，即使VAL 包含“未找到”，它仍然会显示用户名已被使用。

有没有人发现我的脚本有什么问题会导致这种情况发生？我已经查看了大约 100 次，没有任何错误可以帮助进行故障排除，只是它没有按预期工作。任何帮助将不胜感激。

Answer 1

-H 'Client-ID: $clientid' 中的单引号显然是错误的， 因为$clientid 的值不会以这种方式扩展。 请改用双引号。

顺便说一句，你不需要[[ $VAL == *"not found"* ]]， 您可以直接使用管道的grep 的退出代码编写条件：

for username in $list
do
   sleep $sleep
   if curl -H "Client-ID: $clientid" -s "$host$username" | jq -r .message | grep -qE "unavailable|not found"; then
     echo "$username is available"
     echo "$username" >> available.names
   else
     echo "$username is reserved"
   fi
done

正如@CharlesDuffy 在评论中指出的那样，echo -e 毫无意义，所以我也删除了它。 他还写了for username in $list指出了代码异味， 而不是使用正确的数组。 (list=( allison bob charlie ); for username in "${list[@]}"; do ...)