Чому гілка '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, інші форми , як @(), !()і т.д. також будуть визнані зразками. Дивіться відповідність шаблонів .

З такими операторами, як, <і >це все ще може знадобитися, оскільки я колись стикався з помилкою, коли не цитування другого аргументу спричинило різні результати.

Щодо першого операнда, то нічого не стосується.

Розглянемо також цю простішу варіацію:

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розпізнавати назви файлів, починаючи з тире ( -), як недійсні параметри.

Для того, щоб відповісти на питання про те, чому це відбувається, така поведінка [інакше testяк описано в стандарті POSIX :

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

[...]

1 аргумент:

Вийдіть true (0), якщо $ 1 не є нульовим; в іншому випадку вийдіть помилково.

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

Як і інші посади (і shellcheck ) вказують, якщо ви хочете , щоб порівняти рівності, ви б замість цього повинні пройти 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рядку непотрібні, але я вважаю, що найкраще все-таки розміщувати їх там, щоб майбутнім читачам не потрібно було пам’ятати про це.)