Utilisation d'une fonction

Version imprimable

Bonjour,

Je viens de mettre une fonction dans mon script par contre lors de l'exécution j'ai un message bizarre et je ne comprend pas pourquoi

Le script

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 
#!/bin/ksh
 
#set -x
 
control() {
if [ $? !=0 ]
     then
echo " il y a un icident dans le traitement "
     else
echo " Fin du traitement "
fi
}
 
 
noms_users=`cat /etc/passwd|awk -F":" '{print $1}'|grep "$1"` 1>/dev/null
 
#***********CONTROLE COMMANDE*************
control
[...]

Log d'erreur

Code:

1
2
3
4
5
6
7
/reptst : ./lsusers.ksh dmo
./lsusers.ksh: test: 0403-004 Spécifiez un paramètre avec cette commande.
 Fin du traitement
./lsusers.ksh: test: 0403-004 Spécifiez un paramètre avec cette commande.
 Fin du traitement
./lsusers.ksh: test: 0403-004 Spécifiez un paramètre avec cette commande.
 Fin du traitement

21/02/2013, 17h00
AAWOOPY56

Gloupsssss!!!!
J'ai trouvé mon erreur je n'avais pas mis de commande dans ma séquence de z'if et patatrac :bebe:
Ben les débuts sont laborieux mais je ne vais pas lacher l'affaire ça c'est sur :yaisse3:

:lahola:
03/03/2013, 12h57
ok.Idriss

Bonjour.

Quelques remarques de passage :

Code:

awk -F":" '{print $1}'

Pour un découpage aussi simple, il n'est pas forcemment utile de déployer l'artillerie lourde (awk). cut suffit amplement :

Code:

cut -d ";" -f1

Ensuite, dans les shells "modernes", les backquotes sont dépréciés en faveur de la syntaxe $(commande) (plus lisible etc) pour la sous-exécution.

Enfin pour le grep "$1", le $1 correspond à quoi ici ? S'il s'agit du login de l'utilisateur dont on voulait vérifier l’existence dans /etc/passwd, il y a beaucoup plus simple. Ne pas oublier que le grep s'applique ici qu'au résultat du découpage, autrement dit : au nom de l'utilisateur.

Cordialement,
Idriss

Citation:
Envoyé par AAWOOPY56
Le script
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 #!/bin/ksh #set -x control() { if [ $? !=0 ] then echo " il y a un icident dans le traitement " else echo " Fin du traitement " fi } noms_users=`cat /etc/passwd|awk -F":" '{print $1}'|grep "$1"` 1>/dev/null #***********CONTROLE COMMANDE************* control [...]
Bonjour. Autres petites remarques:
Plutôt que faire un grep et envoyer le résultat sur la stdout pour ne tester que le status de retour, il vaut mieux utiliser l'option grep -q qui sert exactement à ça! :ccool:

Ta fonction "control" affiche un message en fonction du code retour de la dernière instruction précédente. Comme ce comportement est extrêmement spécifique, plutôt qu'utiliser un nom très générique comme "control", je choisirais un nom à la hauteur de la spécificité de la fonction comme "checkLastReturnValue".
De plus, comme le code retour est très évanescent, la moindre insertion d'instructions risque de perturber son utilisation.
Je choisirais donc, soit de l'utiliser sur la même ligne:

Code:

noms_users=`cat /etc/passwd|cut -d":" -f1|grep -q "$1"`; checkLastReturnValue

soit de stocker le code retour dans une variable passée explicitement à la fonction:
Code:

1 2 3 noms_users=`cat /etc/passwd|cut -d":" -f1|grep -q "$1"`; RC=$? # blabla checkReturnValue $RC
Dernière remarque: si tu veux du code "blindé" tu peux aussi utiliser l'option "set -e"... mais c'est assez contraignant!