Bash引数の解析ロジックが無限ループに入ります

Question

コマンドライン引数を解析するループがあります。 -dまたは-b引数にパラメータが指定されていれば、すべてうまく動作します。しかし、パラメータなしで-dまたは-bを渡すと、スクリプトは無限ループに入ります。

# Parse command line input 
while [ "$#" -gt 0 ]; do 
    case "$1" in 
    -d) DOMAIN="$2"; shift 2;; 
    -b) BITS="$2"; shift 2;; 
    *) die "unrecognized argument: $1" 
    esac 
done 

空のパラメータでエラーをスローするにはどうすればよいですか？

Answer 1

リストにある値の数を超えて移動しようとすると、shift nコマンドは何も行いません。これがコード内の無限ループの理由です。

getoptsを使用すると、多くの柔軟性が得られます。そうしないとあなたが入力する前に、$ 2の値を確認することができます

while [ "$#" -eq 2 ]; do 

：あなたが常にあなたが最初の行を変えることができる2つのパラメータが必要と思われますので

#!/bin/bash 
# Parse command line input 
d_flag=0 
b_flag=0 
while [ "$#" -gt 0 ]; do 
    case "$1" in 
    -d) 
     d_flag=1 
     shift 
     [[ $1 == -* ]] && continue # argument to -d not given 
     DOMAIN=$1 
     shift 
     ;; 
    -b) 
     b_flag=1 
     shift 
     [[ $1 == -* ]] && continue # argument to -b not given 
     BITS=$1 
     shift 
     ;; 
    *) echo "unrecognized argument: $1"; exit 2;; 
    esac 
done 

if [[ $d_flag = 1 && -z $DOMAIN ]]; then 
    # handle error - option requires an argument 
fi 

if [[ $b_flag = 1 && -z $BITS ]]; then 
    # handle error - option requires an argument 
fi 

Answer 2

：そうでなければ、あなたはあなたのループをこのように書き換えることができ

if [[ -z $2 ]]; then 
echo "missing second parameter" 
else 
#do the while loop 
fi 

Answer 3

引数が足りないためにシフトが失敗します。

しかし、すべてのエラー（set -e）でシェルが失敗するように設定すると、この問題はすぐに明らかになります。サイレント・エラーを避けることでデバッグを容易にし、規律とエラー・チェックを強制します。 set -uは、失敗した変数の展開でエラーを発生させ、非常に便利です。