Discussion :

[decksroy]

Bonjour,
je voudrais faire un script qui ne puisse pas faire un traitement s'il ya au moins un fichier vide dans le repertoire.
voici le code :

Code :

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

1
2
3
4
5
6
 
for fichier in `ls $1`;do	
 if [ -s $fichier ] ; then echo "le fichier n'est pas vide" 
 else echo "le fichier est vide"
 fi
done

Le probleme c'est que ce script ne marche pas sur tout les repertoires, je n'y comprends rien.
Pour tester ce script j'ai donc creer un repertoire avec des fichiers vides avec la commande touch, puis j'ajoute du texte.
sur le premier test cela a marché il trouve les fichiers vides et les non vides.
Puis j'ai refais le test dans un autre repertoire, et la il me trouve toujours des fichier vides.
Je n'arrive pas a savoir d'ou vient le probleme.
Merci.

[D[r]eadLock]

Affiche la valeur de fichier.
Ajoute des "" autour de fichier dans le -s.
Ce serait pas une histoire de blancs dans les noms... Essaie de deviner en fonction des repertoires qui ne marchent pas (ou donne des exemples)

[BlaireauOne]

Ignorer les répertoires, les liens... ( tout ce qui n'est pas fichier )

Code :

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

1
2
3
4
5
6
7
8
9
10
11
for fichier in `ls $1`;do	
  if [ ! -f $fichier ] ; then 
    continue
  fi  
  if [ -s $fichier ] ; then 
    echo "le fichier \"$fichier\" n'est pas vide" 
  else 
    echo "le fichier \"$fichier\" est vide"
 fi
done

[decksroy]

Merci pour vos reponses.
Mais j'ai toujours le meme probleme.
J'ai rajouté les "" mais rien a faire.
j'ai mis le code de BlaireauOne mais c'est toujours pareil
pour tester cet exemple voici la procedure.
1)J'ai creer un repertoire, et dedans j'ai fais un touch de plusieurs fichiers
2) je fais un ls -l et j'ai bien la taille des fichiers a 0
3) je fais un vi sur 3 fichier et j'ajoute des characteres
donc 2 fichiers vide et 3 fichiers plein

j'execute le script
et j'obtiens sur les 5 fichier present dans le repertoire
2 fichiers qui "existe" :
le premier qui est vide avec la commande ls -l et qui n'a rien comme contenu
devient "n'est pas vide" comme resultat du sh
le deuxieme qui n'est pas vide.... deviens "vide" comme resultat
et les 3 autres n'existe pas pour le script.

Je n'y comprends vraiment rien.
J'espere que je vous donne assez de details pour voir d'ou peut venir le probleme.
Merci.

[MarkH]

Bonjour,

quelque chose m'étonne à la lecture du script : ta variable fichier contient le résultat d'un ls sur un nom de répertoire. Donc, tu récupères uniquement le nom du fichier sans le nom du répertoire.

Ton test sur la taille du fichier ne va donc réussir que si dans le répertoire courant il existe un fichier de même nom que dans le répertoire fourni en argument à ton script, et que ce fichier (du répertoire courant donc) est vide.

En gros, si l'argument à ton script est autre chose que '.' (le point, pour le répertoire courant), les résultats sont assez imprévisibles (et donc réciproquement, ton script doit donner le résultat attendu si l'argument est '.').

Me trompé-je ?

MarkH

[decksroy]

j'ai pas bien compris, mais la reponse doit etre peut etre la.
en effet en parametre je donne le repertoire absolue
du style ; /usr/home/repertoireTraitement.
qu'est ce que je dois mettre a la place?

[f-k-z]

Yop,

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
for fichier in `ls $1`;do	
  if [ ! -f $1/$fichier ] ; then 
    continue
  fi  
  if [ -s $1/$fichier ] ; then 
    echo "le fichier \"$1/$fichier\" n'est pas vide" 
  else 
    echo "le fichier \"$1/$fichier\" est vide"
 fi
done

essaies stp avec cette verison du code de BlaireauOne
@++

F-k-z

[decksroy]

Merci beaucoup.

[MarkH]

Bien, et maintenant la solution élégante

Pour avoir la liste des fichiers vides, il vaut mieux utiliser find, comme ça tu te débarrasses du if, et en plus tu récupères les noms des fichiers avec les chemins.

Ex :

Code :

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

1
2
3
4
5
 
fichiers=`find $1 -not -empty`
for fichier in fichiers; do
   echo "$fichier n'est pas vide"
done

Evidemment, pour avoir les fichiers vide, il suffit d'enlever le -not

Demain j'arrête d'être pédant

MarkH