Comment écrire un script qui fonctionne avec ou sans arguments?

J'ai un script bash qui ressemble à ceci:

#!/bin/bash

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

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

fi

Donc, ce que je veux pouvoir faire, c'est non seulement l'exécuter avec ./script --testou ./script -t, mais aussi sans argument (juste ./script), mais apparemment si je le fais avec le code actuel, la sortie est juste:

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

Alors, comment puis-je le programmer pour que le faire tourner sans aucun argument fasse juste le elsesans lancer l'erreur? Qu'est-ce que je fais mal?

La "bonne façon" d'utiliser les options longues dans les scripts shell est via l' utilitaire getopt GNU . Il y a aussi getopts qui est un bash intégré , mais il ne permet que des options courtes comme-t . Quelques exemples d' getoptutilisation sont disponibles ici .

Voici un script qui montre comment j'aborderais votre question. L'explication de la plupart des étapes est ajoutée en tant que commentaires dans le script lui-même.

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

Avec quelques simplifications et commentaires supprimés, cela peut être condensé en:

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

Plusieurs façons; les deux plus évidents sont:

• Mettez $1entre guillemets:if [ "$1" = "--test" ]
• Vérifiez le nombre d'arguments à l'aide de $ #

Mieux encore, utilisez getopts.

Vous devez citer vos variables à l'intérieur de la condition if. Remplacer:

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

avec:

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

Ensuite, cela fonctionnera:

➜ ~ ./test.sh
Pas de test.

Citez toujours toujours vos variables!

Vous utilisez bash. Bash a une excellente alternative à [: [[. Avec [[, vous n'avez pas à vous soucier de la cotation, et vous obtenez beaucoup plus d'opérateurs que [, y compris un support approprié pour ||:

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

Ou vous pouvez utiliser des expressions régulières:

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

Bien sûr, des citations appropriées doivent toujours être faites, mais il n'y a aucune raison de s'en tenir à l' [utilisation de bash.

Pour tester si des arguments sont présents (en général, et effectuez les actions en conséquence) Cela devrait fonctionner:

#!/bin/bash

check=$1

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