Shell aide sur mon script

Shell aide sur mon script - Shell/Batch - Programmation

Marsh Posté le 16-03-2007 à 15:07:13    

J'ai des competences en shell bash mais pas assez pour faire ce script
 
2 paramètres a renseigner en debut de script
- le dossier
- taille du dossier max en mega
 
Si le dossier a une taille > taille max --> on efface LE plus vieux fichier(il peut avoir des sous dossiers, sur plusieur niveaux),
si un sous dossiers est vide le supprimer.
 
J'espère être clair
Merci @+ franck

Message cité 1 fois
Message édité par dd_pak le 20-03-2007 à 11:10:05
Reply

Marsh Posté le 16-03-2007 à 15:07:13   

Reply

Marsh Posté le 17-03-2007 à 19:10:50    

dd_pak a écrit :

J'ai des competences en shell bash mais pas assez pour faire ce script
 
2 paramètres a renseigner en debut de script
- le dossier
- taille du dossier max en mega
 
Si le dossier a une taille > taille max --> on efface LE plus vieux fichier(il peut avoir des sous dossiers, sur plusieur niveaux),
si un sous dossiers est vide le supprimer.
 
J'espère être clair
Merci @+ franck


 
Bon, déjà un gros danger à prévoir => si tu fais "ls -ld dossier" tu n'obtiendras pas la taille du dossier dans le sens "taille cumulée de tous ses fichiers" mais tu obtiendras la taille du dossier dans le sens "taille utilisée pour stocker tous les noms des fichiers qui s'y trouvent"
Pour trouver la taille d'un dossier, tu dois utiliser la commande "du -sk dossier"
 
Ensuite, pour trouver le plus vieux fichier d'un répertoire, tu peux utiliser l'option "-t" de "ls" qui te donne les fichiers classés par date de dernière modif.
 
Enfin tu peux télécharger mon cours de shell ici http://fr.lang.free.fr/cours/Shell_v2.0.pdf. Tout le shell est expliqué. Tu y trouveras par exemple un script qui affiche une arborescence de façon récursive (un peu à la façon "tree" ) et d'autres exemples qui pourront t'aider pour faire ce devoir.
 
PS: Ce devoir n'est pas un devoir de débutant et demande quand-même un certain niveau de connaissances en shell. Tu es donc sensé avoir suivi un cours pour acquérir ces connaissances. Donc ne dis pas que t'as pas les compétences...


---------------
Vous ne pouvez pas apporter la prospérité au pauvre en la retirant au riche.
Reply

Marsh Posté le 19-03-2007 à 08:53:55    

Merci, je m'y colle et si j'ai des problèmes je post
@+

Reply

Marsh Posté le 20-03-2007 à 11:12:57    

Code :
  1. # Programme qui supprime LE plus vieux fichier d'un repertoire, si taille > taille max, et qui supprime les repertoires vides.
  2. # Et controle les extensions de fichers (.txt, .rar, .zip, .avi, .iso, .nfo, .xls, .ppt, .pdf, .doc)
  3. # Exclusions des repertoires /bin /dev /etc /lib...
  5. dossier="/ep10/home/kwado/franck/test/dossier"
  6. taille="50000" #egal 50 mega octet
  8. # taille de dossier
  9. vraitaille=$(du -sk "$dossier" | cut -f1)
  10. echo "La taille du dossier est $vraitaille"
  12. if  test "$vraitaille" -ge "$taille"
  13. then
  14. {
  15. echo "$vraitaille > $taille, suppression on"
  16. supp=1
  17. }
  18. fi
  21. # boucle sur les fichiers
  22. find $dossier |while read file
  23. do
  24. # pour chaque fichier
  25. # echo $file
  26. #Exclusions du repertoire /bin
  27. if [ `expr "$file" : "$dossier/bin/"` -ne 0 ]
  28.  then
  29.  {
  30.  echo "Passage du fichier $file"
  31.  continue
  32.  }
  33. fi
  34. if  test "$file" == "$dossier/bin"
  35.  then
  36.  {
  37.  echo "Passage du dossier $file"
  38.  continue
  39.  }
  40. fi
  41. #####################################Suppression des fichiers non valides
  42. ##################################### suppression des repertoires vides
  43. done
  46. if  test "$supp" == 1
  47. then
  48. {
  49. # suppression du fichier le plus vieux
  50. for file in `ls -ltr $dossier`
  51. #for file in `find . -type f | xargs ls -ltr`
  52. do
  53.   #Exclusions du repertoire /bin
  54.   if  test "$file" == "bin"
  55.    then
  56.    {
  57.    echo "Passage du dossier $file"
  58.    continue
  59.    }
  60.   fi
  61.   #test si c'est un repertoire
  62.   if test "$dossier/$file" == -d
  63.   then {
  64.    echo "$file est un répertoire, on avance"
  65.    # mettre un goto
  66.    }
  67.   fi
  68.   #si c'est un fichier on a gagne
  69.   if test "$dossier/$file" == -f 
  70.   then {
  71.    echo "$file est un fichier, on le supprime"
  72.    #commande de suppression
  73.    break
  74.    }
  75.   fi
  76.   #echo $dossier/$file
  77.   #break
  78. done
  79. }
  80. fi


 
Mon problème es que sur la deuxième boucle, le test fichier ou dossier n'est jamais vraie ?
et j'ai pas d'idée comment recuperé l'extension du fichier?
 
merci

Message cité 1 fois
Reply

Marsh Posté le 20-03-2007 à 12:57:15    

dd_pak a écrit :

Code :
  1. # Programme qui supprime LE plus vieux fichier d'un repertoire, si taille > taille max, et qui supprime les repertoires vides.
  2. # Et controle les extensions de fichers (.txt, .rar, .zip, .avi, .iso, .nfo, .xls, .ppt, .pdf, .doc)
  3. # Exclusions des repertoires /bin /dev /etc /lib...
  5. dossier="/ep10/home/kwado/franck/test/dossier"
  6. taille="50000" #egal 50 mega octet
  8. # taille de dossier
  9. vraitaille=$(du -sk "$dossier" | cut -f1)
  10. echo "La taille du dossier est $vraitaille"
  12. if  test "$vraitaille" -ge "$taille"
  13. then
  14. {
  15. echo "$vraitaille > $taille, suppression on"
  16. supp=1
  17. }
  18. fi
  21. # boucle sur les fichiers
  22. find $dossier |while read file
  23. do
  24. # pour chaque fichier
  25. # echo $file
  26. #Exclusions du repertoire /bin
  27. if [ `expr "$file" : "$dossier/bin/"` -ne 0 ]
  28.  then
  29.  {
  30.  echo "Passage du fichier $file"
  31.  continue
  32.  }
  33. fi
  34. if  test "$file" == "$dossier/bin"
  35.  then
  36.  {
  37.  echo "Passage du dossier $file"
  38.  continue
  39.  }
  40. fi
  41. #####################################Suppression des fichiers non valides
  42. ##################################### suppression des repertoires vides
  43. done
  46. if  test "$supp" == 1
  47. then
  48. {
  49. # suppression du fichier le plus vieux
  50. for file in `ls -ltr $dossier`
  51. #for file in `find . -type f | xargs ls -ltr`
  52. do
  53.   #Exclusions du repertoire /bin
  54.   if  test "$file" == "bin"
  55.    then
  56.    {
  57.    echo "Passage du dossier $file"
  58.    continue
  59.    }
  60.   fi
  61.   #test si c'est un repertoire
  62.   if test "$dossier/$file" == -d
  63.   then {
  64.    echo "$file est un répertoire, on avance"
  65.    # mettre un goto
  66.    }
  67.   fi
  68.   #si c'est un fichier on a gagne
  69.   if test "$dossier/$file" == -f 
  70.   then {
  71.    echo "$file est un fichier, on le supprime"
  72.    #commande de suppression
  73.    break
  74.    }
  75.   fi
  76.   #echo $dossier/$file
  77.   #break
  78. done
  79. }
  80. fi


 
Mon problème es que sur la deuxième boucle, le test fichier ou dossier n'est jamais vraie ?
et j'ai pas d'idée comment recuperé l'extension du fichier?
 
merci


 
Bon, à ta première boucle tu fais un "find |while read" => ok - Je ne sais pas si t'as pris cette idée chez-moi (et donc si t'as compris le principe) ou si tu connaissais déjà l'astuce.
Pourquoi, dans ta seconde boucle, tu ne fais pas pareil ? Style "find -type f |xargs ls -ltr | while read file" ???
Petite remarque en passant : La commande "find" peut recevoir une commande qu'elle appliquera automatiquement sur les fichiers qu'elle trouve. Autrement dit, tu peux remplacer "find -type f |xargs ls -ltr" par "find -type f -exec ls -ltr {} \;"
 
Ensuite, je ne comprends pas trop ce que tu cherches quand tu fais "if test $dossier/$file == -f" car si la variable "$file" récupère une ligne du "ls" (au fait, t'as oublié de sauter la première ligne du "ls" qui indique "total: xxx" ) alors il n'y aura jamais "-f" ou "-d". Si c'est un fichier normal il y aura "-" suivi des droits "rwxrwxrwx", si c'est un répertoire, il y aura "d" suivi des droits. Donc il te faudra extraire le premier caractère de "$file" et comparer ce premier caractère à "-" ou à "d" mais sûrement pas "-f" ni "-d".
De toute façon, c'est une méthode bien compliquée pour déterminer le type du fichier alors que t'as "test -d". Ah, ça y est, j'ai compris. Tu as cru que dans la commande "test" il fallait mettre "test fichier = -d". Non, l'option "-d" ou "-f" sont des options de la commande "test" donc il faut les mettre avant
=> test -f "$dossier/$file"
=> test -d "$dossier/$file"
Bien entendu, il faut que "$file" ne récupère que le nom du fichier et pas les informations au format "long" donc faut enlever l'option "-l" du "ls"...
 
Comment récupérer une extension ? Un extension est une chaîne qui suit un ".". Donc tu peux utiliser la commande "cut" en coupant sur le ".". Exemple:
nom="toto.doc"
base=`echo $nom |cut -f1 -d.`           => base="toto"
ext=`echo $nom |cut -f2 -d.`             => ext="doc"
 
Maintenant, si le nom possède plusieurs extensions, style "toto.tar.gz", alors ça devient plus compliqué et ça dépend de ta façon de voir. Si tu considères que l'extension est "tar.gz" alors ma méthode marche encore ; si tu considères que c'est "gz" alors ça devient plus compliqué (faut compter les "." puis ne prendre que le dernier) et si tu veux récupérer et isoler chaque extension ("tar", "gz" ) alors il faut faire une boucle.
 
Petites remarques en pasant:

  • goto n'existe pas en shell...
  • 50000 ko ne font pas 50 Mo - 50 Mo = 50 * 1024 ko soit 51200 ko soit 51200 * 1024 o = 52428800 octets - Il n'y a que les constructeurs de disques qui font semblant de ne pas le savoir...


Message édité par Sve@r le 20-03-2007 à 13:10:16

---------------
Vous ne pouvez pas apporter la prospérité au pauvre en la retirant au riche.
Reply

Marsh Posté le 20-03-2007 à 14:59:01    

Code :
  1. nom="toto.tar.gz"
  2.     point=`expr "$nom" : .`
  3.     echo $point
  4.     base=`echo $nom |cut -f1 -d.`         
  5.     ext=`echo $nom |cut -f2 -d.`   
  7.     echo $base
  8.     echo $ext


 
Pour la taille, merci mais je me doute que c'est approximatif, je connais cette règle
Pour le test --d et -f, j'ai trouvé il faut un espace derrière sinon ca ne marche pas
Pour l'extension je dirais .tar.gz => c'est un gz
Donc il faut compter le nombre de point pour le cut mais je n'y arrive pas, il me renvoie 0 ou 1 mais pas 2?
 
Je te remercie pour ton aide, oui je me suis inspriré de tes conseils, je code mais la c'est mon premier script shell alors j'ai un peu de mal
merci @+

Message cité 1 fois
Reply

Marsh Posté le 20-03-2007 à 17:07:43    

dd_pak a écrit :

Code :
  1. nom="toto.tar.gz"
  2.     point=`expr "$nom" : .`
  3.     echo $point
  4.     base=`echo $nom |cut -f1 -d.`         
  5.     ext=`echo $nom |cut -f2 -d.`   
  7.     echo $base
  8.     echo $ext


 
Pour la taille, merci mais je me doute que c'est approximatif, je connais cette règle
Pour le test --d et -f, j'ai trouvé il faut un espace derrière sinon ca ne marche pas
Pour l'extension je dirais .tar.gz => c'est un gz
Donc il faut compter le nombre de point pour le cut mais je n'y arrive pas, il me renvoie 0 ou 1 mais pas 2?


 
Ben si tu regardes bien le "cut", je demande le "-f2" c.a.d. le champ n° 2 (sous-entendu "séparé du champ "1" par un "." ) en ayant présumé que le nom n'avait que 2 champs.
Maintenant, si le nom a plusieurs "." non connus à l'avance, il y a plusieurs solutions mais à mon avis, la plus simple est de filtrer le nom par un "awk"
nom="toto.tar.gz"
ext=`echo $nom |awk -F. '{print $NF}' `        => ext="gz"
Explication: chaque ligne traitée par "awk" contient dans sa variable "NF" le nombre de champs de la ligne en question (le séparateur de champs étant indiqué par le "-F" donc ce sera "." ). Donc en affichant la variable "$NF" (qui est une variable spécifique au langage "awk" ), j'affiche le texte correspondant au dernier champ et ce, quel que soit le nombre de champs du flux entrant.


---------------
Vous ne pouvez pas apporter la prospérité au pauvre en la retirant au riche.
Reply

Marsh Posté le 21-03-2007 à 09:51:44    

Merci mon script et enfin fini, merci pour ton aide je crois pas que j'aurais reussi tout seul
Je suis consient que l'on peut l'ameliorer mais pour le moment ca me suffit
Voici le code ca peut toujours servir à quelq'un.
Encore merci pour ton aide et tes explications
 

Code :
  1. # Programme qui supprime LE plus vieux fichier d'un repertoire, si taille > taille max, et qui supprime les repertoires et fichiers vides.
  2. # Et controle les extensions de fichers (.txt, .rar, .zip, .avi, .iso, .nfo, .xls, .ppt, .pdf, .doc)
  3. # Exclusions des repertoires /bin /dev /etc /lib...
  4. # Renome les fichiers avec espace acents...
  6. dossier="/ep10/home/kwado/franck/test/dossier"
  7. taille="50000" #egal-environ 50 mega octet
  8. # Affiche la date pour le log
  9. DATE=`date +"%Hh%M"`
  10. DATE2=`date +"%d/%m"`
  11. echo "Lancé à $DATE le $DATE2"
  13. # Renome les fichiers zarbi...:)
  14. find $dossier |while read file
  15. do
  16.    file_mod=`echo "$file" | tr '[àâäéèêëîïôöùûü ]' '[aaaeeeeiioouuu_]' | sed -e "s/'/_/g"`
  17.    if test "$file" != "$file_mod"
  18. then
  19. {
  20. echo "$file devient $file_mod"
  21. mv "$file" "$file_mod"
  22. }
  23.    fi
  24. done
  26. #################FONCTION
  27. old(){
  28. dossier=$1
  29. if  test "$supp" == 1
  30. then
  31. {
  32. # suppression du fichier le plus vieux
  33. for file in `ls -latrC $dossier`
  34. #for file in `find . -type f | xargs ls -ltr`
  35. do
  36.   #Exclusions du repertoire /bin
  37.   if  test "$file" == "bin"
  38.    then
  39.    {
  40.    echo "Passage du dossier $file"
  41.    continue
  42.    }
  43.   fi
  44.   if  test "$file" == "."
  45.    then
  46.    {
  47.    echo "Passage du dossier $file"
  48.    continue
  49.    }
  50.   fi
  51.   if  test "$file" == ".."
  52.    then
  53.    {
  54.    echo "Passage du dossier $file"
  55.    continue
  56.    }
  57.   fi
  58.   #test si c'est un repertoire
  59.   if test -d $dossier/$file
  60.   then {
  61.    echo "$file est un répertoire, on avance"
  62.    # mettre un goto
  63.    dossier=$dossier/$file
  64.    old $dossier
  65.    }
  66.   fi
  67.   #si c'est un fichier on a gagne
  68.   if test -f $dossier/$file 
  69.   then {
  70.    echo "$file est un fichier, on le supprime car il est le plus vieux"
  71.    #commande de suppression
  72.    rm $dossier/$file
  73.    break
  74.    }
  75.   fi
  76.   #echo $dossier/$file
  77.   #break
  78. done
  79. }
  80. fi
  81. }
  83. ###########################PROGRAMME
  84. # taille de dossier
  85. vraitaille=$(du -sk "$dossier" | cut -f1)
  86. echo "La taille du dossier est $vraitaille"
  88. if  test "$vraitaille" -ge "$taille"
  89. then
  90. {
  91. echo "$vraitaille > $taille, suppression on"
  92. supp=1
  93. }
  94. fi
  97. # boucle sur les fichiers
  98. find $dossier |while read file
  99. do
  100. # pour chaque fichier
  101. # echo $file
  102. #Exclusions du repertoire /bin
  103. if [ `expr "$file" : "$dossier/bin/"` -ne 0 ]
  104.  then
  105.  {
  106.  echo "Passage du fichier $file"
  107.  continue
  108.  }
  109. fi
  110. if  test "$file" == "$dossier/bin"
  111.  then
  112.  {
  113.  echo "Passage du dossier $file"
  114.  continue
  115.  }
  116. fi
  117. # suppression fichiers vides
  118. if test -f $file 
  119.   then {
  120.    if ! test -s $file
  121.     then {
  122.     echo "$file est un fichier vide on le supprime"
  123.     #commande de suppression
  124.     rm $file
  125.     }
  126.    fi
  127.    }
  128. fi
  129. done
  131. #suppression des dossiers vides
  132. find $dossier -type d -empty -delete
  134. #Verification des extensions
  135. find $dossier |while read file
  136. do
  137. #Exclusions du repertoire /bin
  138. if [ `expr "$file" : "$dossier/bin/"` -ne 0 ]
  139.  then
  140.  {
  141.  #echo "Passage du fichier $file"
  142.  continue
  143.  }
  144. fi
  145. #on ne verifie que les fichiers
  146. if test -f $file 
  147.  then {
  148.  ext=1
  149.  extf=`echo $file |awk -F. '{print $NF}' `
  150.  #echo $extf
  152.  if test "$extf" == "gz" || test "$extf" == "txt"
  153.   then
  154.   {
  155.   ext=0
  156.   }
  157.  fi
  158.  if  test "$ext" == 1
  159.   then
  160.   {
  161.   echo "Suppression de $file mauvaise extension"
  162.   rm $file
  163.   }
  164.  fi
  165.  }
  166. fi
  167. done
  168. #lancement de la fonction
  169. old $dossier

Message cité 1 fois
Reply

Marsh Posté le 21-03-2007 à 12:53:28    

dd_pak a écrit :

Merci mon script et enfin fini, merci pour ton aide je crois pas que j'aurais reussi tout seul
Encore merci pour ton aide et tes explications


It's my job...
 

dd_pak a écrit :

# Affiche la date pour le log
DATE=`date +"%Hh%M"`
DATE2=`date +"%d/%m"`
echo "Lancé à $DATE le $DATE2"



Petit danger: les variables en majuscules sont réservées au système. Imagine que, sans le savoir, tu aies utilisé la variable "PATH" ou "HOME" ??? Ben c'est pareil. Tu ne sais pas si le système n'utilise pas, pour lui, "DATE". Evidemment c'est improbable mais c'est pour le principe. Et puis un nom en minuscule est souvent plus lisible...
Sinon amélioration rapide:

# Affiche la date pour le log
echo "Lancé à `date +'%Hh%M'` le `date +'%d/%m'`"


 

dd_pak a écrit :

Code :
  1. # Renome les fichiers zarbi...:)
  2. find $dossier |while read file
  3. do
  4.    file_mod=`echo "$file" | tr '[àâäéèêëîïôöùûü ]' '[aaaeeeeiioouuu_]' | sed -e "s/'/_/g"`
  5.    if test "$file" != "$file_mod"
  6. then
  7. {
  8. echo "$file devient $file_mod"
  9. mv "$file" "$file_mod"
  10. }
  11.    fi
  12. done
  14. #################FONCTION
  15. old(){
  16. dossier=$1
  17. if  test "$supp" == 1
  18. then
  19. {
  20. # suppression du fichier le plus vieux
  21. for file in `ls -latrC $dossier`
  22. #for file in `find . -type f | xargs ls -ltr`
  23. do
  24.   #Exclusions du repertoire /bin
  25.   if  test "$file" == "bin"
  26.    then
  27.    {
  28.    echo "Passage du dossier $file"
  29.    continue
  30.    }
  31.   fi
  32.   if  test "$file" == "."
  33.    then
  34.    {
  35.    echo "Passage du dossier $file"
  36.    continue
  37.    }
  38.   fi
  39.   if  test "$file" == ".."
  40.    then
  41.    {
  42.    echo "Passage du dossier $file"
  43.    continue
  44.    }
  45.   fi
  46.   #test si c'est un repertoire
  47.   if test -d $dossier/$file
  48.   then {
  49.    echo "$file est un répertoire, on avance"
  50.    # mettre un goto
  51.    dossier=$dossier/$file
  52.    old $dossier
  53.    }
  54.   fi
  55.   #si c'est un fichier on a gagne
  56.   if test -f $dossier/$file 
  57.   then {
  58.    echo "$file est un fichier, on le supprime car il est le plus vieux"
  59.    #commande de suppression
  60.    rm $dossier/$file
  61.    break
  62.    }
  63.   fi
  64.   #echo $dossier/$file
  65.   #break
  66. done
  67. }
  68. fi
  69. }
  71. ###########################PROGRAMME
  72. # taille de dossier
  73. vraitaille=$(du -sk "$dossier" | cut -f1)
  74. echo "La taille du dossier est $vraitaille"
  76. if  test "$vraitaille" -ge "$taille"
  77. then
  78. {
  79. echo "$vraitaille > $taille, suppression on"
  80. supp=1
  81. }
  82. fi
  85. # boucle sur les fichiers
  86. find $dossier |while read file
  87. do
  88. # pour chaque fichier
  89. # echo $file
  90. #Exclusions du repertoire /bin
  91. if [ `expr "$file" : "$dossier/bin/"` -ne 0 ]
  92.  then
  93.  {
  94.  echo "Passage du fichier $file"
  95.  continue
  96.  }
  97. fi
  98. if  test "$file" == "$dossier/bin"
  99.  then
  100.  {
  101.  echo "Passage du dossier $file"
  102.  continue
  103.  }
  104. fi
  105. # suppression fichiers vides
  106. if test -f $file 
  107.   then {
  108.    if ! test -s $file
  109.     then {
  110.     echo "$file est un fichier vide on le supprime"
  111.     #commande de suppression
  112.     rm $file
  113.     }
  114.    fi
  115.    }
  116. fi
  117. done
  119. #suppression des dossiers vides
  120. find $dossier -type d -empty -delete
  122. #Verification des extensions
  123. find $dossier |while read file
  124. do
  125. #Exclusions du repertoire /bin
  126. if [ `expr "$file" : "$dossier/bin/"` -ne 0 ]
  127.  then
  128.  {
  129.  #echo "Passage du fichier $file"
  130.  continue
  131.  }
  132. fi
  133. #on ne verifie que les fichiers
  134. if test -f $file 
  135.  then {
  136.  ext=1
  137.  extf=`echo $file |awk -F. '{print $NF}' `
  138.  #echo $extf
  140.  if test "$extf" == "gz" || test "$extf" == "txt"
  141.   then
  142.   {
  143.   ext=0
  144.   }
  145.  fi
  146.  if  test "$ext" == 1
  147.   then
  148.   {
  149.   echo "Suppression de $file mauvaise extension"
  150.   rm $file
  151.   }
  152.  fi
  153.  }
  154. fi
  155. done
  156. #lancement de la fonction
  157. old $dossier



 
Petites remarques mineures
1) en général, une fonction est écrite en premier car on considère qu'on pourrait avoir à s'en servir de partout. Imagines que t'aies à t'en servir lors du renommage des fichiers zarbi, ben tu pourrais pas.
2) je ne comprends pas à quoi sert la variable "supp" dans ta fonction. Tu testes si elle est à "1" mais sinon tu ne fais rien. Autant ne pas appeler "old" si ce n'est pas utile. Par ailleurs, si vraiment ta fonction "old" devait recevoir un paramètre à "1", autant le lui passer directement comme paramètre justement et non pas par le biais d'une variable venue du dehors. Si dans ton code principale il te prend l'envie de modifier le nom "supp" en autre chose, faudra reporter cette modif dans la fonction !!!
3) je ne comprends pas trop pourquoi tu mets l'option "-l" dans ton "ls" ce qui t'oblige à mettre l'option "C" sinon t'aurais un affichage au format long. Autant faire un "ls -atr" ou, au pire, si t'es pas sûr de toi, un "ls -atr1" qui force l'affichage à n'être que sur une seule colonne...
4) une fonction ne travaille pas sur une copie des variables mais sur les variables elle-mêmes ça c'est très dangereux. Dans ta fonction "old" tu utilises les variables "$dossier" et "$supp" et "$file". Tu utilises aussi ces mêmes variables dans ton code principal.
Imagines que t'aies un truc de ce style

for $file in truc
do
      old $file
      <...>
      traitement suivant sur $file
done


Ben le passe par "old" a écrasé le "$file" par celui qui est dans "old". Et ton traitement suivant plante.
 
Pour éviter ce danger, toujours mettre ton code de fonction entre parenthèses - Ca ne change pas l'accès aux valeurs des variables mais ce ne sont que des copies et les originaux ne sont pas modifiés
 

old(){
(
 dossier=$1
 if  test "$supp" == 1
 then
  # suppression du fichier le plus vieux
  for file in `ls -atr $dossier`
  do
   #Exclusions du repertoire /bin, "." et ".."
   if  test "$file" == "bin" -o "$file" = "." -o "$file" = ".."
   then
    echo "Passage du dossier $file"
    continue
   fi
   #test si c'est un repertoire
   if test -d $dossier/$file
   then
    echo "$file est un répertoire, on entre dedans"
    old $dossier/$file
   fi
   #si c'est un fichier on a gagne
   if test -f $dossier/$file  
   then
    echo "$file est un fichier, on le supprime car il est le plus vieux"
    #commande de suppression
    rm -f $dossier/$file
    break
   fi
   #echo $dossier/$file
   #break
  done
 fi
)
}


 
PS: Les accolades dans les corps des structures for/if/while ne sont pas obligatoires. Les mots clefs "then/else/fi/do/done" permettent au shell de se repérer dans le code. Sinon j'aime bien ton script. On sent que t'es à l'aise dans la syntaxe shell et t'as bien pigé la récursivité (ta fonction "old" qui s'appelle elle-même). Le reste viendra avec le temps...


Message édité par Sve@r le 21-03-2007 à 13:10:02

---------------
Vous ne pouvez pas apporter la prospérité au pauvre en la retirant au riche.
Reply

Marsh Posté le 21-03-2007 à 14:30:48    

Citation :

On sent que t'es à l'aise dans la syntaxe shell


 
Merci encore, je corrige tout les petites remarques, sinon non je suis pas à l'aise c'est pour ca que tu m'a aidé, c'est mon premier script...
mais je code en perl,php,sql,javascript,html donc ca aide

Reply

Sujets relatifs:

Leave a Replay

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