Discussion :

[abou33]

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]

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]

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]

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]

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]

bonjour,

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

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]

Merci à tous.

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

Abou33.

[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 !

[abou33]

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

Merci
Abou33

[Flodelarab]

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

[N_BaH]

non. pourquoi ? à quoi tu penses ?

[jlliagre]

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]

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

[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 ... ;-)

[N_BaH]

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

[Flodelarab]

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]

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]

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