Discussion :

[bleuerouge]

Bonjour à tous,
J'ai développé ce script qui me permet de ranger les noms des repertoires contenu à l'interieur du repertoire courrant dans un tableau indexé. Hors celui-ci n'a pas l'air de fonctionner puisque l'affichage est toujours vide...

Pouvez vus m'aider svp

Merci d'avance

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
 
# liste tout les repertoire et les place dans un tableau
liste_rep ()
{
	let compteur=1 # declaration du compteur
	declare -a liste # declaration de la variable tableau 
	REP="ls"
	for FICHIER in REP # Parser le répertoire
	do
	if [ -d $FICHIER ] # Si l'occurence est un repertoire
	then
		$liste{compteur}=$FICHIER # Alors l'ajouter à la liste
		compteur ++ # Incrémenter le compteur
	fi
	done
 
	return $liste # Retourner le compteur
}
 
result= liste_rep
echo ${result[*]}

[_Mac_]

Essaie avec REP="$(ls)" au lieu de REP="ls".

[bleuerouge]

Essaie avec REP="$(ls)" au lieu de REP="ls".

Non ça ne marche pas non plus

[_Mac_]

Sinon, essaie avec REP=`ls`.

Ce que tu peux faire, c'est un echo $REP juste en dessous pour vérifier qu'il récupère bien le contenu du répertoire.

Tu es avec quel shell ? J'ai des doutes sur le "compteur ++" et "result= liste_rep" (l'espace après le = et j'aurais mis result=`liste_rep` ou result=$(liste_rep)).

[bleuerouge]

Je suis en bourne shell, j'ai modifié le script comme-ci dessous, seulement j'ai une erreur de type : numeric argument réquired. Peut on retourner un tableau avec une fonction.

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
liste_rep ()
{
	compteur=1 # declaration du compteur
	declare -a liste # declaration de la variable tableau 
 
	for FICHIER in ls # Parser le répertoire
	do
	if [ -d $FICHIER ] # Si l'occurence est un repertoire
	then
		liste[$compteur]=$FICHIER # Alors l'ajouter à la liste
		compteur=$(($compteur+1)) # Incrémenter le compteur
	fi
	done
 
	return liste # Retourner le compteur
}

[_Mac_]

Je sais pas : je ne connais pas bourn shell Faut pas faire un return $liste ?

[al1_24]

La valeur de retour d'une fonction shell est toujours une valeur numérique.
Si tu veux une fonction qui retourne une chaîne, affiche cette chaîne dans la fonction et récupère l'affichage dans le shell appelant.
Si tu veux une fonction qui retourne un tableau, tu peux soit afficher des lignes à prendre en compte comme les éléments du tableau, soit mettre à jour une variable tableau dans ta fonction...

[Arnaud F.]

Au passage, dans ta boucle FOR, il faut mettre des backquotes autour de la commande a exécuter!

Ce qui donne

Code :

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

for FICHIER in `ls`

Il les faut pour que le shell interprète ta commande

++

[_shuriken_]

Donc si je comprends bien, ton script positionne dans un tableau les noms des répertoires à partir du rép courant avec bien entendu 1 compteur qui s'incremente automatiquement à chaque rép trouver?!!! .
Je connais pas bien le Bourne Shell, mais je penses que les autres ont raison, révérifie ta syntaxe, ls est une commande et elle doit être utiliser comme ceci for FICHIER in `ls`.
Sinon la philosophie du script est correct.