Por que o ramo 'if [$ 1 = “1”]' sempre é selecionado, mesmo que $ 1 não seja 1?

Eu tenho um script de shell chamado 'teleport.sh' assim:

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

Quando executo:

sh teleport.sh 2 testfile

Isso testfileé movido para o ~/lab/Sundiretório, o que me confunde muito porque eu não passei 1 ou '1' para esse script.

O que há de errado aqui?

O uso de espaços corrige seu problema.

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

Embora isso seja mais puro:

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

A primeira coisa óbvia é que você deve fornecer espaços entre os argumentos de [, testou [[:

if [ "$1" = 1 ];

No Bash, o uso [[ ]]é recomendado, pois não faz coisas desnecessárias para a expressão condicional, como divisão de palavras e expansão do nome do caminho. Também não é necessário citar aspas duplas. Um operador mais legível ==também pode ser usado.

if [[ $1 == 1 ]];

Nota acrescentou: Se segundo operando também contém variáveis, citando é necessária, pois podem estar sujeitos a correspondência de padrão se ele contém personagens reconhecíveis como *, ?, [], etc .. Se estendida englobamento ou correspondência padrão é ativado com shopt -s extglob, outras formas como @(), !(), etc. também será reconhecido como padrões. Consulte Correspondência de padrões .

Com operadores como <e >ainda pode ser necessário, pois uma vez encontrei um bug em que não citar o segundo argumento causou resultados diferentes.

Quanto ao primeiro operando, nada se aplica.

Considere também essa variação mais simples:

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

Ou condensado:

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

"${@:2}"é uma forma de expansão de substring ou expansão de membro da matriz onde 2está o deslocamento. Isso faz com que a expansão comece com o segundo valor. Com isso, talvez não precisemos usar shift.

O --item adicionado impede o mvreconhecimento de nomes de arquivos iniciados por dash ( -) como opções inválidas.

Para responder à pergunta de por que isso está acontecendo, esse comportamento de [aka testestá documentado no POSIX :

Na lista a seguir, $ 1, $ 2, $ 3 e $ 4 representam os argumentos apresentados para testar:

[...]

1 argumento:

Sair verdadeiro (0) se $ 1 não for nulo; caso contrário, saia falso.

Você está passando um argumento, 2=1que não é nulo e, portanto, testsai com sucesso.

Como outras postagens (e verificação de shell ) apontam, se você quiser comparar a igualdade, terá que passar os 3 argumentos 2, =e 1.

Eu só quero recomendar uma alternativa portátil, mas também mais limpa. O Bash não é universal (e se você não precisa de universal, por que está escrevendo um shell script?)

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

(Observação para os pedantes: sim, eu sei que as citações $actionna case "$action" inlinha são desnecessárias, mas acho que é melhor colocá-las lá de qualquer maneira, para que futuros leitores não precisem se lembrar disso.)