Discussion :

[poullodiery]

Bonjour,

J'ai un répertoire "data" qui peut contenir n sous-dossiers. Le dernier sous-dossier contient un fichier "config.yml" que je souhaite parser.
Je ne vois pas ou se situe le problème.
Merci de votre aide.

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
 
 
#!/bin/sh
 
fonction()
{
	home=$(pwd)
	fichier="config.yml"
 
	for i in `ls $home/data/*`
	do
		if [ -e $fichier ]
		then
			while read line
			do
				case $line in
					*"jdkversion"*)
						param=`echo $line | awk -F":" '{ print $2}'`
						echo ${param}
					;;
				esac
			done < $fichier
		fi
	done
}
 
fonction

[N_BaH]

Bonjour,

- home=$(pwd) est inutile : il existe $HOME, ou $PWD.
- on ne parcourt pas la sortie d'une commande avec une boucle for, parce que for lit des mots, pas des lignes.
- on n'utilise plus `, et on évite de mélanger l'emploi de $() et `.
- on n'utilise pas ls dans un script; parce que c'est inutile (une boucle for sur le Développement des chemins (cf. man bash) fait mieux le travail en protégeant les caractères de l'IFS)
- il faut toujours mettre les variables entre guillemets. TOUJOURS; à moins de savoir pourquoi !!!

avant de répondre, il faut qu'on sache :
/bin/sh c'est parce que tu dois rester strictement POSIX, ou parce que tu ne sais pas que /bin/sh n'est pas la même chose que /bin/bash ?

[Flodelarab]

Bonjour

À noter : un site de vérification de syntaxe shell : https://www.shellcheck.net/

J'ai bien ri quand il a dit que la variable "i" de la boucle n'était utilisée nulle-part.
Relis ton code.

[balkany]

Ça sera toujours intéressant pour toi de corriger ton code bash, mais en gros, j'ai l'impression que tu peux le remplacer par

Code :

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

find ~/data -type f -name 'config.yml' -print0 | xargs -0 awk -F':' '/jdkversion/{print FILENAME ": " $2}'

[disedorgue]

Ou en bash, avec une boucle for:

Code :

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

1
2
3
4
5
shopt -s globstar
for i in "$HOME"/**/config.yml
do
  awk -F':' '/jdkversion/{print FILENAME ": " $2}' "$i"
done