Discussion :

[plnew]

Bonjour

lorsque le grep trouve la valeur pas de problème par contre si il n'y a aucune correspondance le script ne fait rien ?

fichier listecopieurs.txt
COPI-101-1-Dir;Photocopieur;192.168.1.1;00:00:74:48:A8:A1
COPI-101-2;-Photocopieur;192.168.1.2;00:00:74:48:A8:AD
COPI-101-ACHATS;Photocopieur;192.168.1.3;00:00:74:48:A8:A2

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
#!/bin/bash
set -e
themac=$1
 
listeimp=$(grep -i "${themac}" listecopieurs.txt)
if [ -n "${listeimp}" ] ; then
        echo "liste pleine : $listeimp"
         resultatIMP=1
 else
         echo "liste vide"
         resultatIMP=0
 fi

je ne vois pas mon erreur ?

[zipe31]

Salut,

Commente ton set -e et ça devrait aller mieux

[plnew]

Merci pour ta réponse , effectivement cela fonctionne

j'aimerai avoir un complément d'information

la commande set -e = provoque une sortie immédiate si toute commande simple produit une erreur.

pourquoi ma commande provoque une erreur que lorsque le grep ne trouve rien ?

Code :

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

1
2
 
listeimp=$(grep -i "${themac}" listecopieurs.txt)

[zipe31]

la commande set -e = provoque une sortie immédiate si toute commande simple produit une erreur.

pourquoi ma commande provoque une erreur que lorsque le grep ne trouve rien

Parce que si tu testes le code retour de ta commande (echo $?), tu t'apercevras qu'il est égal à "1", autrement dit à "faux".

Code :

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

1
2
3
4
5
6
7
8
9
10
11
$ echo "Bonjour" | grep -q 'j'
 
$ echo $?
0
 
$ echo "Bonjour" | grep -q 'a'
 
$ echo $?
1
 
$

[plnew]

merci pour l'explication

y a t-il un moyen de contourner cette effet
car si grep répond 1 en code retour parce qu'il ne trouve rien , ce n'est pas pour moi une
erreur comme si la commande c'etait mal passée

question : a quelle moment utilise t-on le "set -e" ?

[zipe31]

la commande set -e = provoque une sortie immédiate si toute commande simple produit une erreur.

En fait non, la commande set -e ne provoque pas une sortie immédiate si toute commande simple produit une erreur, mais un code retour différent de zéro, ce qui n'est pas pareil.

car si grep répond 1 en code retour parce qu'il ne trouve rien , ce n'est pas pour moi une erreur comme si la commande c'etait mal passée

C'est vrai, mais comme dit au-dessus, c'est le fait que le code retour soit différent de zéro qui provoque l'arrêt du script

Code :

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

question : a quelle moment utilise t-on le "set -e" ?

Comme avec les options "x" et/ou "v" de la commande "set", à des fins de débogage, mais en tenant bien compte des deux remarques ci-dessus (code retour différent de zéro).

Après la question serait plutôt : Pourquoi avais-tu mis cette commande dans ton script ? Qu'en attendais-tu ?

[plnew]

comme je pensais que le set -e arrêtait le script pour une erreur cela me permettait de voir dès qu'il y avait un problème.

Ton explication est très claire, je ne peux plus mettre le set -e dans mes scripts si j'ai une commande qui peut retourner une valeur différente de 0.


encore merci pour le temps passé

[Sve@r]

Ton explication est très claire, je ne peux plus mettre le set -e dans mes scripts si j'ai une commande qui peut retourner une valeur différente de 0.

Surtout qu'il n'y en n'a pas qu'une !!!

• ls retourne 1 si on l'applique sur un nom de fichier présumé et qu'il n'existe pas
• expr retourne 1 si elle affiche 0
• false retourne systématiquement 1
• sans oublier test qui retourne 1 si l'évaluation demandée est fausse

...

[jack-ft]

comme je pensais que le set -e arrêtait le script pour une erreur cela me permettait de voir dès qu'il y avait un problème.

Ton explication est très claire, je ne peux plus mettre le set -e dans mes scripts si j'ai une commande qui peut retourner une valeur différente de 0.


encore merci pour le temps passé

Si, tu peux continuer à mettre set -e au début de tes scripts, à condition d'encadrer chaque appel qui peut retourner autre chose que 0 entre set +e et set -e !!!
et attention à bien capturer les $? au bon moment!
J'avoue que ce n'est pas très commode, mais ça peut servir!

[Sve@r]

Si, tu peux continuer à mettre set -x au début de tes scripts, à condition d'encadrer chaque appel qui peut retourner autre chose que 0 entre set +x et set -x !!!
et attention à bien capturer les $? au bon moment!

set -e !!!

[jack-ft]

comme je pensais que le set -e arrêtait le script pour une erreur cela me permettait de voir dès qu'il y avait un problème.

Ton explication est très claire, je ne peux plus mettre le set -e dans mes scripts si j'ai une commande qui peut retourner une valeur différente de 0.

Bonjour,

je viens de trouver quelque chose d'intéressant dans le man en rapport avec le set -e et qui semble le rendre utilisable!

Je ne sais pas pour quels shells c'est valable! Apparemment, au moins pour le bash (car mon man set m'affiche les BASH_BUILTINS(1)) et pour le pdksh (d'après mon man ksh).

Je n'ai rien trouvé dans la "bible" (Le KornShell, Bolsky & Korn). ça ne fonctionne donc peut-être pas pour des ksh autres que le pdksh (mais j'en doute (je n'ai pas la possibilité de le tester maintenant (mon mac est à la maison))):

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
$ echo $BASH_VERSION
3.2.25(1)-release
$ echo $KSH_VERSION
@(#)PD KSH v5.2.14 99/07/13.2
$ man set
[snip]
-e Exit immediately if a simple command (see SHELL GRAMMAR  above)
   exits  with  a non-zero status.  The shell does not exit if the
   command that fails is part of the command list immediately fol-
   lowing  a  while  or  until  keyword, part of the test in an if
   statement, part of a && or || list, or if the command’s  return
   value  is being inverted via !.  A trap on ERR, if set, is exe-
   cuted before the shell exits.

Du coup, ça rend le set -e tout à fait utilisable!
En effet, il suffit effectivement de tester le retour de chaque fonction!

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 #!/bin/bash
set -e
themac=$1
 
if listeimp=$(grep -i "${themac}" listecopieurs.txt) ; then
        echo "liste pleine : $listeimp"
        resultatIMP=1
else
        echo "liste vide"
        resultatIMP=0
fi

Par contre, on ne peut plus écrire le traditionnel:

Code :

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

1
2
ma_fonction; res=$?
if [[ $res -ne 0 ]]; then yaunblem ; fi

Il faut écrire (pour les shells qui connaissent le '!'):

Code :

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

if ! ma_fonction; then yaunblem ; fi

ou (pour les shells qui ne connaissent pas le '!'):

Code :

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

if ma_fonction; res=$?; [[ $res -ne 0 ]]; then yaunblem ; fi

Cette perspective m'ouvre de nouveaux horizons...
)jack(