Boucle While dans un IF

Bonjour,
Je suis débutant en shell et je cherche à copier plusieurs lignes d'un fichier (correspondant à des fichiers XML) vers un autre emplacement.

Voici mon script :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16

#!/bin/bash source=/chemin1/ cible=/chemin2/   fichier="$(find $source -mmin -$((`date +%H`*60)) -iname '*.xml')"   # copie du fichier si le fichier du jour est existant if [ -f "$fichier" ] then while read line echo "$(date +%d/%m/%y-%kh%M) : copie du fichier $line" >> $(date +%Y-%m-%d).copie.log cp "$line" $cible done < $fichier else echo "$(date +%d/%m/%y-%kh%M) : pas de nouveau fichier pour ce jour" >> $(date +%Y-%m-%d).copie.log fi

---

Le IF me sert à tester si des fichiers existent pour un jour J. S'il n'y a pas de fichiers, on ne doit rien copier.

Mon problème, c'est que cela ne fonctionne pas. Le résultat me renvoie systématiquement vers le ELSE, même lorsqu'il y a des fichiers dans le répertoire source.

Pouvez-vous m'aider ?

Je vous remercie !!

Salut,

C'est normal, if [ -f "$fichier" ] teste si un fichier existe, or, $fichier est une variable et non un fichier :?

Il te faut tester avec if [ -n "$fichier" ] qui vérifie si ta variable n'est pas vide...

Si ta commande find retourne plusieurs fichiers, la variable fichier contient ces noms de fichiers séparés par un espace.
La commande [ -f "fic1 fic2" ] cherche le fichier fic1 fic2, contenant un espace dans son nom, et non les fichiers fic1 et fic2.

Tu peux simplifier tes tests et boucles :

Code:

---

1 2 3 4 5 6 7 8

log=$(date '+%Y-%m-%d').copie.log nbfic=0 find ${source} -mmin -$(( $(date '+%H') * 60 )) -iname '*.xml' | while read nomfic echo "$(date '+%d/%m/%y-%kh%M') : copie du fichier ${nomfic}" >> ${log} cp "${nomfic}" ${cible} (( nbfic += 1 )) done (( nbfic )) || echo "$(date +%d/%m/%y-%kh%M) : pas de nouveau fichier pour ce jour" >> ${log}

---

Bonjour
En dehors du mot clef "do" manquant, je ne comprends pas trop ce cp "$line" $cible en ligne 12. En tout état de cause, la variable "$line" contient une ligne du fichier traité et non le fichier lui-même. Or la commande cp copie des fichiers, pas des lignes situées dans les fichiers...

Accessoirement tu gagneras en productivité et en mise à jour si tu prends l'habitude d'indenter proprement ton code.

Le même code indenté

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16

#!/bin/bash source=/chemin1/ cible=/chemin2/   fichier="$(find $source -mmin -$((`date +%H`*60)) -iname '*.xml')"   # copie du fichier si le fichier du jour est existant if [ -f "$fichier" ] then while read line echo "$(date +%d/%m/%y-%kh%M) : copie du fichier $line" >> $(date +%Y-%m-%d).copie.log cp "$line" $cible done < $fichier else echo "$(date +%d/%m/%y-%kh%M) : pas de nouveau fichier pour ce jour" >> $(date +%Y-%m-%d).copie.log fi

---

Merci pour votre aide.

En fait, à travers ce script, je récupère le nom des fichiers XML trouvés dans un répertoire ($source), et je les inscrits dans un fichier ($fichier).

Ensuite, pour chaque ligne inscrite, je copie la ligne (donc le fichier XML) vers une destination ($cible).

J'avais déjà fait un tel script qui fonctionne, mais sans la condition IF. Ce qui m'intéresse dans ce cas, c'est de pouvoir retourner un résultat si la condition n'est pas remplie.

Par contre, je me complique peut être la vie avec cette méthode. Si vous en avez d'autres, je suis preneur.

al1_24, merci pour ta réponse. Ton script renvoie une erreur au niveau du "While" :

line 10: Erreur de syntaxe près du symbole inattendu « done »

Comme l'a justement relevé Sve@r, il manque un do entre les lignes 3 et 4.

Bonjour,

Je vous remercie et vous fait un petit retour sur mon script.

Je me suis basé sur celui de "al1_24" qui est effectivement plus simple.

En revanche après le done :

Code:

---

(( nbfic )) || echo "$(date +%d/%m/%y-%kh%M) : pas de nouveau fichier pour ce jour" >> ${log}

---

J'ai toujours cette ligne qui s'affiche dans mon fichier de log, même lorsqu'il y a de nouveaux fichiers à transférer, n'y a t-il pas une erreur de syntaxe ?

exemple : 30/01/14-15h37 : pas de nouveau fichier pour ce jour

C'est sans doute la ligne 6 (( nbfic += 1 )) qui s'exécute mal.
Essaye de la remplacer par (( nbfic = nbfic + 1 ))

Bonjour,

al1_24 a oublié qu'un pipe s'exécute dans un sous-shell en dehors duquel les variables qui y sont créées n'existent pas.

Code:

---

1 2 3 4

$ echo "foo" | while read line; do var="$line"; done; echo "$var"   $ echo "foo" | { while read line; do var="$line"; done; echo "$var";} foo

---

ou, alors dans un script BASH, on peut activer l'option lastpipe :

Code:

---

1 2 3 4 5

#!/bin/bash   shopt -s lastpipe echo "foo" | while read line; do var="$line"; done echo $var

---

J'ai surtout oublié que c'était du bash et pas du ksh !

Ok cela fonctionne. un GRAND MERCI pour votre aide !!

Sympa de voir que quand on est une bille en shell on trouve quand même des gens dispos pour aider ;)

Je vais essayer de me former petit à petit...