Discussion :

[AAWOOPY56]

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 :

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
 
#!/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 :

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

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

[AAWOOPY56]

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

[ok.Idriss]

Bonjour.

Quelques remarques de passage :

Code :

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

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 :

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

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

[jack-ft]

Le 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
#!/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!

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 :

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

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 :

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

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!