Почему ветвь 'if [$ 1 = “1”] ”всегда выбирается, даже если $ 1 не равен 1?

У меня есть сценарий оболочки с именем «teleport.sh», как это:

if [ $1="1" ];
    then
    shift
        mv "$@" ~/lab/Sun
elif [ $1="2" ];
    then
    shift
        mv "$@" ~/lab/Moon
elif [ $1="3" ];
    then
    shift
        mv "$@" ~/lab/Earth
fi

Когда я выполню:

sh teleport.sh 2 testfile

Это testfileперемещено в ~/lab/Sunкаталог, что меня очень смущает, так как я не передал 1 или '1' в этот скрипт.

Что здесь не так?

Использование пробелов решает вашу проблему.

if [ "$1" = 1 ];
    then
    shift
        mv "$@" ~/lab/Sun
elif [ "$1" = 2 ];
    then
    shift
        mv "$@" ~/lab/Moon
elif [ "$1" = 3 ];
    then
    shift
        mv "$@" ~/lab/Earth
fi

Хотя это аккуратнее

#!/bin/bash

action=$1
shift
files=("$@")
case $action in  
  1) mv -- "${files[@]}" ~/lab/Sun     ;;
  2) mv -- "${files[@]}" ~/lab/Moon    ;;
  3) mv -- "${files[@]}" ~/lab/Earth   ;;
esac

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

if [ "$1" = 1 ];

В Bash [[ ]]рекомендуется использовать, так как он не делает ненужных вещей для условного выражения, таких как разбиение слов и расширение пути. Цитировать вокруг двойных кавычек тоже не нужно. Также ==может быть использован более читаемый оператор .

if [[ $1 == 1 ]];

Добавлено примечание: Если второй операнд также содержит переменные, цитирует необходимо , так как это может быть при условии сопоставления с образцом , если он содержит узнаваемые символы , такие как *, ?, []и т.д .. Если расширенная подстановки или соответствующий шаблон включен shopt -s extglob, другие формы , как @(), !()и т.д. также будет распознаваться как шаблоны. Смотрите Pattern Matching .

С такими операторами, как <и >это, все еще может быть необходимо, поскольку я однажды столкнулся с ошибкой, при которой не цитирование второго аргумента приводило к другим результатам.

Что касается первого операнда, ничего не применяется.

Рассмотрим и этот более простой вариант:

case "$1" in
1)
    mv -- "${@:2}" ~/lab/Sun
    ;;
2)
    mv -- "${@:2}" ~/lab/Moon
    ;;
3)
    mv -- "${@:2}" ~/lab/Earth
    ;;
esac

Или сжатый:

case "$1" in
1) mv -- "${@:2}" ~/lab/Sun ;;
2) mv -- "${@:2}" ~/lab/Moon ;;
3) mv -- "${@:2}" ~/lab/Earth ;;
esac

"${@:2}"является формой расширения подстроки или расширения элемента массива, где 2это смещение. Это заставляет расширение начинаться со второго значения. С этим нам, возможно, не понадобится пользоваться shift.

Добавленное --предотвращает mvраспознавание имен файлов, начинающихся с dash ( -), как недопустимых параметров.

Для того, чтобы ответить на вопрос о том, почему это происходит, такое поведение [иначе testкак описано в стандарте POSIX :

В следующем списке $ 1, $ 2, $ 3 и $ 4 представляют аргументы, представленные для проверки:

[...]

1 аргумент:

Выйдите из true (0), если $ 1 не равно нулю; в противном случае выйдите из false.

Вы передаете ему 1 аргумент, 2=1который не является нулевым, и поэтому testзавершается с успехом.

Как указывают другие сообщения (и проверка оболочки ), если вы хотите сравнить на равенство, вам вместо этого нужно будет передать 3 аргумента 2, =и 1.

Я просто хочу порекомендовать портативную, но также более аккуратную альтернативу. Bash не универсален (и если вам не нужен универсальный, зачем вы пишете скрипт оболочки?)

#! /bin/sh
action="$1"
shift
case "$action" in
    1) dest=Sun   ;;
    2) dest=Moon  ;;
    3) dest=Earth ;;
    *) echo "Unrecognized action code '$action' (must be 1, 2, or 3)" >&2; exit 1 ;;
esac
mv -- "$@" ~/lab/"$dest"

(Примечание для педантов: да, я знаю, что в кавычках $actionна case "$action" inстрочке нет необходимости, но я чувствую, что в любом случае лучше поместить их туда, чтобы будущие читатели не должны были это помнить.)