Discussion :

[kondor76]

Bonjour,

Je cherche à faire le script suivant: "recup_scheduled_reports.sh"

je souhaite qu'on puisse l'appeler avec les options -d, -w ou -m.
on peut mettre autant d'options que l'on veut, mais le script ne peu être appelé sans option.

voici le code que j'ai fais par rapport à cette problématique:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
 
while getopts ":d:w:m" freq; do
        case $freq in
                d)
                        WORKING_REMOTE_DIR_DAY=1
                        ;;
                w)
                        WORKING_REMOTE_DIR_WEEK=1
                        ;;
                m)
                        WORKING_REMOTE_DIR_MONTH=1
                        ;;
                ?)
                        Usage
                        ;;
        esac
shift $(( OPTING -1 ))
done
if [ $WORKING_REMOTE_DIR_DAY = "1" ]; then
        echo "WORKING_REMOTE_DIR_DAY=1"
fi
if [ $WORKING_REMOTE_DIR_WEEK = "1" ]; then
        echo "WORKING_REMOTE_DIR_WEEK=1"
fi
if [ $WORKING_REMOTE_DIR_MONTH = "1" ]; then
        echo "WORKING_REMOTE_DIR_MONTH=1"
fi

voici le résultat de l'exécution (script en mode debug):

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
if [ $WORKING_REMOTE_DIR_DAY = "1" ]; then
        echo "WORKING_REMOTE_DIR_DAY=1"
fi
+ '[' = 1 ']'
./recup_scheduled_reports.sh: line 66: [: =: unary operator expected
if [ $WORKING_REMOTE_DIR_WEEK = "1" ]; then
        echo "WORKING_REMOTE_DIR_WEEK=1"
fi
+ '[' = 1 ']'
./recup_scheduled_reports.sh: line 69: [: =: unary operator expected
if [ $WORKING_REMOTE_DIR_MONTH = "1" ]; then
        echo "WORKING_REMOTE_DIR_MONTH=1"
fi
+ '[' = 1 ']'
./recup_scheduled_reports.sh: line 72: [: =: unary operator expected

Pourriez-vous m'aider?

merci

[disedorgue]

Bonjour,

Dans ton contexte, il faut que tu initialise tes variables avant, dans le cas ou si on ne passe pas l'option qui permet de la mettre à 1, que celle-ci existe.

Pourquoi as-tu mis des ":" dans tes options si tu ne les exploites pas ?

Conseil: protège le "?" de ?) par un "\" donc : \?)

[jack-ft]

Sauf exception valable, il vaut mieux entourer les variables de guillemets.
Essaie ceci:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
if [ "$WORKING_REMOTE_DIR_DAY" = "1" ]; then
        echo "WORKING_REMOTE_DIR_DAY=1"
fi
if [ "$WORKING_REMOTE_DIR_WEEK" = "1" ]; then
        echo "WORKING_REMOTE_DIR_WEEK=1"
fi
if [ "$WORKING_REMOTE_DIR_MONTH" = "1" ]; then
        echo "WORKING_REMOTE_DIR_MONTH=1"
fi

et la lumière jaillira...

[N_BaH]

Bonjour,

pourquoi vérifier si les variables valent 1 ?
tu pourrais tester, avant le parcours des options, que le premier argument du script est effectivement une option,
c'est-à-dire $1 commence par -, qui est suivi de d, ou m, ou w.

[kondor76]

Bonjour,

pourquoi vérifier si les variables valent 1 ?
tu pourrais tester, avant le parcours des options, que le premier argument du script est effectivement une option,
c'est-à-dire $1 commence par -, qui est suivi de d, ou m, ou w.

Ca c'était juste un test comme ca.

Par contre quand je lance mon script sans argument, il ne passe pas dans Usage et je ne vois pas pourquoi?

Une idée?

[disedorgue]

Voici un exemple de getopt:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
$ cat testgetop.sh
#!/bin/bash
 
if ( ! getopts ":a:b:cd:h" opt); then
        echo "Usage: `basename $0` -h aide" >&2
        exit $E_OPTERROR
fi
 
while getopts ":a:b:cd:h" opt; do
 
case $opt in
  a)
    echo "vous avez choisi -a, Parametre: $OPTARG"
    ;;
  b)
    echo "vous avez choisi -b, Parametre: $OPTARG"
    ;;
  c)
    echo "vous avez choisi -c, Parametre: $OPTARG"
    ;;
  d)
    echo "vous avez choisi -d, Parametre: $OPTARG"
    ;;
  h)
    echo "vous avez choisi -h, Parametre: $OPTARG"
    ;;
  \?)
    echo "option invalide: -$OPTARG" >&2
    exit 1
    ;;
  :)
    echo "un argument est requis pour -$OPTARG" >&2
    exit 1
    ;;
  *)
    echo "Option obligatoire" >&2
    exit 1
    ;;
esac
done

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
$ ./testgetop.sh
Usage: testgetop.sh -h aide
$ ./testgetop.sh -a
un argument est requis pour -a
$ ./testgetop.sh -a titi
vous avez choisi -a, Parametre: titi
$ ./testgetop.sh -c
vous avez choisi -c, Parametre:
$ ./testgetop.sh -c titi
vous avez choisi -c, Parametre:
$ ./testgetop.sh -e
option invalide: -e
$ ./testgetop.sh -d
un argument est requis pour -d
$ ./testgetop.sh -d toto
vous avez choisi -d, Parametre: toto

La partie qui vérifie que l'on a bien des options de passée est la partie du if
Les options qui prennent un paramètre sont les lettres suivi d'un ':', donc ici a,b,d

[kondor76]

Voici un exemple de getopt:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
$ cat testgetop.sh
#!/bin/bash
 
if ( ! getopts ":a:b:cd:h" opt); then
        echo "Usage: `basename $0` -h aide" >&2
        exit $E_OPTERROR
fi
 
while getopts ":a:b:cd:h" opt; do
 
case $opt in
  a)
    echo "vous avez choisi -a, Parametre: $OPTARG"
    ;;
  b)
    echo "vous avez choisi -b, Parametre: $OPTARG"
    ;;
  c)
    echo "vous avez choisi -c, Parametre: $OPTARG"
    ;;
  d)
    echo "vous avez choisi -d, Parametre: $OPTARG"
    ;;
  h)
    echo "vous avez choisi -h, Parametre: $OPTARG"
    ;;
  \?)
    echo "option invalide: -$OPTARG" >&2
    exit 1
    ;;
  :)
    echo "un argument est requis pour -$OPTARG" >&2
    exit 1
    ;;
  *)
    echo "Option obligatoire" >&2
    exit 1
    ;;
esac
done

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
$ ./testgetop.sh
Usage: testgetop.sh -h aide
$ ./testgetop.sh -a
un argument est requis pour -a
$ ./testgetop.sh -a titi
vous avez choisi -a, Parametre: titi
$ ./testgetop.sh -c
vous avez choisi -c, Parametre:
$ ./testgetop.sh -c titi
vous avez choisi -c, Parametre:
$ ./testgetop.sh -e
option invalide: -e
$ ./testgetop.sh -d
un argument est requis pour -d
$ ./testgetop.sh -d toto
vous avez choisi -d, Parametre: toto

La partie qui vérifie que l'on a bien des options de passée est la partie du if
Les options qui prennent un paramètre sont les lettres suivi d'un ':', donc ici a,b,d

Merci à tous!!

[N_BaH]

Par contre quand je lance mon script sans argument, il ne passe pas dans Usage et je ne vois pas pourquoi?

getopts renvoie vrai si une option, indiquée ou non, est trouvée. Il renvoie faux si la fin des options est atteinte ou si une erreur se produit.

donc, sans options, la boucle while n'est pas exécutée.

[disedorgue]

Il faut espérer, puisque le if vérifie justement l'inverse pour sortir si pas d'option.

Mais, c'est pas cette partie que je voulais montrer le plus, c'est plutot la partie de l'option avec paramètre ou sans paramètre.

[N_BaH]

en fait, je répondais à kondor76
j ' é cris t r è s len te ment

[jack-ft]

Voici un exemple de getopt:

ou de getopts...

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
$ cat testgetop.sh
#!/bin/bash
 
if ( ! getopts ":a:b:cd:h" opt); then

Àmha, les parenthèses (et le sous-shell qu'elles génèrent) sont inutiles...

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

while getopts ":a:b:cd:h" opt; do

Les options qui prennent un paramètre sont les lettres suivi(es) d'un ':', donc ici a,b,d

Du coup, quel est le sens du ':' qui précède le 'a' dans ":a:b:cd:h" ?

[Flodelarab]

Du coup, quel est le sens du ':' qui précède le 'a' dans ":a:b:cd:h" ?

getopts: getopts optstring name [arg]
Parse option arguments.

Getopts is used by shell procedures to parse positional parameters
as options.

OPTSTRING contains the option letters to be recognized; if a letter
is followed by a colon, the option is expected to have an argument,
which should be separated from it by white space.

Each time it is invoked, getopts will place the next option in the
shell variable $name, initializing name if it does not exist, and
the index of the next argument to be processed into the shell
variable OPTIND. OPTIND is initialized to 1 each time the shell or
a shell script is invoked. When an option requires an argument,
getopts places that argument into the shell variable OPTARG.

getopts reports errors in one of two ways. If the first character
of OPTSTRING is a colon, getopts uses silent error reporting. In
this mode, no error messages are printed. If an invalid option is
seen, getopts places the option character found into OPTARG. If a
required argument is not found, getopts places a ':' into NAME and
sets OPTARG to the option character found. If getopts is not in
silent mode, and an invalid option is seen, getopts places '?' into
NAME and unsets OPTARG. If a required argument is not found, a '?'
is placed in NAME, OPTARG is unset, and a diagnostic message is
printed.

If the shell variable OPTERR has the value 0, getopts disables the
printing of error messages, even if the first character of
OPTSTRING is not a colon. OPTERR has the value 1 by default.

Getopts normally parses the positional parameters ($0 - $9), but if
more arguments are given, they are parsed instead.

Exit Status:
Returns success if an option is found; fails if the end of options is
encountered or an error occurs.

[disedorgue]

ou de getopts...

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
$ cat testgetop.sh
#!/bin/bash
 
if ( ! getopts ":a:b:cd:h" opt); then

Àmha, les parenthèses (et le sous-shell qu'elles génèrent) sont inutiles...



Du coup, quel est le sens du ':' qui précède le 'a' dans ":a:b:cd:h" ?

Ok, je copierais 100 fois (via une boucle, faut pas pousser tout de même ):
Eviter autant que possible les sous shell inutiles.

Le premier ':' me sert ici à dire à getopts de ne pas généré une erreur si mauvaise option ou pas d'argument pour une option sur laquelle c'est obligatoire.
Après, on peut ne pas le mettre et dans ce cas le laisser gérer tout seul, donc plus besoin des 3 derniers cas du case.