Pourquoi ne comptabilise-t-il rien ?

Bonjour à toutes et à tous j'ai un petit problème dans ce programme :

Code:

---

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

#!/bin/bash     #Déclaration des variables chemin=$HOME/menu j=0 i=0   #Verrification si on passe bien un parametre if [ "$#" -eq 1 ] then #Verrification que $1 est bien un repetoire if [ -d "$chemin/$1" ] then # Si il l'est on regarde de quel type sont les fichier du repertoire for fichier in "$chemin/$1/*.*" do if [ -d "$chemin/$1/$fichier"] then i=`expr $i + 1` fi if [ -f "$chemin/$1/$fichier"] then j=`expr $j + 1` fi done echo "nombre de repetoire $i" echo "nombre de fichier $j" else echo "$1 n'est pas un repertoire" fi else echo "nombre de parametre incorrecte" fi

---

C'est lorsque je lui passe le repertoire livre qui contient deux fichiers et qui se trouve dans menu il m'indique qu'il y a 0 fichiers et 0 repertoire ? !

J'ai pu remarqué en faisant un test qu'il ne parcourt qu'une fois la boucle for alors qu'il devrait le faire 2 fois vu qu'il y a deux fichiers.

Mais je ne vois pas où est l'erreur dans l'écriture de la boucle ? !

Amicalement crodilus

ligne 19 il y a une apostrophe de trop ... et ce test est sensé faire quoi donc ?

Salut,

Citation:

---

Envoyé par mvvvv

ligne 19 il y a une apostrophe de trop ... et ce test est sensé faire quoi donc ?

---

Sans compter que if [ "$chemin/$1/$chemin'"] si on ne met pas la bonne variable ($fichier et non deux fois $chemin) ça ne va pas le faire non plus :?

Et quand bien même tu aurais mis la bonne, peux-tu nous dire ce que sont censées faire ces deux tests ??? Tes tests ne testent rien, si ce n'est qu'ils renvoient vrai systématiquement, que ce soit un fichier ou un répertoire :(

Citation:

---

Envoyé par mvvvv

ligne 19 il y a une apostrophe de trop ... et ce test est sensé faire quoi donc ?

---

Désolé ce sont des fautes de retranscription car dans le script initiale elle n'y est pas

Je viens de corriger le script dans le premier message

Code:

---

for fichier in "/$chemin/$1/*.*"

---

Pourtant cette écriture veut bien dire pour tout les fichiers du repertoire $1 ?

Bonjour,

les astérisques ne sont pas développés entre guillemets, donc, à moins d'avoir un fichiers nommé littéralement *.*, le compte de fichier ne décollera pas ;)

expr n'est pas une commande bash.
Même un shell strictement POSIX sait faire des évaluations arithmétiques par lui-même :

Code:

---

var=$(( $var + 1))

---

en bash :

Code:

---

((var++))

---

ce sont les quotes dans ton for le problème :

Code:

---

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

#!/bin/bash   #au cas ou il n'y a pas de fichier (bash only) shopt -s nullglob   #Déclaration des variables chemin=$HOME/menu j=0 i=0   #Verrification si on passe bien un parametre if [ "$#" -eq 1 ] then #Verrification que $1 est bien un repetoire if [ -d "$chemin/$1" ] then # Si il l'est on regarde de quel type sont les fichier du repertoire for fichier in $chemin/$1/* do # pour tester c'est utile echo $fichier if [ -d "$fichier" ] then i=`expr $i + 1` elif [ -f "$fichier" ] then j=`expr $j + 1` fi done echo "nombre de repetoire $i" echo "nombre de fichier $j" else echo "$1 n'est pas un repertoire" fi else echo "nombre de parametre incorrecte" fi

---

Alors effectivement en retirant les *

Code:

---

""

---

cela fonctionne

Il passe bien dans la boucle 2 fois mais il ne comptabilise toujours pas les nombres de répertoire et de fichiers..

J'ai remarqué que si je mets

Code:

---

for fichier $chemin/$1/*.*

---

ça ne marche pas,pourquoi ? Cette ecriture diffère telle de celle-ci

Code:

---

for fichier $chemin/$1/*

---

Je travaille sur putty (pour information) et j'ai déjà utilisé des expr et cela marche >.< !

*.* ne marche que si ton nom de fichier contient un point.

Ta variable $fichier contient le nom absolu de ton fichier. Elle seule doit apparaître dans tes tests (voir mon code).

Citation:

---

j'ai déjà utilisé des expr et cela marche >.< !

---

je n'ai pas dit que ça ne pouvait pas fonctionner.
j'ai dit qu'il est inutile d'appeler un programme externe, quand le shell peut le faire au moins aussi bien.

Citation:

---

Envoyé par mvvvv

*.* ne marche que si ton nom de fichier contient un point.

Ta variable $fichier contient le nom absolu de ton fichier. Elle seule doit apparaître dans tes tests (voir mon code).

---

Citation:

---

Envoyé par N_BaH

je n'ai pas dit que ça ne pouvait pas fonctionner.
j'ai dit qu'il est inutile d'appeler un programme externe, quand le shell peut le faire au moins aussi bien.

---

Ok j'ai compris ;) ! Et tout fonctionne !

Merci pour votre aide !

Amicalement crodilus.