question sur la commande test

Bonjour,

J'aimerais avoir un éclaircissement sur la commande test avec le parametre e.

Mon code :

Code:

---

1 2 3 4 5 6 7

if /usr/bin/test -e "$DATA_EXTRACT/rating_by_postal_code/*.csv"; then echo " Move completed successfully" else echo " ERROR " fi

---

si je fais un ls de ce qu'il y a entre "", j'ai un résultat de plusieurs fichiers .csv.

Par contre lorque j'execute ce code, je passe toujours par le else (echo error) et ce malgré que j'ai des fichiers dans le répertoire mentionné.

Le but est seulement de vérifier l'existence d'au moins un fichier... peut-etre il y a une meilleure facon de procéder et votre aide là dessus est apprécié.

merci

bonjour,

les expansions ne sont pas réalisées entre guillemets.
dans ton cas, j'utiliserais un tableau, et vérifierais qu'il existe au moins un élément :

Code:

---

1 2 3

shopt -s nullglob array=( $DATA_EXTRACT/rating_by_postal_code/*.csv ) if test -n ${array[0]}; then echo "ok"; else echo "KO"; fi

---

ou bien, que le nombre d'éléments du tableau est supérieur à zéro :

Code:

---

1 2 3

shopt -s nullglob array=( $DATA_EXTRACT/rating_by_postal_code/*.csv ) if ((${#array[@]})); then echo "ok"; else echo "KO"; fi

---

NB: /usr/bin/test n'est pas une commande intégrée de l'interpréteur de commande; alors que test, elle, l'est.

Bonjour,

Autre façon de faire (je n'ai jamais utilisé la commande test):

Code:

---

1 2 3 4 5 6 7

if [ -e $DATA_EXTRACT/rating_by_postal_code/*.csv ] then echo "OK" else echo "KO" fi

---

Attention cependant aux espaces ! Cette façon de faire est très sensible à la syntaxe. Il faut un espace avant ET après chaque chose (le crochet, le '-e' etc).

Cordialement,
Elverion

bonjour Elverion,

ça ne marchera pas si :
- le nom de fichier contient un ou des espaces
- le répertoire contient plusieurs fichiers

ok je vais regarder les 2 options...

La solution du -e me semblait vraiment plus facile mais bon je vais les essayer et vous revenir.

Bonjour,

La piste de Elverion fonctionne... voici le code :

Code:

---

1 2 3 4 5 6 7

if [ -e $DATA_EXTRACT/rating_by_postal_code/*.csv ]; then echo " Move completed successfully"   else echo " ERROR : The file $DATA_EXTRACT/rating_by_postal_code/*.csv doesn't exist" fi

---

donc le if fonctionne de cette facon. Mais avec ta mise en garde N_BaH je ne sais plus quoi penser :

Il y a plusieurs fichiers dans mon répertoire par contre il n'y a pas d'espace dans les noms...


merci

Salut,

Je reste persuadé qu'un simple "ls *.csv" sur le répertoire et un test sur le code retour serait le plus approprié, d'autant plus que ton test actuel je ne suis pas sûr qu'il marche aussi bien que tu le dis, la preuve :

Code:

---

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

$ ls -1 abc.txt aoiu.txt baytre.txt cabiu.txt cuypo.txt iuyabcoiu.txt oiuapoi.txt ubiuy.txt $ if [ -e *.txt ];then echo ok;else echo Nok;fi -bash: [: trop d'arguments Nok $ set -xv;if [ -e *.txt ];then echo ok;else echo Nok;fi;set - + '[' -e abc.txt aoiu.txt baytre.txt cabiu.txt cuypo.txt iuyabcoiu.txt oiuapoi.txt ubiuy.txt ']' -bash: [: trop d'arguments + echo Nok Nok + set - $ ls *.txt abc.txt aoiu.txt baytre.txt cabiu.txt cuypo.txt iuyabcoiu.txt oiuapoi.txt ubiuy.txt $ echo $? 0 $ ls *.png ls: ne peut accéder *.png: Aucun fichier ou dossier de ce type $ echo $? 1 $

---

voilà, voilà. Hormis, si on peux ne pas utiliser une commande externe, on s'en passe; par économie de temps et de ressources.

Il faut donc privilégier l'usage de test, [, ou [[ dans les scripts, pour ... faire des tests :)

A noter, que les doubles crochets acceptent les noms de fichiers avec espaces (mais qu'ils ne marchent qu'en bash).

Je me permets d'apporter mon point de vue qui n'engage que moi.

Certes, chercher à optimiser les temps de traitements etc est une bonne chose, mais pas si cela se fait au détriment de la lecture !

Or entre

Code:

---

1 2	array=( $DATA_EXTRACT/rating_by_postal_code/*.csv ) if ((${#array[@]})); then echo "ok"; else echo "KO"; fi

---

et

Code:

---

1 2	if ls "$DATA_EXTRACT/rating_by_postal_code/"*.csv >/dev/null 2>&1; then echo "ok"; else echo "KO"; fi

---

Je trouve la second formule un peu plus facile à comprendre non ?

Et je doute que cela impacte beaucoup sur les perfs.

Bonjour;

@ Alek-C : Les deux codes sont suffisamment lisibles si on les indentes correctement.
On ne va pas proscrire les expressions bash (au profit de commandes externes en plus :roll: ) pour le fait que ça parait moins lisible (ou tout simplement parce qu'on ne les connait pas).

Il en est de même pour les regex, on ne va pas les proscrire et faire tout un tas de traitement pour compenser, sous prétexte qu'elles ne nous paraissent pas lisibles.

Cordialement,
Idriss

Citation:

---

Envoyé par Alek-C

Et je doute que cela impacte beaucoup sur les perfs.

---

si ce n'est que ça, peut-être pas...
mais, une commande externe plus une commande externe...plus une autre...et, là, ça devient sensible;

donc, j'évite d'utiliser la première :)