Utilisation du getopts dans une fonction (Bash/ksh)

**abou33** · 19/05/2015, 10h29

Bonjour à tous,

Je viens de m’apercevoir que si j'utilise une deuxième fois la même fonction (avec des paramètres lus par getopts) dans un même script, la seconde exécution de la fonction ne lit pas les paramètres. Si je remplace le getopts par une lecture des paramètres avec $1, $2, ... ça fonctionne... Ouf, je ne suis pas fou...

Savez-vous pourquoi ou comment résoudre ce soucis ?

Je me monte une bibliothèque de fonction pour réécrire une p'tite appli et l'utilisation du getopts dans les fonctions m'aide énormément.

Voici mon script :

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
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
#!/bin/ksh
calculdate() {
# Calcule une date à plus ou moins n jours
# 
# Exemple :
#   Quelle est la date dans 15 jours (date début = date du jour) ?
#     calculdate -d 8 (aujourd'hui = 18/05/2015)
#        --> 26/05/2015
#   Quelle est la date de demain (date début = date du jour) ?
#     calculdate (aujourd'hui = 18/05/2015)
#        --> 19/05/2015
#   Quelle est la date il y a 7 jour (date début = date du jour) ?
#     calculdate -d 8 -s moins (aujourd'hui = 18/05/2015)
#        --> 10/05/2015
#   Quelle est la date 5 jours après le 30/06/2015 ?
#     calculdate -j 30 -m 05 -a 2015 -d 5
#        --> 04/06/2015
  echo debug $# $*  
  typeset -u sens
 
#  jour=$1
#  mois=$2
#  annee=$3
#  delai=$4
#  sens=$5
  while getopts :j:m:a:d:s: c
  do
    case $c in
         j)      jour=$OPTARG;;
         m)      mois=$OPTARG;;
         a)      annee=$OPTARG;;
         d)      delai=$OPTARG;;
         s)      sens=$OPTARG;;
         *)      echo "Parametre non connu !!!"
                 echo "calculdate [ -j <JJ> -m <MM> -a <CCAA> ] [ -d <nn> ] [ -s <PLUS|MOIS> ]"
                 return 99;;
    esac
  done
 
  # Initialisation des variables
  test -z $jour && jour=$(date '+%d')
  test -z $mois && mois=$(date '+%m')
  test -z $annee && annee=$(date '+%Y')
  test -z $delai && delai=1
  test -z $sens && sens="PLUS"
  test $sens == "MOINS" && avant="ago" || avant=""
 
  # formatage sur 2 chiffres du jour et du mois
  #jour=$(printf "%02d" $jour)
  #mois=$(printf "%02d" $mois)
  # Calcul de la date voulue
  newdate=$(date -d "$annee-$mois-$jour $delai day $avant" '+%d/%m/%Y')
  # Réponse de la fonction
  echo $newdate
  return 0
}
echo "Fonction calculdate :"
echo "--> Quelle est la date 5 jours après le 30/04/2015 ? (calculdate -j 30 -m 4 -a 2015 -d 5)"
calculdate -j 30 -m 04 -a 2015 -d 5
calculdate -j 20 -m 03 -a 2014 -d 4
exit 0

Abou33

**Flodelarab** · 19/05/2015, 11h11

Bonjour

Pourquoi tu n'utilises pas date ?

Code :

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

1
2
3
4
5
6
7
8
9
$ date
mar. 19 mai 2015 11:08:18
$ date -d "1 day ago"
lun. 18 mai 2015 11:09:08
$ date -d "3 day ago"
sam. 16 mai 2015 11:09:16
$ date -d "-3 day ago"
ven. 22 mai 2015 11:10:38
$

**abou33** · 19/05/2015, 11h15

J'suis d'accord mais ce n'est qu'un exemple de fonction que j'ai mis ici.

Le soucis est vraiment sur le passage de paramètres avec getopts.

Merci de ta réponse rapide.

**Flodelarab** · 19/05/2015, 11h57

Tu dois rajouter la ligne suivante dans ta fonction juste avant le while:

Code :

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

OPTIND=1

**jlliagre** · 19/05/2015, 11h59

Puisque tu utilises ksh, une solution simple consiste à définir ta fonction comme ça:

Code :

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

function calculdate {

plutôt que

Code :

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

calculdate() {

En bash, tu peux déclarer OPTIND comme variable locale à la fonction:

Code :

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

local OPTIND

**N_BaH** · 19/05/2015, 12h11

bonjour,

voir : Fonction avec getopts ne s'exécute qu'une seule fois

Envoyé par jlliagre

Puisque tu utilises ksh, une solution simple consiste à définir ta fonction comme ça:

Code :

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

function calculdate {

plutôt que

Code :

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

calculdate() {

euh, mais pourquoi ?
j'aurais pensé que tu aurais penché pour la version POSIX.

**abou33** · 19/05/2015, 12h43

Merci à tous.

Je vais prendre le "local OPTIND". Je trouve cette solution propre.

Abou33.

**N_BaH** · 19/05/2015, 12h50

sauf qu'un jour, tu n'utiliseras pas getopts dans une fonction, et alors tu devras faire comme j'ai appris, et comme je le préconise.
autant commencer tout de suite !

**abou33** · 19/05/2015, 13h59

Ne t'inquiètes pas. Je garde toutes les réponses pour en tirer le mieux plus tard.

Merci
Abou33

**Flodelarab** · 19/05/2015, 14h24

Est-ce que la variable OPTIND peut porter un autre nom ?

**N_BaH** · 19/05/2015, 14h33

non. pourquoi ? à quoi tu penses ?

**jlliagre** · 19/05/2015, 14h35

Envoyé par N_BaH

euh, mais pourquoi ?
j'aurais pensé que tu aurais penché pour la version POSIX.

Parce que:

1: "shift $((OPTIND-1))" ne suffit pas, il faut de toute façon réinitialiser OPTIND, c'est d'ailleurs bien la méthode documentée par POSIX
2: j'étais pressé d'aller déjeuner donc je n'ai pas argumenté ma réponse ... ;-)

**N_BaH** · 19/05/2015, 14h51

ma question portait sur function funcname plutôt que funcname().

**jlliagre** · 19/05/2015, 15h44

POSIX ne supporte pas les variables locales à une fonction et l'OP utilise ksh, et puis on n'est pas dans le forum POSIX ... ;-)

**N_BaH** · 19/05/2015, 16h06

ça ne répond pas à mon interrogation : pourquoi l'un plus que l'autre ?

**Flodelarab** · 19/05/2015, 16h06

Envoyé par N_BaH

non. pourquoi ? à quoi tu penses ?

A une banane.

Mais revenons au sujet:
Tu sais que tu as parcouru OPTIND options et tu veux revenir en arrière. Tu fais donc calculer à ton script le nombre de marches que tu voudrais qu'il franchisse puis tu fais un shift. Que d'efforts! Alors que tu sais que tu souhaites OPTIND=1 comme je l'ai proposé en message #4.

Je ne vois pas la plus-value d'utiliser la méthode "shift $((OPTIND-1))" ou "shift $((--OPTIND))" sur la méthode "OPTIND=1".

**jlliagre** · 19/05/2015, 16h17

Envoyé par Flodelarab

A une banane.

Mais revenons au sujet:
Tu sais que tu as parcouru OPTIND options et tu veux revenir en arrière. Tu fais donc calculer à ton script le nombre de marches que tu voudrais qu'il franchisse puis tu fais un shift. Que d'efforts! Alors que tu sais que tu souhaites OPTIND=1 comme je l'ai proposé en message #4.

Je ne vois pas la plus-value d'utiliser la méthode "shift $((OPTIND-1))" ou "shift $((--OPTIND))" sur la méthode "OPTIND=1".

Les deux commandes n'ont rien à voir, "shift $((OPTIND-1))" est nécessaire si l'on souhaite récupérer dans $1, $2, etc.. les arguments subsistants qui ne sont pas des options et leurs paramètres, alors que OPTIND=1 est nécessaire pour refaire un getopts avec de nouveaux arguments.

**N_BaH** · 19/05/2015, 16h26

peut-être pour ça :

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
$ cat ./getopts.example 
#!/bin/bash
echo $_
usage() {
   echo "usage : ${0##*/} [OPTION(S)]
-a     : affiche a
-b     : affiche b
-c arg : affiche <arg>" >&2
}
(($#)) || {
   echo "pas d'arguments" >&2
   usage
   exit 1
}
while getopts :abc:d opt
do
   case $opt in
      a) echo a;;
      b) echo b;;
      c) echo "argument : $OPTARG";;
      :) echo "grosse err #1 : argument requis absent";;
      \?) echo "grosse err #2 : opt = $opt ; OPTARG = $OPTARG ; OPTIND = $OPTIND";;
      *) echo "err ? : opt = $opt ; OPTARG = $OPTARG ; OPTIND = $OPTIND";;
   esac
done
shift $((--OPTIND))
#OPTIND=1
echo "reste : $@"
echo "fini"

Code :

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

1
2
3
4
5
6
$ ./getopts.example -a -c bla bla
./getopts.example
a
argument : bla
reste : bla
fini

Code :

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

1
2
#shift $((--OPTIND))
OPTIND=1

Code :

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

1
2
3
4
5
6
$ ./getopts.example -a -c bla bla
./getopts.example
a
argument : bla
reste : -a -c bla bla
fini

Utilisation du getopts dans une fonction (Bash/ksh)

Shell et commandes GNU

Discussions similaires

Partager

Partager