Логика логического вывода 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 - это правильная вещь, так как она дает вам большую гибкость. В противном случае, вы можете переписать ваш цикл таким образом:

#!/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

Так как кажется, вам всегда нужно два параметра можно изменить первую строку:

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

В противном случае вы можете проверить значение $ 2 до войти петля вот так:

Answer 3

Сдвиг сбой из-за недостаточного количества аргументов.

Эта проблема, однако, будет очевидна, так как вы устанавливаете свою оболочку для отказа при всех ошибках (с set -e). Во многих случаях отладка во многих случаях значительно упрощается, избегая бесшумных сбоев и обеспечивает проверку дисциплины и ошибок. set -u вызывает ошибки при неудачных расширениях переменных, также очень полезно.