Discussion :

[DienF]

Bonjour,
je dois écrire un script Bash qui prend facultativement deux arguments : un répertoire & une extension (par exemple sh).
Le script doit :
-vérifier le nombre d’arguments (0 ou 2) et, s’il est 0 demander à l’utilisateur de les saisir ; ils seront nommés dans le script respectivement rep et ext ;
-vérifier que $rep existe et qu’il soit bien un répertoire, sinon sortir ;
-utiliser une boucle de type for pour compter dans le répertoire $rep :

• les fichiers de type répertoire (fr) ;
• les fichiers ordinaires (fo) ;
• les fichiers d’extension $ext (fe) & parmi eux ceux qui sont exécutables (fex).

-afficher les différents compteurs sur des lignes séparées.
Je suis encore novice & j'aimerais donc que quelqu'un puisse me dire si mon script est correct :

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
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
#! /bin/bash
# nbfichiers.sh
 
if (( $# = 0 ));then
        read rep
        read ext
else
        rep=$1
        ext=$2
 
if [ ! -d "$rep" ];then
        exit 1
 
fr=0
for i in $rep
do
  if  [ -d "${rep[i]}" ]; then
          ((fr+=1))
 
fo=0
for i in $rep
do
  if  [ -f "${rep[i]}" ]; then
          ((fo+=1))
 
fe=0
fex=0
for i in $rep
do
  if  [ -d "${rep[i]}" ]; then
          if  [ -x "${rep[i]}" ] && [ "${$ext##*.}" = "$ext" ]; then
                  ((fex+=1))
          ((fe+=1))
 
echo $fo
echo $fe
echo $fex

Merci d'avance de votre aide.

[N_BaH]

Bonjour,

quand tu l'exécutes,
est-ce qu'il fonctionne sans erreurs ?
et, fait-il ce qu'il est censé faire ?

il est aberrant d'écrire un script sans savoir utiliser les commandes qui le composent.

[DienF]

Bonjour,
j'ai modifié un peu 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
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
#! /bin/bash
# nbfichiers.sh
 
if [ "$#" -eq 0 ];then
        read rep
        read ext
else
        rep=$1
        ext=$2
fi
if [ ! -d "$rep" ];then
        exit 1
fi
fr=0
for i in $rep
do
         if  [ -d "${rep[i]}" ];then
                   ((fr+=1))
         fi
done
fo=0
for i in $rep
do
         if  [ -f "${rep[i]}" ];then
                   ((fo+=1))
         fi
done
fe=0
fex=0
for i in $rep
do
       if  [ -d "${rep[i]}" ];then
                 if  [ -x "${rep[i]}" ] && [ "${$ext##*.}" = "$ext" ];then
                           ((fex+=1))
                 ((fe+=1))
                 fi
       fi
done
echo $fr
echo $fo
echo $fe
echo $fex

Lors que j'exécute mon script avec par exemple Documents et txt (en sachant que dans ce dossier j'ai 1 dossier vide, 1 fichier sans extension & 1 fichier avec l'extension txt) et le script me renvoie fr=1, fo=0, fe=1 & fex=1.
Je pense que le script est correct, dites-moi si il remplit bien les conditions.
Je me demande juste ce que sont les caractéristiques d'un ficier "ordinaire" (sans ext) ?
Merci

[N_BaH]

Non, ton script ne fonctionne pas.
les boucles n'itère sur aucun autre élément que celui indiqué : il n'y a aucun Développement de chemins !

un fichier "ordinaire" n'est ni un répertoire, ni un lien, ni... autre chose qu'un fichier "régulier" (selon les termes de help test).

de plus :
il faut faire toutes les incrémentations et donc tous les tests (sauf ceux qui vérifient les arguments du script, bien sûr) dans une seule boucle.
il faut compter les fichiers exécutables, qui font parties du nombre des fichiers portant une extension, et qui font eux-mêmes parties du nombre des fichiers ordinaires/réguliers. #inception

il faudrait ajouter un prompt aux commandes read pour que l'utilisateur sache quelle information il doit entrer.

je préfère la commande test aux crochets, que je trouve moche; ça a aussi le bénéfice de montrer que if utilise le retour de commandes, et que ce n'est pas une forme toute faite (if [...).

et si le script ne reçoit qu'un seul argument, que se passe-t-il ?