Discussion :

[jean-marc69]

Bonjour le Forum,

Débutant en bash, je souhaiterai optimiser mon premier script.

Lecture d'un fichier avec split sur ";".

Version 1.0

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
 
while read line
do
   N_ENV=$(echo "$line" | cut -d';' -f1)
   N_APP=$(echo "$line" | cut -d';' -f2)
   N_JOB=$(echo "$line" | cut -d';' -f3)
   V_COM=$(echo "$line" | cut -d';' -f4)
 
   # Application du changement
#  ${PATHVTOM}/vtaddjob /Nom="${N_ENV}/${N_APP}/${N_JOB}" /Comm="${V_COM}"
   if [ $? -eq 0 ] ; then
      echo -e "[${VERT}OK${NORMAL}] `date +"%D %H:%M"` # $i \t ENV=${N_ENV} APP=${N_APP} JOB=${N_JOB} \t COM=${V_COM}"
   else
      echo -e " [${ROUGE}KO${NORMAL}]"
      exit 2
   fi
 
done < ${FILE_INPUT}

Ce script est fonctionnel, mais ayant lu que l'utilisation d'une boucle "while" était déconseillée, j'essaie d'extraire mes champs de cette façon:

Code :

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

1
2
3
4
5
 
for item in $(cut -d";" -f1,4 $FILE_INPUT)
do
  echo $item
done

La variable item me renvoie la ligne complète.

Question: comment lire f1, f2, f3 et f4 ?

Merci par avance de vos suggestions.

[N_BaH]

Bonjour,

l'utilisation d'une boucle "while" était déconseillée,

où ça ?
qui ça ?
dans quel contexte ?

une boucle while est tout à fait indiquée ici.

voir Comment parser un fichier.

par contre, il est fortement déconseillé d'utiliser une boucle for pour parcourir la sortie d'une substitution de commandes !

[Flodelarab]

Lecture d'un fichier avec split sur ";".

Mon premier reflexe est de répondre:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
$ cat initial.txt 
toto;tata;tsoin;
et;pareil;ici
presque
$ grep -o '[^;]*' initial.txt 
toto
tata
tsoin
et
pareil
ici
presque
$

Sinon, si tu as un fichier bien organisé (genre fichier csv) tu as intérêt à utiliser awk.
Mais on comprend pas trop le but pour l'instant. La balle est dans ton camp

[jean-marc69]

Bonjour N_BaH et Flodelarab,

Merci de vos réponses.

Dans le tutoriel "http://eric-sanchis.developpez.com/linux/shell/bash/", celui-ci précise

La lecture ligne par ligne d’un fichier à l’aide d’une commande interne
while est lente et peu élégante. Il est préférable d’utiliser une suite de filtres permettant d’aboutir au résultat voulu.
Par exemple, en utilisant un filtre supplémentaire (la commande unix cut), on peut s’affranchir de l’itération while dans le programme

Je souhaite simplifier les 4 exécutions de la fonction cut.

Code :

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

1
2
3
4
5
6
7
8
 
   N_ENV=$(echo "$line" | cut -d';' -f1)
   N_APP=$(echo "$line" | cut -d';' -f2)
   N_JOB=$(echo "$line" | cut -d';' -f3)
   V_COM=$(echo "$line" | cut -d';' -f4)
 
   # Application du changement
#  ${PATHVTOM}/vtaddjob /Nom="${N_ENV}/${N_APP}/${N_JOB}" /Comm="${V_COM}"

Merci pour le lien "Comment parser un fichier".

[Sve@r]

Bonjour N_BaH et Flodelarab,

Merci de vos réponses.

Dans le tutoriel "http://eric-sanchis.developpez.com/linux/shell/bash/", celui-ci précise

La lecture ligne par ligne d’un fichier à l’aide d’une commande interne
while est lente et peu élégante. Il est préférable d’utiliser une suite de filtres permettant d’aboutir au résultat voulu.
Par exemple, en utilisant un filtre supplémentaire (la commande unix cut), on peut s’affranchir de l’itération while dans le programme

Salut
C'est vrai que si on peut obtenir directement le résultat voulu par le biais d'une commande spécifique sans avoir à traiter le fichier ligne par ligne, c'est mieux. Bien évidemment quelque part dans la chaine il y a fatalement un traitement ligne à ligne mais si c'est la commande (compilée) qui le fait, ce sera fait bien plus rapidement qu'en shell.

Mais si on peut pas utiliser cet avantage, alors le cat fichier |while read ligne redevient nécessaire et reste préférable à un for ligne in $(cat fichier)...

Je souhaite simplifier les 4 exécutions de la fonction cut.

Code :

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

1
2
3
4
5
6
7
   N_ENV=$(echo "$line" | cut -d';' -f1)
   N_APP=$(echo "$line" | cut -d';' -f2)
   N_JOB=$(echo "$line" | cut -d';' -f3)
   V_COM=$(echo "$line" | cut -d';' -f4)
 
   # Application du changement
#  ${PATHVTOM}/vtaddjob /Nom="${N_ENV}/${N_APP}/${N_JOB}" /Comm="${V_COM}"

Pour moi cette solution reste la meilleure (le cut est un des programmes de traitement de flux les plus rapides qui soient). Et le quadruple appel reste obligé puisque tu veux récupérer 4 éléments différents.
Mais si vraiment tu veux éviter de faire appel à un programme externe, tu peux y arriver en ne passant que par des outils internes shell de cette façon:

Code :

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

1
2
3
4
5
old="$IFS"; IFS=";"; set $line; IFS="$old"; unset old
# Je mets tout sur une seule ligne pour qu'on comprenne bien que ces 4 commandes sont liées
 
# Application du changement
$PATHVTOM/vtaddjob /Nom="$1/$2/$3" /Comm="$4"

Moi personnellement j'aime pas (en plus de toucher à l'IFS ce qui est méga dangereux, on perd les arguments précédents) mais bon, je ne suis pas un intégriste (comme certains modos avec le ls... suivez mon regard )

PS: évite les accolades systématiques aux noms de variables. Ca n'a que peu d'utilité et ça alourdit l'écriture (et la lecture)...

[jean-marc69]

Merci Sve@r de ces précieux conseils.

[disedorgue]

Bonjour,
Un peut moins portable, mais si c'est du bash (pas trop vieux tout de même )

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
$ echo $N_APP
 
$ echo $N_ENV
 
$ echo $N_JOB
 
$ echo $V_COM
 
$ read N_ENV N_APP N_JOB V_COM <<<$(echo "titi;toto;tata;tutu" | cut  --output-delimiter=' ' -d';' -f1-4)
$ echo $N_APP
toto
$ echo $N_ENV
titi
$ echo $N_JOB
tata
$ echo $V_COM
tutu

[N_BaH]

Code :

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

read N_ENV N_APP N_JOB V_COM < <( cut  --output-delimiter=' ' -d';' -f1-4 <<<"titi;toto;tata;tutu")

voire avec un tableau si le nom des variables n'est pas indispensable.

mais bon, tant qu'on ne verra pas de quoi a l'air le fichier...

[disedorgue]

Génial

En me foulant un peu, j'aurai pu effectivement me passer du echo, par contre je ne connaissais pas du tout cette syntaxe de "< <(....)".
Il y a de la documentation la dessus ?

[N_BaH]

oui, Substitution de Processus.