Perché il ramo 'if [$ 1 = “1”]' è sempre selezionato anche se $ 1 non è 1?

Ho uno script di shell chiamato 'teleport.sh' come questo:

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 eseguo:

sh teleport.sh 2 testfile

Questo testfileviene spostato nella ~/lab/Sundirectory, il che mi confonde molto perché non ho passato 1 o '1' a quello script.

Cosa c'è che non va qui?

L'uso degli spazi risolve il 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

Anche se questo è più ordinato:

#!/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 prima cosa ovvia è che dovresti fornire spazi tra gli argomenti di [, testo [[:

if [ "$1" = 1 ];

Quando sei in Bash, [[ ]]si consiglia di utilizzare in quanto non fa cose inutili per l'espressione condizionale come la suddivisione delle parole e l'espansione del percorso. Non è inoltre necessario citare virgolette doppie. È ==inoltre possibile utilizzare un operatore più leggibile .

if [[ $1 == 1 ]];

Aggiunta di una nota: se secondo operando contiene anche variabili, citando è necessario in quanto può essere soggetto a pattern matching se contiene caratteri riconoscibili come *, ?, [], ecc .. Se il globbing esteso o pattern matching viene abilitata con shopt -s extglob, altre forme come @(), !()ecc saranno anche riconosciuti come modelli. Vedi Corrispondenza dei motivi .

Con operatori simili <e >potrebbe essere ancora necessario, poiché una volta avevo riscontrato un bug in cui non citare il secondo argomento causava risultati diversi.

Per quanto riguarda il primo operando, non si applica nulla.

Considera anche questa variazione più semplice:

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

O condensato:

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

"${@:2}"è una forma di espansione della sottostringa o espansione dei membri dell'array in cui si 2trova l'offset. Questo fa iniziare l'espansione dal secondo valore. Con questo potrebbe non essere necessario utilizzare shift.

L'aggiunta --impedisce mvdi riconoscere i nomi di file che iniziano con trattino ( -) come opzioni non valide.

Per rispondere alla domanda sul perché questo accada, questo comportamento di [aka testè documentato in POSIX :

Nel seguente elenco, $ 1, $ 2, $ 3 e $ 4 rappresentano gli argomenti presentati per il test:

[...]

1 argomento:

Esci da true (0) se $ 1 non è null; altrimenti, esci da falso.

Lo stai passando 1 argomento, 2=1che non è nullo, e quindi testesce con successo.

Come altri posti (e shellcheck ) indicare, se si voleva confrontare per l'uguaglianza, si dovrebbe invece avere per passare gli argomenti 3 2, =e 1.

Voglio solo raccomandare un'alternativa portatile ma anche più ordinata. Bash non è universale (e se non hai bisogno di universal, perché stai scrivendo uno script di 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 per i pedanti: sì, so che le virgolette $actionsulla case "$action" inriga non sono necessarie, ma penso che sia meglio metterle lì, in modo che i futuri lettori non debbano ricordarlo.)