Ce script ne veut pas fonctionner

Ce script ne veut pas fonctionner - Codes et scripts - Linux et OS Alternatifs

Marsh Posté le 14-02-2004 à 21:26:59    

Bon, c'est le dernier....
 
Qu'est-ce qui marche pas dans ce foutu script.... :pfff:  
 
Tu rentre la valeur et le dossier et il affiche tous les fichiers qui dépasse la valeur dans le dossier...
 

Code :
  1. #!/bin/bash
  2. echo "Taille min des fichiers a afficher : " $1
  3. echo "dans le repertoire                 : " $2
  4. echo
  6. rm -rf temp
  7. ls -lS $2 | cut -b39-50,57- >> temp
  8. while read ligne
  9. do
  11.            taille=`echo $ligne | cut -f1 -d' '`
  12.            fichier=`echo $ligne | cut -f4 -d' '`
  13.            date=`echo $ligne | cut -f2 -d' '`
  14.            mois=`echo $ligne | cut -f3 -d' '`
  16. for i in $2; do
  17. if [ -f $i ]; then
  18. if [ ${taille} -gt ${1} ];then
  19. echo -e "$fichier $date $mois $taille"
  20. fi
  21. fi
  22. done
  23. done <temp


Reply

Marsh Posté le 14-02-2004 à 21:26:59   

Reply

Marsh Posté le 14-02-2004 à 21:30:33    

Messages d'erreurs ?


---------------
-~- Libérez Datoune ! -~- Camarade, toi aussi rejoins le FLD pour que la flamme de la Révolution ne s'éteigne pas ! -~- A VENDRE
Reply

Marsh Posté le 14-02-2004 à 21:34:19    

Si je laisse le "if" qui teste si c'est un fichier, j'ai aucun message d'erreur mais il liste rien
 
Si j'enlève ce "if", il liste ce que je veux, mais AUSSI les dossiers, et ca il ne faudrait pas. En plus il me faire un "unary error expected"...Pourtant je compare bien 2 valeures qui se compare...
 
Je suis tellement concentré dedans que je n'y vois plus rien...

Reply

Marsh Posté le 14-02-2004 à 23:31:31    

Execute ton script avec affichage des commandes exécutées et analyse se qui se passe.
 
bash -x the_script taille rep 2>&1 | more


---------------
Jean Pierre.
Reply

Marsh Posté le 15-02-2004 à 00:06:04    

Peut être que mon code ne fait pas ce que je pense qu'il doit faire...
 
 

Code :
  1. [---------------------]$ bash -x t.sh 40 coco
  2. + echo 'Taille min des fichiers a afficher : ' 40
  3. Taille min des fichiers a afficher :  40
  4. + echo 'dans le repertoire                 : ' coco
  5. dans le repertoire                 :  coco
  6. + echo
  8. + rm -rf temp
  9. + ls -lS coco
  10. + cut -b39-50,57-
  11. + read ligne
  12. ++ echo
  13. ++ cut -f1 '-d '
  14. + taille=
  15. ++ echo
  16. ++ cut -f4 '-d '
  17. + fichier=
  18. ++ echo
  19. ++ cut -f2 '-d '
  20. + date=
  21. ++ echo
  22. ++ cut -f3 '-d '
  23. + mois=
  24. + '[' -f coco ']'
  25. + read ligne
  26. ++ echo 4096 Feb 14 sub
  27. ++ cut -f1 '-d '
  28. + taille=4096
  29. ++ echo 4096 Feb 14 sub
  30. ++ cut -f4 '-d '
  31. + fichier=sub
  32. ++ echo 4096 Feb 14 sub
  33. ++ cut -f2 '-d '
  34. + date=Feb
  35. ++ echo 4096 Feb 14 sub
  36. ++ cut -f3 '-d '
  37. + mois=14
  38. + '[' -f coco ']'
  39. + read ligne
  40. ++ echo 12 Feb 14 goe
  41. ++ cut -f1 '-d '
  42. + taille=12
  43. ++ echo 12 Feb 14 goe
  44. ++ cut -f4 '-d '
  45. + fichier=goe
  46. ++ echo 12 Feb 14 goe
  47. ++ cut -f2 '-d '
  48. + date=Feb
  49. ++ echo 12 Feb 14 goe
  50. ++ cut -f3 '-d '
  51. + mois=14
  52. + '[' -f coco ']'
  53. + read ligne
  54. [------------------]$ bash t.sh 40 coco
  55. Taille min des fichiers a afficher :  40
  56. dans le repertoire                 :  coco
  58. [------------------]$

Reply

Marsh Posté le 15-02-2004 à 06:20:08    

Je l'ai modifié un peu mais ca donne pas encore le résultat escompté....
 
-Il ne doit pas lister les répertoires, juste les fichiers
-Il me fait cette erreur
-J'arrive pas à le formater
 

Code :
  1. #!/bin/bash
  2. echo "Taille min des fichiers a afficher : " $1
  3. echo "dans le repertoire                 : " $2
  4. echo
  6. rm -rf temp
  7. #ls -lS $2 | cut -b39-50,57- >> temp
  8. ls -lS $2 >> temp
  9. while read ligne
  10. do
  12.            taille=`echo $ligne | cut -f 5 -d' '`
  13.            fichier=`echo $ligne | cut -f 9 -d' '`
  14.            date=`echo $ligne | cut -f 7 -d' '`
  15.            mois=`echo $ligne | cut -f 6 -d' '`
  17. #for i in $2; do
  18. #if [ -f "$i" ]; then
  19. if [ ${taille} -gt ${1} ];then
  20. echo -e "\t$fichier\t$date\t$mois$taille"
  21. fi
  22. #fi
  23. #done
  24. done <temp
  26. #fin
  28. [--------@--------]$ bash t.sh 60 coco
  29. Taille min des fichiers a afficher :  60
  30. dans le repertoire                 :  coco
  32. t.sh: line 19: [: -gt: unary operator expected
  33.         sub     14      Feb4096
  34.         tp1no5.sh       14      Feb996
  35.         t.sh    14      Feb492
  36.         coloce  14      Feb166
  37. [--------@--------]$

Reply

Marsh Posté le 15-02-2004 à 07:44:55    

tu veux du récursif aussi, c'est à dire lister les fichiers des sous-répertoires aussi ?

Reply

Marsh Posté le 15-02-2004 à 08:26:07    

bon, avec du récursif :
 


#!/bin/sh
 
taille=$1
rep="$2"
 
[ "$1" = "" ] && read -p "Entrer la taille maximale en octet : " taille || echo "Taille maximale en octet : $taille"
[ "$2" = "" ] && read -p "Entrer le répertoire racine à lister : " rep || echo "Répertoire racine à lister : $rep"
 
count=0
echo
for i in `find "$rep" -type f`
   do
      tf=`du -b $i | cut -f1 -d'/'`
      [ $tf -gt $taille ] && count=`expr $count + 1` && echo "$i : $tf"
   done
 
[ $taille -gt 1 ] && s="s "
[ $count -gt 1 ] && ss="s"
 
echo
echo "Nombre de fichier$ss dans '$rep' dont la taille est supérieure à $taille octet$s: $count"
echo


 
résultat :


<-=[ ~/test-scripts ]=-> ./test
Entrer la taille maximale en octet : 500000
Entrer le répertoire racine à lister : /tmp
 
/tmp/kde-bmo/ksycoca : 1052672
/tmp/coreutils_5.0.91-2_i386.deb : 2449408
/tmp/coreutils-5.1.2-1mdk.i586.rpm : 2244608
/tmp/coreutils-5.0.91.tgz : 2461696
/tmp/coreutils-5.1.2.tgz : 2252800
 
Nombre de fichiers  dans /tmp dont la taille est supérieure à 500000 octets : 5


Message édité par BMOTheKiller le 15-02-2004 à 08:46:38
Reply

Marsh Posté le 15-02-2004 à 10:03:59    

_Faust_, je pense que ton problème vient des 'cut'.Dasn le résultat de la commande 'ls' les colonnes sont séparées par plusieurs espaces et pour cut chaque espace délimite un champ (ce n'est pas comme pour le shell).


-rwxr-xr-x    1 jp mkgroup        36 Feb  6 19:43 a.aw
^f1           ^f5


Tu te retrouves avec taille="", ce qui te donne pour le test
if [ -gt val$1 ]; then
d'où le pb de -gt: unary operator expected  
 
Il vaut mieux mettre les opérandes entre guillemets dans le test
if [ "${taille}" -gt "${1}" ]; then
 
Il vaut mieux utiliser 'read' pour décomposer la ligne.
De plus comme tu liste les fichiers par ordre décroissant (-S),
tu peux arrêter de tester dés que tu tu rencontres un fichier plus petit.
[/fixed]
ls -lS $2 | \  
while read perms hlinks owner group taille date mois fichier  
do  
   if [ "${taille}" -gt "${1}" ]
   then  
      echo -e "\t$fichier\t$date\t$mois$taille"  
   else
      break    
   fi  
done
[/fixed]


---------------
Jean Pierre.
Reply

Marsh Posté le 15-02-2004 à 10:05:11    

aigles a écrit :

_Faust_, je pense que ton problème vient des 'cut'.Dasn le résultat de la commande 'ls' les colonnes sont séparées par plusieurs espaces et pour cut chaque espace délimite un champ (ce n'est pas comme pour le shell).


-rwxr-xr-x    1 jp mkgroup        36 Feb  6 19:43 a.aw
^f1           ^f5


Tu te retrouves avec taille="", ce qui te donne pour le test
if [ -gt val$1 ]; then
d'où le pb de -gt: unary operator expected  
 
Il vaut mieux mettre les opérandes entre guillemets dans le test
if [ "${taille}" -gt "${1}" ]; then
 
Il vaut mieux utiliser 'read' pour décomposer la ligne.
De plus comme tu liste les fichiers par ordre décroissant (-S),
tu peux arrêter de tester dés que tu tu rencontres un fichier plus petit.


ls -lS $2 | \  
while read perms hlinks owner group taille date mois date3 fichier  
do  
   if [ "${taille}" -gt "${1}" ]
   then  
      echo -e "\t$fichier\t$date\t$mois$taille"  
   else
      break    
   fi  
done


 


---------------
Jean Pierre.
Reply

Marsh Posté le 15-02-2004 à 10:05:11   

Reply

Marsh Posté le 15-02-2004 à 17:27:40    

aigles, ca marche ceci chez toi ou il y a qq chose que je n'ai pas compris?
 

Code :
  1. #!/bin/bash
  2.   echo "Taille min des fichiers a afficher : " $1
  3.   echo "dans le repertoire                 : " $2
  4.   echo
  7. ls -lS $2 | \
  8. while read perms hlinks owner group taille date mois date3 fichier
  9. do
  10.    if [ "${taille}" -gt "${1}" ]
  11.    then
  12.       echo -e "\t$fichier\t$date\t$mois$taille"
  13.    else
  14.       break
  15.    fi
  16. done
  20.   #fin


 

Reply

Marsh Posté le 15-02-2004 à 19:06:25    

Désolé, ça ne marche effectivement pas, 'read' ne semble pas capable de lire des données depuis un pipe (avec ksh pas de pb).
 
Voila donc une nouvelle version du script:

Code :
  1. #!/usr/bin/bash
  2. echo "Taille min des fichiers a afficher : " $1
  3. echo "dans le repertoire                 : " $2
  4. echo
  6. ls -lS $2 | tail +2 > $$.tmp
  7. while read perms hlinks owner group taille date mois date3 fichier
  8. do
  9.    if [ "${taille}" -gt "${1}" ]
  10.    then
  11.       echo -e "\t$fichier\t$date\t$mois$taille"
  12.    else
  13.       break
  14.    fi
  15. done < $$.tmp
  16. rm -f $$.tmp


---------------
Jean Pierre.
Reply

Marsh Posté le 15-02-2004 à 22:08:06    

bon, grâce à vous ca très bien. J'ai changé quelques trucs pour le peaufiner, et j'ai fais une boucle pour ne pas lister les dossiers.  
 
D'ailleurs, c'est la seule facon que j'ai trouvé pour ne pas lister les dossiers, mais je me demandais si c'était "de la programmation conforme" ou si c'est inhabituel.
 

Code :
  1. #!/usr/bin/bash
  3. min=${1:?"Vous devez specifiez la taille minimum"}
  4. folder=${2:?"Specifiez un dossier"}
  6.   echo -e "\nTaille min des fichiers a afficher : " $min
  7.   echo "dans le repertoire                 : " $folder
  8.   echo
  9.   if [ ! -d $folder ]
  10.   then
  11.   echo "$folder n'est pas un dossier"
  12. else
  13.   ls -lS $folder | tail +2 > $$.tmp
  14.   while read perms hlinks owner group taille date mois date3 fichier
  15.   do
  16.      if [ "${taille}" -gt "${min}" ]
  17.      then
  18.       if  [ "$perms" != "drwx------" ]
  19. then
  20.         echo -e "\t$fichier   \t\t$date $mois\t\t$taille"
  21.     fi
  22. else
  23.           break
  24.      fi
  26.   done < $$.tmp
  27. fi
  28.   rm -f $$.tmp


Message édité par _Faust_ le 15-02-2004 à 22:11:58
Reply

Marsh Posté le 16-02-2004 à 13:08:37    

Tu ne devrais pas tester de cette façon si le fichier est un répertoire.
Les permissions sur le répertoires ne sont pas obligatoirement "drwx------", par contre cela commence toujours par "d". Il suffit fonc d'éliminer les fichiers pour lesquels perms commence par "d" :

Code :
  1. ls -lS $folder | tail +2 > $$.tmp
  2. while read perms hlinks owner group taille date mois date3 fichier
  3. do 
  4.    [[ "$perms" =  d* ]] && continue
  5.    [[ "${taille}" -gt "${min}" ]] && break
  6.    echo -e "\t$fichier   \t\t$date $mois\t\t$taille"
  7. done < $$.tmp


---------------
Jean Pierre.
Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed