Question

我已经在bash_aliases中创建了一个脚本，以便更轻松地将SSH和服务器放到服务器上。但是，我得到了一些我不理解的奇怪行为。除了重复使用之外，下面的脚本可以按照您的预期运行。

如果我第一次在shell中使用它，它的工作方式完全符合预期：

$>sdev -s myservername
ssh -i ~/.ssh/id_rsa currentuser@myservername.devdomain.com

但是，如果我第二次运行，而没有指定-s|--server，那么它将使用上次运行时的服务器名称，似乎缓存了它：

$>sdev
ssh -i ~/.ssh/id_rsa currentuser@myservername.devdomain.com

它应该退出并显示错误并输出以下消息：/bin/bash: A server name (-s|--server) is required.

任何参数都会发生这种情况;也就是说，如果我指定一个参数，然后下次我不这样做，这个方法将使用上次提供的参数。

显然，这不是我想要的行为。我的脚本负责这样做，我该如何解决？

#!/bin/bash

sdev() {
    getopt --test > /dev/null

    if [[ $? -ne 4 ]]; then
        echo "`getopt --test` failed in this environment"
        exit 1
    fi

    OPTIONS=u:,k:,p,s:
    LONGOPTIONS=user:,key:,prod,server:

    # -temporarily store output to be able to check for errors
    # -e.g. use “--options” parameter by name to activate quoting/enhanced mode
    # -pass arguments only via   -- "$@"   to separate them correctly
    PARSED=$(getopt --options=$OPTIONS --longoptions=$LONGOPTIONS --name "$0" -- "$@")

    if [[ $? -ne 0 ]]; then
        # e.g. $? == 1
        #  then getopt has complained about wrong arguments to stdout
        exit 2
    fi

    # read getopt’s output this way to handle the quoting right:
    eval set -- "$PARSED"

    domain=devdomain
    user="$(whoami)"
    key=id_rsa

    # now enjoy the options in order and nicely split until we see --
    while true; do
        case "$1" in
            -u|--user)
                user="$2"
                shift 2
                ;;
            -k|--key)
                key="$2".pem
                shift 2
                ;;
            -p|--prod)
                domain=proddomain
                shift
                ;;
            -s|--server)
                server="$2"
                shift 2
                ;;
            --)
                shift
                break
                ;;
            *)
                echo "Programming error"
                exit 3
                ;;
        esac
    done

    if [ -z "$server" ]; then
        echo "$0: A server name (-s|--server) is required."
        kill -INT $$
    fi

    echo "ssh -i ~/.ssh/$key.pem $user@$server.$domain.com"
    ssh -i ~/.ssh/$key $user@$server.$domain.com
}

Answer 1

server是一个全局shell变量，因此它在函数的运行之间共享（只要它们在同一个shell中运行）。也就是说，当您运行sdev -s myservername时，它会将变量server设置为＆＃34; myservername＆＃34;。稍后，当您仅运行sdev时，它会检查$server是否为空，发现它不是，然后继续使用它。

解决方案：使用局部变量！实际上，最好将函数中使用的所有变量声明为local;这样，您就不会冒着干扰尝试使用相同变量名称的其他内容的风险。我还建议避免使用全大写变量名称（例如OPTIONS，LONGOPTIONS和PARSED） - 有一堆全大写变量具有特殊含义shell和/或其他程序，如果你错误地使用其中一个，它可能会导致奇怪的问题。

无论如何，这是一个简单的解决方案：在剧本开头附近添加：

local server=""

getopt错误地缓存参数

1 个答案: