Wie schreibe ich ein Skript, das mit oder ohne Argumente arbeitet?

Ich habe ein Bash-Skript, das ungefähr so ​​aussieht:

#!/bin/bash

if [ $1 = "--test" ] || [ $1 = "-t" ]; then
    echo "Testing..."
    testing="y"

else
    testing="n"
    echo "Not testing."

fi

Ich möchte es also nicht nur mit ./script --testoder ausführen können ./script -t, sondern auch ohne Argumente (nur ./script). Wenn ich das mit dem aktuellen Code mache, ist die Ausgabe anscheinend einfach:

./script: line 3: [: =: unary operator expected
./script: line 3: [: =: unary operator expected
Not testing.

Wie programmiere ich es also so, dass es ohne Argumente funktioniert, elseohne den Fehler auszulösen? Was mache ich falsch?

Die "richtige Art", lange Optionen in Shell- Skripten zu verwenden, ist das GNU-Dienstprogramm getopt . Es gibt auch getopts, die eine eingebaute Bash-Funktion sind , die jedoch nur kurze Optionen wie zulässt -t. Einige getoptAnwendungsbeispiele finden Sie hier .

Hier ist ein Skript, das zeigt, wie ich mit Ihrer Frage umgehen würde. Erläuterungen zu den meisten Schritten werden als Kommentare im Skript selbst hinzugefügt.

#!/bin/bash

# GNU getopt allows long options. Letters in -o correspond to
# comma-separated list given in --long.

opts=$(getopt -o t --long test -- "$*")
test $? -ne 0 && exit 2 # error happened

set -- $opts # some would prefer using eval set -- "$opts"
# if theres -- as first argument, the script is called without
# option flags
if [ "$1" = "--"  ]; then
    echo "Not testing"
    testing="n"
    # Here we exit, and avoid ever getting to argument parsing loop
    # A more practical case would be to call a function here
    # that performs for no options case
    exit 1
fi

# Although this question asks only for one 
# option flag, the proper use of getopt is with while loop
# That's why it's done so - to show proper form.
while true; do
    case "$1" in
        # spaces are important in the case definition
        -t | --test ) testing="y"; echo "Testing" ;;
    esac
    # Loop will terminate if there's no more  
    # positional parameters to shift
    shift  || break
done

echo "Out of loop"

Mit ein paar Vereinfachungen und entfernten Kommentaren kann dies zu Folgendem zusammengefasst werden:

#!/bin/bash
opts=$(getopt -o t --long test -- "$*")
test $? -ne 0 && exit 2 # error happened
set -- $opts    
case "$1" in
    -t | --test ) testing="y"; echo "Testing";;
    --) testing="n"; echo "Not testing";;
esac

Verschiedene Wege; Die zwei offensichtlichsten sind:

• Setzen Sie $1doppelte Anführungszeichen:if [ "$1" = "--test" ]
• Überprüfen Sie die Anzahl der Argumente mit $ #

Besser noch, benutze getopts.

Sie müssen Ihre Variablen in der if-Bedingung angeben. Ersetzen:

if [ $1 = "--test" ] || [ $1 = "-t" ]; then

mit:

if [ "$1" = '--test' ] || [ "$1" = '-t' ]; then  

Dann klappt es:

➜ ~ ./test.sh Testet
nicht.

Setzen Sie Ihre Variablen immer in doppelte Anführungszeichen!

Sie verwenden Bash. Bash hat eine gute Alternative zu [: [[. Mit [[müssen Sie sich keine Gedanken über das Zitieren machen, und Sie erhalten viel mehr Operatoren als [, einschließlich der richtigen Unterstützung für ||:

if [[ $1 = "--test" || $1 = "-t" ]]
then
    echo "Testing..."
    testing="y"
else
    testing="n"
    echo "Not testing."
fi

Oder Sie können reguläre Ausdrücke verwenden:

if [[ $1 =~ ^(--test|-t) ]]
then
    echo "Testing..."
    testing="y"
else
    testing="n"
    echo "Not testing."
fi

Natürlich sollte immer korrekt zitiert werden, aber es gibt keinen Grund, [bei der Verwendung von Bash zu bleiben .

So testen Sie, ob Argumente vorhanden sind (im Allgemeinen und führen Sie die entsprechenden Aktionen aus):

#!/bin/bash

check=$1

if [ -z "$check" ]; then
  echo "no arguments"
else echo "there was an argument"
fi