为什么我的 shell 脚本在第二次尝试时失败

Question

此脚本应接受一组搜索词，并返回格式化的URL以搜索Google。

$ ./google_search.sh albert einstein
https://www.google.com/search?q=albert+einstein

它做得很好，所以我决定添加一个选项来搜索特定站点，或者使用 -s 或 -S 标志忽略该站点。

$ ./google_search.sh -s wikipedia.org albert einstein
https://www.google.com/search?q=albert+einstein+site%3Awikipedia.org

这在您第一次运行脚本时有效，但在每次后续尝试时都会失败。

$ ./google_search.sh -s wikipedia.org albert einstein
https://www.google.com/search?q=albert+einstein
$ ./google_search.sh -s wikipedia.org albert einstein
https://www.google.com/search?q=albert+einstein

打开新的终端窗口或重新启动终端都可以清除此问题，并在失败之前允许再尝试一次。

脚本：

#!/bin/bash
# original source of concatenate_args function by Tyilo:
# http://stackoverflow.com/questions/9354847/concatenate-inputs-in-bash-script
function concatenate_args
{
    string=""
    ignorenext=0
    for a in "$@" # Loop over arguments
    do
        if [[ "${a:0:1}" != "-" && $ignorenext = 0 ]] # Ignore flags (first character is -)
        then
            if [[ "$string" != "" ]]
            then
                string+="+" # Delimeter
            fi
            string+="$a"
        elif [[ $ignorenext = 1 ]]
        then
            ignorenext=0
        else
            ignorenext=1
        fi
    done
    echo "$string"
}
qry="$(concatenate_args "$@")"
glink="https://www.google.com/search?q="
site=""
while getopts :s:S: opt; do
    case $opt in
        s) site="+site%3A$OPTARG" ;; 
        S) site="+-site%3A$OPTARG" ;; 
    esac
done
url=$glink$qry$site
echo $url
# open -a Firefox $url

需要更改哪些内容才能使此脚本更可靠？

Answer 1

这表现得像您正在获取脚本而不是执行它。 如果在脚本前使用点和空格，则会导致脚本在当前 shell 中逐行执行，而不是创建新的 shell。 这允许在脚本中更改的环境变量泄漏到当前 shell 的环境中，这会使脚本的一次运行与下一次运行的行为不同。

在这种情况下，它似乎是使用getopts。 getOpts 每次调用环境变量时都会更新该变量，以便它可以跟踪正在检查的参数。 第二次获取脚本时，它认为所有参数都已经过检查，因此您的参数最终被忽略。

Answer 2

您的脚本，简化：

#!/bin/bash
glink="https://www.google.com/search?q="
site=""
# if you're "source"ing, uncomment the following:
# OPTIND=1
while getopts :s:S: opt; do
    case $opt in
        s) site="+site:$OPTARG" ;; 
        S) site="+-site:$OPTARG" ;; 
        ?) echo "invalid option: -$OPTARG" >&2 ;;
    esac
done
shift $((OPTIND - 1))
# the positional parameters are now clear of "-s" and "-S" options
qry=$(IFS="+"; echo "$*")
url=$glink$qry$site
echo "$url"
# open -a Firefox "$url"