Come scrivere uno script che funziona con o senza argomenti?

Ho uno script bash che va in questo modo:

#!/bin/bash

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

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

fi

Quindi quello che voglio poter fare non è solo eseguirlo con ./script --testo ./script -t, ma anche senza argomenti (solo ./script), ma a quanto pare se lo faccio con il codice corrente l'output è solo:

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

Quindi, come posso programmarlo in modo che eseguirlo senza argomenti farà semplicemente ciò elsesenza generare l'errore? Che cosa sto facendo di sbagliato?

Il "modo corretto" di usare opzioni lunghe negli script di shell è tramite l' utilità getopt GNU . Ci sono anche getopts che sono integrati in bash , ma consentono solo opzioni brevi come -t. Alcuni esempi di getoptutilizzo sono disponibili qui .

Ecco una sceneggiatura che dimostra come affrontare la tua domanda. La spiegazione della maggior parte dei passaggi viene aggiunta come commento all'interno dello script stesso.

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

Con alcune semplificazioni e commenti rimossi, questo può essere condensato in:

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

Svariati modi; i due più ovvi sono:

• Metti $1tra virgolette:if [ "$1" = "--test" ]
• Controlla il numero di argomenti usando $ #

Meglio ancora, usa getopts.

Devi citare le tue variabili all'interno della condizione if. Sostituire:

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

con:

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

Quindi funzionerà:

➜ ~ ./test.sh
Non testato.

Cita sempre sempre due volte le tue variabili!

Stai usando bash. Bash ha una grande alternativa a [: [[. Con [[, non devi preoccuparti di quotare e ottieni molti più operatori di [, incluso il supporto adeguato per ||:

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

Oppure puoi usare le espressioni regolari:

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

Ovviamente, si dovrebbe sempre fare una quotazione corretta, ma non c'è motivo di attenersi [quando si utilizza bash.

Per verificare se sono presenti argomenti (in generale e eseguire le azioni di conseguenza) Questo dovrebbe funzionare:

#!/bin/bash

check=$1

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