Boucle for file in <quelquechose de vide>

Bonjour,

J'ai un script shell qui doit faire quelque chose pour chaque fichier xls contenu dans un dossier.

J'ai donc le code :

Code:

---

1 2 3 4 5

for file in $DATADIRECTORY/*.xls do echo $file; <fait quelque chose> done;

---

Sauf que si il n'y a pas de fichier xls dans mon dossier il passe quand même dans le do avec la valeur <DATADIRECTORY>/*.xls.
Autant dire que ce que j'appel dans le do n'apprécie pas trop.

Quel est la meilleure solution pour éviter ce petit problème ?
(note : je suis quasi débutante en shell)

Merci d'avance !!

Pour répondre exactement à ta question, le mieux est de passer par la commande ls :

Code:

---

1 2 3 4 5

for file in $(ls -1 $DATADIRECTORY/*.xls) do echo $file; <fait quelque chose> done

---

Par contre, si le traitement à faire n'est pas trop complexe, tu peux aussi faire ça directement dans une commande find :

Code:

---

find $DATADIRECTORY -maxdepth 1 -type f -iname "*.xls" -exec <ta_commande> {} \;

---

les accolades ({}) sont un token spécifique à la commande find et symbolisant le nom du fichier trouvé.

Du coup, pour chaque fichier trouvé, tu fais le traitement.

La version ls me parle bien et me convient.

Merci :ccool:

Edit : bah en fait non ça ne marche pas si fichier : ok sinon erreur

Code:

---

ls: impossible d'accéder Ã* /opt/ImportOC/data/*.xls: Aucun fichier ou dossier de ce type

---

Salut,

Code:

---

1 2 3 4 5 6 7

for file in $DATADIRECTORY/*.xls do if [ -s "${file}" ]; then echo $file; <fait quelque chose> fi done

---

Code:

---

1 2 3 4 5

find ${DATADIRECTORY} -maxdepth 1 -type f -name '*.xls' -print | while read do filename="${REPLY}" <fait quelque chose> done

---

Comme ça on ne rentre dans la boucle que s'il y a bien au moins un fichier.

Citation:

---

Envoyé par Lady

La version ls me parle bien et me convient.

---

Attention! ça peut poser des problèmes s'il y a des fichiers avec des noms "bizarres" (avec des espaces, returns ou tabulations dans leur nom (comme c'est le cas sous certains os (Program\ Files, Documents\ and \ Settings, etc.))).

Citation:

---

Code:

---

ls: impossible d'accéder Ã* /opt/ImportOC/data/*.xls: Aucun fichier ou dossier de ce type

---

---

C'est pas grave, c'est juste un message "normal" de ls sur la stderr que tu peux ignorer superbement:

Code:

---

1 2 3 4

for file in $(ls ${DATADIRECTORY}/*.xls 2> /dev/null) ; do echo "${file}" # fait quelque chose avec ${file} done

---

àmha, l'option '-1' de 'ls' est inutile car c'est le défaut lorsque la sortie n'est pas le tty.

Je veux bien comprendre pourquoi j'ai été moinsé là (et Lady aussi par la même occasion).

J'ai dit une grosse connerie ou quelqu'un avait juste besoin de passer ses nerfs ?

Citation:

---

Envoyé par Loceka

Je veux bien comprendre pourquoi j'ai été moinsé là (et Lady aussi par la même occasion).

J'ai dit une grosse connerie ou quelqu'un avait juste besoin de passer ses nerfs ?

---

Quelqu'un qui ne support pas qu'on soit débutant ou qu'on se trompe je suppose ...

Enfin bref merci à tous j’hésite entre la version avec while qui n'entre pas dans le do ce qui me semble plus logique et celle avec test sur l’existence du fichier


Edit : c'est finalement la version de zipe31 qui va me servir !! Merci à tous.

je le dit encore une fois : ON N'UTILISE PAS ls DANS UN SCRIPT !!!
et je moinsse systèmatiqement ceux qui suggèrent une telle solution.

en bash, il existe une option nullglob:

Code:

---

1 2	shopt -s nullglob for f in fichier\ qui\ n\'existe\ pas; do echo "$f";done

---

en shell strictement POSIX, le mieux est sans doute d'utiliser find.
ou alors de faire un test :

Code:

---

for f in fichier\ qui\ n\'existe\ pas; do test -f "$f" && echo "actions sur \"$f\"" || echo ":( \"$f\" n'existe pas"'!'; done

---