Discussion :

[alex94130]

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 :

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

[zipe31]

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

[al1_24]

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 :

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

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}

[Sve@r]

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

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

[alex94130]

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 »

[al1_24]

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