Pourquoi la branche 'if [$ 1 = “1”]' est-elle toujours sélectionnée même si $ 1 n'est pas 1?

J'ai un script shell nommé 'teleport.sh' comme ceci:

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

Quand j'exécute:

sh teleport.sh 2 testfile

Ceci testfileest déplacé vers le ~/lab/Sunrépertoire, ce qui me déroute beaucoup car je n'ai pas passé 1 ou '1' à ce script.

Qu'est-ce qui ne va pas ici?

L'utilisation d'espaces résout votre problème.

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

Bien que ce soit plus propre:

#!/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 première chose évidente est que vous devez fournir des espaces entre les arguments de [, testou [[:

if [ "$1" = 1 ];

Dans Bash, l'utilisation [[ ]]est recommandée car elle ne fait pas de choses inutiles pour l'expression conditionnelle comme le fractionnement de mots et l'expansion de noms de chemin. Il n'est pas non plus nécessaire de citer des guillemets doubles. Un opérateur plus lisible ==peut également être utilisé.

if [[ $1 == 1 ]];

Ajout d'une note: Si un deuxième opérande contient également des variables, citant est nécessaire car il peut être soumis à la correspondance de motif si elle contient des caractères reconnaissables comme *, ?, [], etc .. si elle était étendue englobement ou correspondance de motif est activé avec shopt -s extglob, d' autres formes comme @(), !(), etc. seront également reconnus comme des modèles. Voir Correspondance de motifs .

Avec des opérateurs comme <et >cela peut encore être nécessaire car j'avais rencontré un bug où ne pas citer le deuxième argument provoquait des résultats différents.

Quant au premier opérande, rien ne s'applique.

Considérez également cette variante plus simple:

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

Ou condensé:

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

"${@:2}"est une forme d'extension de sous-chaîne ou d'extension de membre de tableau où 2est le décalage. Cela fait démarrer l'expansion à la deuxième valeur. Avec cela, nous n'aurons peut-être pas besoin d'utiliser shift.

L'ajout --empêche mvde reconnaître les noms de fichiers commençant par dash ( -) comme des options invalides.

Pour répondre à la question de savoir pourquoi cela se produit, ce comportement de [aka testest documenté dans POSIX :

Dans la liste suivante, $ 1, $ 2, $ 3 et $ 4 représentent les arguments présentés pour tester:

[...]

1 argument:

Quittez true (0) si $ 1 n'est pas nul; sinon, quittez false.

Vous lui passez 1 argument, 2=1qui n'est pas nul, et se testtermine donc avec succès.

Comme le soulignent d'autres articles (et shellcheck ), si vous souhaitez comparer pour l'égalité, vous devrez plutôt passer les 3 arguments 2, =et 1.

Je veux juste recommander une alternative portable mais aussi plus soignée. Bash n'est pas universel (et si vous n'avez pas besoin d'universel, pourquoi écrivez-vous un script 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"

(Remarque pour les pédants: oui, je sais que les citations $actionsur la case "$action" inligne sont inutiles, mais je pense qu'il est préférable de les mettre de toute façon, afin que les futurs lecteurs n'aient pas à s'en souvenir.)