Bash, если оператор с оператором И

Question

Попытка сравнения с переменной против строк, я попытался код, согласно решению от https://unix.stackexchange.com/questions/67898/using-the-not-equal-operator-for-string-comparison

if [ "$ACTION" != "dry" ] && [ "$ACTION" != "prune" ] 
then 
    echo "Invalid" 
fi 

Это не работает для меня, я не получаю сообщения об ошибках, это как это просто пропускает блок кода.

Я также пробовал так, как в ответ здесь Bash if statement with multiple conditions

if [[ "$ACTION" != "dry" && "$ACTION" != "prune" ]] 
then 
    echo "Invalid" 
fi 

Это перекликается «Invalid», если $ ACTION ничего, кроме «сухой», даже если его «подрезать»

Все предложения ?

---

EDIT

Полный код

OPTIND=1 
while getopts "b:a:" option 
do 
    case "${option}" 
    in 
     b) MERGE_BRANCH=${OPTARG};; 
     a) ACTION=${OPTARG};; 
    esac 
done 

if [[ "$ACTION" != "dry" && "$ACTION" != "prune" ]] 
then 
    echo "Invalid" 
fi 

shift $((OPTIND-1)) 
[ "$1" = "--" ] && shift 

((1 <= ${#})) || { echo "missing mandatory argument" 2>&1 ; exit 1;  }; 

Answer 1

Ну, в большинстве ситуаций, когда возникает такая команда, с помощью case обеспечивает более ремонтопригодны решение, потому что в течение жизненного цикла вашего сценария вы захотите изменить имена действий, справиться с новыми ситуациями и так далее:

case "$ACTION" in 
    ("dry"|"prune") 
    : # Insert appropriate code 
    ;; 
    (*) 
    echo Invalid 
    ;; 
esac 

Во-вторых, нет необходимости использовать синтаксис [[ в этом случае

if [ "$ACTION" != dry ] && [ "$ACTION" != prune ]; then 
    echo Invalid 
fi 

достаточно в этом случае. Для того, чтобы проверить, что все работает, как ожидалось, можно добавить else ветвь:

if [ "$ACTION" != dry ] && [ "$ACTION" != prune ]; then 
    echo Invalid 
else 
    echo Valid 
fi 

Answer 2

Решение

Пользуясь случаем согласно @ ответ Дарио и перемещение чек после операции сдвига делает код работу как предполагалось.

OPTIND=1 

while getopts "b:a:" option 
do 
    case "${option}" 
    in 
     b) MERGE_BRANCH=${OPTARG};; 
     a) ACTION=${OPTARG};; 
    esac 
done 

shift $((OPTIND-1)) 
[ "$1" = "--" ] && shift 

((1 <= ${#})) || { echo "missing mandatory argument" 2>&1 ; exit 1; }; 

case "$ACTION" in 
    (dry|prune) 
    : 
    ;; 
    (*) 
    echo "Invalid argument" 
    exit 1 
    ;; 
esac