Utilisation du getopts dans une fonction (Bash/ksh)

Version imprimable

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:

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

Bonjour :coucou:

Pourquoi tu n'utilises pas date ?

Code:

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
$

19/05/2015, 11h15
abou33

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.
19/05/2015, 11h57
Flodelarab

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

Code:

OPTIND=1
19/05/2015, 11h59
jlliagre

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

Code:

function calculdate {

plutôt que

Code:

calculdate() {

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

Code:

local OPTIND
19/05/2015, 12h11
N_BaH

bonjour,

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

Citation:

Envoyé par jlliagre

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

Code:

function calculdate {

plutôt que

Code:

calculdate() {

euh, mais pourquoi ?
j'aurais pensé que tu aurais penché pour la version POSIX.
19/05/2015, 12h43
abou33

Merci à tous.

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

Abou33.
19/05/2015, 12h50
N_BaH

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 !
19/05/2015, 13h59
abou33

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

Merci
Abou33
19/05/2015, 14h24
Flodelarab

Est-ce que la variable OPTIND peut porter un autre nom ?
19/05/2015, 14h33
N_BaH

non. pourquoi ? à quoi tu penses ?
19/05/2015, 14h35
jlliagre

Citation:

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 ... ;-)
19/05/2015, 14h51
N_BaH

ma question portait sur function funcname plutôt que funcname().
19/05/2015, 15h44
jlliagre

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

ça ne répond pas à mon interrogation : pourquoi l'un plus que l'autre ?
19/05/2015, 16h06
Flodelarab

Citation:

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".
19/05/2015, 16h17
jlliagre

Citation:

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.

peut-être pour ça :

Code:

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:

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

Code:

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

Code:

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