¿Por qué la rama 'if [$ 1 = “1”]' siempre se selecciona incluso si $ 1 no es 1?

Tengo un script de shell llamado 'teleport.sh' como este:

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

Cuando ejecuto:

sh teleport.sh 2 testfile

Esto testfilese mueve al ~/lab/Sundirectorio, lo que me confunde mucho ya que no pasé 1 o '1' a ese script.

¿Qué pasa aquí?

Usar espacios soluciona tu 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

Aunque esto es más ordenado:

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

La primera cosa obvia es que debe proporcionar espacios entre los argumentos de [, testo [[:

if [ "$1" = 1 ];

Cuando está en Bash, [[ ]]se recomienda el uso, ya que no hace cosas innecesarias para la expresión condicional, como la división de palabras y la expansión de nombres de ruta. Citar alrededor de comillas dobles tampoco es necesario. ==También se puede utilizar un operador más legible .

if [[ $1 == 1 ]];

Se agregó una nota: Si segundo operando también contiene variables, citando es necesario, ya que puede estar sujeta a la coincidencia de patrones si contiene caracteres reconocibles como *, ?, [], etc .. Si englobamiento extendido o coincidencia de patrones está habilitado con shopt -s extglob, otras formas como @(), !(), etc. También será reconocido como patrones. Ver coincidencia de patrones .

Con operadores similares <y >todavía puede ser necesario, ya que una vez me encontré con un error en el que no citar el segundo argumento causaba resultados diferentes.

En cuanto al primer operando, no se aplica nada.

Considere también esta variación más simple:

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

O condensado:

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

"${@:2}"es una forma de expansión de subcadena o expansión de miembro de matriz donde 2está el desplazamiento. Esto hace que la expansión comience en el segundo valor. Con esto puede que no necesitemos usar shift.

El agregado --impide mvreconocer nombres de archivo que comienzan con guión ( -) como opciones no válidas.

Para responder a la pregunta de por qué sucede esto, este comportamiento de [aka testestá documentado en POSIX :

En la siguiente lista, $ 1, $ 2, $ 3 y $ 4 representan los argumentos presentados para probar:

[...]

1 argumento:

Salga verdadero (0) si $ 1 no es nulo; de lo contrario, salga falso.

Le está pasando 1 argumento, 2=1que no es nulo y, por lo tanto, testsale con éxito.

Como señalan otras publicaciones (y shellcheck ), si quisiera comparar por igualdad, en su lugar tendría que pasar los 3 argumentos 2, =y 1.

Solo quiero recomendar una alternativa portátil pero también más ordenada. Bash no es universal (y si no necesita universal, ¿por qué está escribiendo un script de shell?)

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

(Nota para los pedantes: sí, sé que las citas $actionen la case "$action" inlínea son innecesarias, pero creo que es mejor ponerlas allí de todos modos, para que los futuros lectores no tengan que recordar eso).