Discussion :

[zephyre]

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 :

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

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 :

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
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

[Flodelarab]

Bonjour

Code :

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

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

[Sve@r]

Bonjour

Ci dessous le script qui fonctionne

Code :

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

1
2
3
4
5
#!/bin/bash
#Script qui fonctionne
#============================================
ligne=$(grep "Path: ")
...

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

[disedorgue]

Bonjour

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

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

Code :

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

script.bash <fichier

ça le fait

[becket]

Code :

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

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 :

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

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 :

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
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

[zephyre]

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

[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.

[zephyre]

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 :

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

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 :

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

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

Ou ce script :

Code :

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

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 :

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

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

[Sve@r]

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 bash :

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

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".

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 bash :

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

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)"

[N_BaH]

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 :

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

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

[Flodelarab]

(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 ?

[disedorgue]

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 )

[Sve@r]

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)".

[disedorgue]

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

[Sve@r]

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

[disedorgue]

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 :

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

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

Code :

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

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 :

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

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