Warum wird der Zweig "if [$ 1 =" 1 "]" immer ausgewählt, auch wenn $ 1 nicht 1 ist?

Ich habe ein Shell-Skript namens 'teleport.sh' wie folgt:

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

Wenn ich ausführe:

sh teleport.sh 2 testfile

Dies testfilewird in das ~/lab/SunVerzeichnis verschoben , was mich sehr verwirrt, da ich nicht 1 oder '1' an dieses Skript übergeben habe.

Was ist hier los?

Die Verwendung von Leerzeichen behebt Ihr Problem.

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

Obwohl dies ordentlicher ist:

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

Die erste offensichtliche Sache ist , Sie Räume zwischen den Argumenten liefern sollten [, testoder [[:

if [ "$1" = 1 ];

In Bash wird die Verwendung [[ ]]empfohlen, da hierdurch keine für bedingte Ausdrücke unnötigen Funktionen wie Wortaufteilung und Pfadnamenerweiterung erforderlich sind. Das Zitieren um doppelte Anführungszeichen ist ebenfalls nicht erforderlich. Ein besser lesbarer Operator ==kann ebenfalls verwendet werden.

if [[ $1 == 1 ]];

Hinzugefügt Anmerkung: Wenn zweiter Operand auch Variablen enthält, unter Angabe ist notwendig , da es je nach Musteranpassung sein kann , wenn es erkennbar Zeichen enthält wie *, ?, [], etc .. Wenn die erweiterte Globbing oder Pattern - Matching aktiviert mit shopt -s extglob, anderen Formen wie @(), !()usw. wird auch als Muster erkannt. Siehe Mustervergleich .

Bei Operatoren wie <und >kann es dennoch notwendig sein, da ich einmal auf einen Fehler gestoßen war, bei dem das Nichtzitieren des zweiten Arguments zu unterschiedlichen Ergebnissen führte.

Für den ersten Operanden gilt nichts.

Betrachten Sie auch diese einfachere Variante:

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

Oder verdichtet:

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

"${@:2}"ist eine Form der Teilstringerweiterung oder Arrayelementerweiterung, wobei 2der Versatz ist. Dadurch beginnt die Erweiterung beim zweiten Wert. Damit müssen wir möglicherweise nicht verwenden shift.

Das hinzugefügte --verhindert, mvdass Dateinamen, die mit dash ( -) beginnen, als ungültige Optionen erkannt werden .

Zur Beantwortung der Frage, warum dies geschieht, dieses Verhalten von [aka testwird in POSIX dokumentiert :

In der folgenden Liste stehen $ 1, $ 2, $ 3 und $ 4 für die zu testenden Argumente:

[...]

1 Argument:

Beenden Sie true (0), wenn $ 1 nicht null ist. Andernfalls beenden Sie false.

Sie übergeben 1 Argument, 2=1das nicht null ist und daher testmit Erfolg beendet wird.

Wie andere Stellen (und shellcheck ) weisen darauf hin, wenn Sie auf Gleichheit vergleichen wollten, würden Sie stattdessen die drei Argumente übergeben müssen 2, =und 1.

Ich möchte nur eine tragbare, aber auch ordentlichere Alternative empfehlen. Bash ist nicht universell (und wenn Sie kein universelles benötigen, warum schreiben Sie ein Shell-Skript?)

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

(Hinweis für Pedanten: Ja, ich weiß, dass die Anführungszeichen $actionin der case "$action" inZeile unnötig sind, aber ich denke, es ist am besten, sie trotzdem dort zu platzieren, damit zukünftige Leser sich nicht daran erinnern müssen.)