Amélioration d'un script (parameter expansion)

Bonjour,

J'aimerai améliorer mon script, c'est à dire fusionner la ligne 7 et la ligne 8 en une et une seule ligne.

Le script fait la chose suivante :
On va supposer que dans un fichier il y est une série de ligne avec le format suivant :
Path: premier_serveur!deuxieme_serveur!troisième_serveur!...!n_serveur!not-for-mail

Le but est d'avoir ceci :
premier_serveur
deuxieme_serveur
troisieme_serveur
...
n_serveur
not-for-mail

J'ai créé un fichier texte que j'ai appelé message.txt

Code:

---

1 2	cat message.txt Path: premier_serveur!deuxieme_serveur!troisieme_serveur!not-for-mail

---

Le script bash que j'ai fait fonctionne.
Cependant les améliorations que j'ai tenté d'apporter afin de fusionner la ligne 7 et la ligne 8 en une et une seule ligne se sont avérées infructueuses.
En effet cela boucle à l'infini et je ne comprends pas pourquoi.
Es ce que vous pouvez m'expliquer pourquoi ?

Ci dessous le script qui fonctionne + les 2 tentatives d'améliorations (échecs) avec les commentaires :

Code:

---

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

#!/bin/bash #Script qui fonctionne #============================================ ligne=$(grep "Path: ") liste_serveurs=${ligne#Path: } while [ -n "$liste_serveurs" ] do serveur=${liste_serveurs%%\!*} # premier élement (le point d'exclamation est supp) liste_serveurs=${liste_serveurs#$serveur} # on l'enlève de la liste liste_serveurs=${liste_serveurs#\!} # on supprimer le point d'exclamation restant echo $serveur done #============================================   #=========================================================================================== # POSSIBILITE D'AMÉLIORATION : # faire de la ligne 7 et la ligne 8 une et une seule ligne #===========================================================================================   <<commentaire Première tentative : ================== ligne=$(grep "Path: ") liste_serveurs=${ligne#Path: } while [ -n "$liste_serveurs" ] do serveur=${liste_serveurs%%\!*} liste_serveurs=${liste_serveurs#${serveur}\!} echo $serveur done   Résultat : Boucle infinie et affiche not-for-mail (le dernier élément de la liste) ==================   Deuxième tentative : ================== ligne=$(grep "Path: ") liste_serveurs=${ligne#Path: } while [ -n "$liste_serveurs" ] do serveur=${liste_serveurs%%\!*} liste_serveurs=${liste_serveurs#$\{serveur\}\!} echo $serveur done   Résultat : Boucle infinie et affiche premier_serveur (le premier élément de la liste) ================== commentaire

---

Bonjour :coucou:

Code:

---

1 2 3 4 5 6 7 8 9

$ cat message.txt Path: premier_serveur!deuxieme_serveur!troisième_serveur!...!n_serveur!not-for-mail $ awk -F '!' '{sub(/^[^ ]* /,"",$1); for (i=1;i<=NF;i++) print $i;}' message.txt premier_serveur deuxieme_serveur troisième_serveur ... n_serveur not-for-mail

---

Bonjour

Citation:

---

Envoyé par zephyre

Ci dessous le script qui fonctionne

Code:

---

1 2 3 4 5

#!/bin/bash #Script qui fonctionne #============================================ ligne=$(grep "Path: ") ...

---

---

Ca fonctionne ça? Ce grep que tu fais sur stdin??? 8O

Citation:

---

Envoyé par Sve@r

Bonjour

Ca fonctionne ça? Ce grep que tu fais sur stdin??? 8O

---

Bah, il ne montre pas comment il lance son script...

Code:

---

script.bash <fichier

---

ça le fait :aie:

Code:

---

1 2 3 4 5 6

cat expension.txt premier_serveur!deuxieme_serveur!troisieme_serveur!l4!l5!l6!l7!l8!l9!l10!not-for-mail premier_serveur!deuxieme_serveur!troisieme_serveur!l4!l5!l6!l7!l8!l9!l10!not-for-mail premier_serveur!deuxieme_serveur!troisieme_serveur!l4!l5!l6!l7!l8!l9!l10!not-for-mail premier_serveur!deuxieme_serveur!troisieme_serveur!l4!l5!l6!l7!l8!l9!l10!not-for-mail

---

Code:

---

1 2	awk -F'!' '{ for ( i=1 ; i < NF ; i++ ) { if ( i == 7 ) { print $7 " " $8 ; i=i+1 } else print $i }}' expension.txt

---

Code:

---

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

premier_serveur deuxieme_serveur troisieme_serveur l4 l5 l6 l7 l8 l9 l10 premier_serveur deuxieme_serveur troisieme_serveur2 l4 l5 l6 l7 l8 l9 l10 premier_serveur deuxieme_serveur troisieme_serveur3 l4 l5 l6 l7 l8 l9 l10 premier_serveur deuxieme_serveur troisieme_serveur4 l4 l5 l6 l7 l8 l9 l10

---

Vous ne répondez pas à la question.
Je voudrai comprendre pourquoi je boucle à l'infini.

Tu ne boucles pas à l'infini. Sve@r t'a répondu. Le script attend une entrée sur l'entrée standard car tu fais un filtre (grep) sur aucune cible. Donc la cible par défaut est ce que tu vas entrer au clavier. Pour mettre fin à la saisie, il suffit de faire Ctrl+D.

Citation:

---

Envoyé par Flodelarab

Tu ne boucles pas à l'infini. Sve@r t'a répondu. Le script attend une entrée sur l'entrée standard car tu fais un filtre (grep) sur aucune cible. Donc la cible par défaut est ce que tu vas entrer au clavier. Pour mettre fin à la saisie, il suffit de faire Ctrl+D.

---

Je peux t'assurer que je boucle à l'infini quand j'exécute ce script ci dessous :

Code:

---

1 2 3 4 5 6 7 8 9 10 11

ligne=$(grep "Path: ") liste_serveurs=${ligne#Path: } while [ -n "$liste_serveurs" ] do serveur=${liste_serveurs%%\!*} liste_serveurs=${liste_serveurs#${serveur}\!} echo $serveur done   # Résultat : # Boucle infinie et affiche not-for-mail

---

Avec comme entrée/input, le fichier nommé message.txt:

Code:

---

1 2	cat message.txt Path: premier_serveur!deuxieme_serveur!troisieme_serveur!not-for-mail

---

Ou ce script :

Code:

---

1 2 3 4 5 6 7 8 9 10 11

ligne=$(grep "Path: ") liste_serveurs=${ligne#Path: } while [ -n "$liste_serveurs" ] do serveur=${liste_serveurs%%\!*} liste_serveurs=${liste_serveurs#$\{serveur\}\!} echo $serveur done   # Résultat : # Boucle infinie et affiche premier_serveur (le premier élément de la liste)

---

Avec la même entrée/input.

Je lance les deux scripts avec la commande ci dessous :

Code:

---

./chemin/vers/le/script < message.txt

---

Citation:

---

Envoyé par Flodelarab

Tu ne boucles pas à l'infini. Sve@r t'a répondu. Le script attend une entrée sur l'entrée standard car tu fais un filtre (grep) sur aucune cible. Donc la cible par défaut est ce que tu vas entrer au clavier. Pour mettre fin à la saisie, il suffit de faire Ctrl+D.

---

Non, j'ai répondu un peu vite et disedorgue l'a bien vu. Si on demande cat fichier |./script.sh ça marche.
Ce qu'il veut dire (et j'ai testé), c'est qu'avec ces deux lignes...

Code:

---

1 2	liste_serveurs=${liste_serveurs#$serveur} # on l'enlève de la liste liste_serveurs=${liste_serveurs#\!} # on supprimer le point d'exclamation restant

---

... ça fonctionne bien... mais si on tente de supprimer la variable intermédiaire comme par exemple liste_serveurs=${liste_serveurs#${serveur}\!} là ça boucle à l'infini sur le dernier champ "not-for-mail".

Citation:

---

Envoyé par zephyre

Je voudrai comprendre pourquoi je boucle à l'infini.

---

J'en sais rien. Ca doit être une histoire de priorité dans l'expansion des # mais pour le détail... J'ai cherché 2 mn en tentant de prioriser via liste_serveurs=${{liste_serveurs#${serveur}}\!} mais j'ai eu une erreur de syntaxe et j'ai laissé tomber. Après-tout, peut-être que cette histoire d'expansion ne permet pas de faire autrement.
J'ai eu un souci analogue sur tty. Je devais utiliser mon tty pour chercher une info dans un flux mais si j'écrivais flux |grep "$(tty)" j'obtenais que dalle parce que la commande "tty" après un pipe renvoie la chaine littérale "pas un tty"
Exemple

Code:

---

1 2	tty # Fonctionne true |tty # Ne fonctionne pas

---

... et donc j'ai été obligé de passer par une ligne intermédiaire pour créer une variable => tty="$(tty)"

zephyre: ton script est inutilement compliqué.

d'abord, ne passe pas le contenu du fichier dans le script (pourquoi fais-tu ça ?), passe le nom du fichier en argument, ou met-le en variable dans le script
ensuite, puisqu'il s'agit d'une liste avec des séparateurs connus, on fait un tableau

Code:

---

1 2 3 4 5 6 7 8

#!/bin/bash   fichier=message.txt #il faut indiquer le chemin absolu ! ligneListe=$(grep 'Path: ' "$fichier") IFS=' !' read -ra liste <<<"$ligneListe" #ou en une ligne : IFS=' !' read -ra liste < <(grep Path: ' "$fichier")   #je suppose que tu vas faire la même chose avec tous les éléments de la liste, non ? for ((i=1; i<${#liste[*]}-1; i++)); do : your stuff with ${liste[i]}; done

---

(j'admets, cependant, que je me suis trompé sur le fait qu'il attende. Il n'attend pas)

1. Il y aura toujours un suffixe n'ayant pas de point d'exclamation. Donc il tourne en boucle. Tu ne supprimes jamais le résidu.
2. Ce n'est pas tty, mais /dev/tty, n'est-ce pas ?

Peux pas tester dans l'immédiat mais est ce que ce ne serait pas une histoire de protection de variable avec des guillemets
(Car là, on n'a pas l'air de savoir pourquoi on n'en met pas :aie:)

Citation:

---

Envoyé par Flodelarab

Ce n'est pas tty, mais /dev/tty, n'est-ce pas ?

---

Non, je parlais bien de la commande "tty" qui permet d'afficher le terminal courant (qui lui, est effectivement associé à un fichier spécial caractère situé dans "/dev").
Le but de mon script était de déterminer si la connexion courante est locale ou par ssh. Dans ce dernier cas, la commande "who" affiche l'IP de provenance.
Donc l'idée était d'extraire de "who" la ligne correspondant à mon tty pour ensuite regarder si elle contient ou pas une IP. D'où le ... |grep "$(tty)".

Pour l'histoire du tty à droite du pipe, je ne suis pas surpris, tu es dans un sous shell qui n'a pas besoin directement d'un terminal

Oui quand on examine le truc après coup effectivement c'est logique. Mais quand on l'écrit on n'y pense pas forcément. On se dit "il faut que je regarde si l'info de la commandeX est présente dans le flux" on tape directement flux |grep "$(commandeX)" par habitude sans penser plus loin :aie:

Bon, maintenant que Flodelarab a donner la raison de la boucle infinie, voici une solution plus simple en gardant un peu les concepts du demandeur:

Code:

---

1 2 3 4 5 6

$ cat scr.bash #!/bin/bash #Script qui fonctionne #============================================ ligne=$(grep -Po "Path: \K.*") echo "${ligne//!/$'\n'}"

---

Code:

---

1 2 3 4

$ cat fic.txt Path: premier_serveur!deuxieme_serveur!troisieme_serveur!not-for-mail bob Path: premier_serveur!deuxieme_serveur!troisieme_serveur!not-for-mail2

---

Code:

---

1 2 3 4 5 6 7 8 9

$ ./scr.bash < fic.txt premier_serveur deuxieme_serveur troisieme_serveur not-for-mail premier_serveur deuxieme_serveur troisieme_serveur not-for-mail2

---