Verwenden Sie $? in einer if-Anweisung

function foo {
   (cd $FOOBAR;
   <some command>
   if [$? -ne 0]
   then
      echo "Nope!"
   else
      echo "OK!"
   fi
   )
}

Ich versuche, eine Funktion wie die oben beschriebene zu schreiben und in meine .bashrc-Datei einzufügen. Nachdem ich die Datei bezogen und ausgeführt habe, erhalte ich:

Gesamtzeit: 51 Sekunden
-Bash: [1: Befehl nicht gefunden
OK!

Kann mir jemand helfen zu verstehen, was ich falsch gemacht habe?

Fügen Sie ein Leerzeichen nach dem [und ein weiteres vor ]:

function foo {
   (cd $FOOBAR;
   <some command>
   if [ $? -ne 0 ]
   then
      echo "Nope!"
   else
      echo "OK!"
   fi
   )
}

[ein Shell - builtin ist, ist es ein Befehl wie echo, read, expr... es einen Raum , nachdem er braucht, und erfordert eine passende ].

Schreiben [ $? -ne 0 ]ist tatsächlich aufgerufen wird [und es vier Parameter geben: $?, -ne, 0, und ].

Hinweis: Die Tatsache, dass Sie eine Fehlermeldung erhalten, [1: command not foundbedeutet, $?dass der Wert von angegeben wurde 1.

Oder Sie könnten $?ganz überspringen . Wenn Ihr Befehl lautet cmd, sollte Folgendes funktionieren:

function foo {
   (cd $FOOBAR;
   if cmd
   then
      echo "OK!"
   else
      echo "Nope!"
   fi
   )
}

Es wird empfohlen, den Rückgabewert einer Variablen zuzuweisen, bevor Sie ihn verwenden

retval="$?"
if [ $retval -ne 0 ]

Sie können den Rückgabewert wiederverwenden. zB in einer if ... elif ... else ... Anweisung.

Der einzige Grund, warum Sie $?als Argumente für einen [Befehl verwenden möchten (unabhängig davon, ob dieser [Befehl im Bedingungsteil einer ifAnweisung ausgeführt wird oder nicht), besteht darin, dass Sie einen bestimmten Rückgabestatus unterscheiden möchten, z.

until
  cmd
  [ "$?" -gt 1 ]
do
  something
done

Die Syntax für alle diejenigen if, while, until... Aussagen

if cmd-list1
then cmd-list2
else cmd-list3
fi

Welches läuft, cmd-list2wenn cmd-list1es erfolgreich ist oder cmd-list3nicht.

Der [ "$?" -eq 0 ]Befehl ist ein No-Op. Es wird $?auf 0 gesetzt, wenn $?0 ist, und $?auf ungleich Null, wenn es nicht Null war.

Wenn Sie etwas ausführen möchten, wenn es cmdfehlgeschlagen ist, ist es:

if ! cmd
then ...
fi

Im Allgemeinen müssen Sie nicht basteln, $?geschweige denn wissen, welcher Wert bedeutet trueoder false. Die einzigen Fälle sind, wie oben erwähnt, wenn Sie einen bestimmten Wert unterscheiden müssen oder wenn Sie ihn für später speichern müssen (um ihn beispielsweise als Rückgabewert einer Funktion zurückzugeben), wie:

f() {
  cmd; ret=$?
  some cleanup
  return "$ret"
}

Denken Sie auch daran, dass der Operator split + glob eine Variable ohne Anführungszeichen bleibt. Es ist nicht sinnvoll, diesen Operator hier aufzurufen, daher sollte es sein:

[ "$?" -ne 0 ]

nicht [ $? -ne 0 ], geschweige denn [$? -ne 0 ](was den [Befehl nur aufrufen würde, wenn $IFSes das erste Zeichen von enthalten würde $?).

Beachten Sie auch, dass die Bourne-Methode zum Definieren einer Funktion darin besteht, function-name()vor einem Befehl zu bleiben . Das ist der Fall in jedem Bourne wie Shell außer bashund yash(und neueren Versionen von posh) , die nur einer Verbindung Befehl (Verbindung Befehle zu sein erlauben {...}oder (...)oder Dinge wie for...done, if...fi...

function foo { ... }ist die kshSyntax der Funktionsdefinition. Es gibt keinen Grund, warum Sie es hier verwenden möchten.

Ihr Code kann portabel (POSIXly) geschrieben werden:

foo() (
  cd -P -- "$FOOBAR" || return # what if the cd failed!
  if
    <some command>
  then
    echo 'OK!'
  else
    echo 'Nope!'
  fi
)

Beachten Sie auch, dass cdwithout -Peine ganz besondere Bedeutung hat (behandelt Pfade, die ..Komponenten enthalten, die sich von anderen Befehlen unterscheiden). Daher ist es besser, sie in Skripte aufzunehmen, um Verwirrungen zu vermeiden.

(Diese Funktion wird zurückgegeben, falsewenn sie cdfehlschlägt, aber nicht, wenn sie <some command>fehlschlägt.)

Ich glaube, der folgende Befehl wird alles, was Sie wollen, in einer Zeile tun.

(( verify = $?!=0?'Nope!':'OK!' ))