$ 1が1でなくても、 'if [$ 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 ]];

メモを追加した：第2オペランドはまた、変数を含む場合、引用それはのような認識可能な文字が含まれている場合、それは、パターンマッチングの対象とすることができるように必要であり*、?、[]、など。場合はグロブ延長またはパターンマッチングを用いて有効になっているshopt -s extglobような、他の形態@()、!()などパターンとしても認識されます。パターンマッチングを参照してください。

のような演算子を使用する<と>、2番目の引数を引用しないと異なる結果が発生するバグにかつて遭遇したことがあるので、まだ必要な場合があります。

第1オペランドについては、何も適用されません。

この単純なバリエーションも検討してください。

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オフセットである部分文字列展開または配列メンバー展開の形式です。これにより、2番目の値から拡張が開始されます。これにより、使用する必要がない場合がありますshift。

追加された--機能によりmv、ダッシュ（-）で始まるファイル名が無効なオプションとして認識されなくなります。

なぜこれが起こっているの質問に答えるために、この振る舞い[通称testされるPOSIXで文書化：

次のリストで、$ 1、$ 2、$ 3、および$ 4は、テストのために提示された引数を表します。

[...]

1つの引数：

$ 1がnullでない場合はtrue（0）を終了します。それ以外の場合は、falseを終了します。

2=1nullではない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"

（pedantsへの注意：はい、私は前後に引用符を知っている$action上のcase "$action" in不要なラインが、私はそれが将来の読者はそれを覚えておく必要はありませんので、とにかくそこにそれらを置くことが最善であると感じています。）