Discussion :

[juin29]

Bonjour,

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

Mon code :

Code :

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

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

[N_BaH]

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 :

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

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 :

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

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.

[Elverion]

Bonjour,

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

Code :

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

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

[N_BaH]

bonjour Elverion,

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

[juin29]

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.

[juin29]

Bonjour,

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

Code :

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

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

[zipe31]

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 :

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

$

[N_BaH]

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

[FRUiT]

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

[lennelei]

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 :

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

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

et

Code :

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

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.

[ok.Idriss]

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

[N_BaH]

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