Discussion :

[Lady]

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 :

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

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

[Loceka]

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

Code :

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

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 :

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

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.

[Lady]

La version ls me parle bien et me convient.

Merci

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

Code :

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

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

[zipe31]

Salut,

Code :

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

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

[al1_24]

Code :

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

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.

[jack-ft]

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

Code :

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

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 :

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

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.

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

[Lady]

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.

[N_BaH]

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 :

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

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 :

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

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