Script qui prend 3 arguments

Bonjour la communauté :)
Durant la période de confinement j'apprends le bash, je coince pour réliser ce script de l'exercice suivant... :(

Exercice 1
Écrire un script bash qui prend les 3 arguments suivants :
• un chemin de dossier : si celui-ci n'est pas valide (n'est pas un dossier, indice : tester avec -d) écrire un message d'erreur et demander à saisir un chemin jusqu'à ce qu'il soit valide.
• une extension de fichier : si celle-ci n'est pas correcte (ne suit pas une certaine expression régulière sachant que les extensions commencent par un caractère . suivi d'au moins une lettre ou un nombre, indice : tester avec =~) écrire un message d'erreur et demander à saisir une extension jusqu'à ce qu'elle soit valide.
• un nombre : si celui-ci n'est pas valide (ne suit pas une certaine expression régulière, indice : tester avec =~) écrire un message d'erreur et demander à saisir un nombre jusqu'à ce qu'il soit valide.
Ce script va compter le nombre de fichiers qui ont l'extension passée en argument (ou saisie) (indice : tester avec =~) dans le dossier passé en argument (ou saisi).
Si le nombre de fichiers est inférieur au nombre passé en argument (ou saisi) affiche le message "Vous n'avez pas assez de fichiers d'extension "X, X étant l'extension passée en argument ou saisie.
Si le nombre de fichiers est supérieur au nombre passé en argument (ou saisi) affiche le message "Vous avez trop de fichiers d'extension "X, X étant l'extension passée en argument ou saisie.
Si le nombre de fichiers est égal au nombre passé en argument (ou saisi) affiche le message "Vous le bon nombre de fichiers d'extension "X, X étant l'extension passée en argument ou saisie.
Bonus : si le nombre est correct, afficher l'ensemble des fichiers.

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 37 38 39 40 41 42 43 44 45 46

pwd echo 'Bonjour, vous vous trouvez dans le répertoire 'pwd' '   if [ "$#" -eq 0 ]; then echo "Saisir le répertoire" read -r dossier else dossier=$dossier fi   cd $dossier echo "Veuillez saisir le chemin de dossier à analyser" #echo au fonctionnement similaire au précédent read dossier #lecture du dossier dossier=( $dossier ) # il s'agit de la valeur entrée par le user qui sera stockée dans $dossier   nbFichier=0 #initialisation auto à valeur 0 pour que le script puisse compter les fichiers nbDossier=0 #initialisation auto à valeur 0 pour que le script puisse compter des dossiers nbExe=0 #initialisation auto à valeur 0 pour que le script puisse compter les executables   for d in ${dossier[@]} #boucle en place qui va s'appliquee pour chaque dossiers présents dans le répertoire sélectionner do if [ ! -d "$d" ] #condition, si le user entre une valeur inconnue (nom de dossier inconnu) then echo 'Erreur, le dossier '$d' n existe pas' #le user sera prévenu que le dossier n'existe pas! continue fi cd $d for file in * #pour chaque fichier(s) présent(s) dans l'intégralité du répertoire sélectionné do #nouvelle tâche à effectuée si condition précédente remplie if [ -d "$file" ] then nbDossier=$((nbDossier+1)) fi if [ -f "$file" ] then nbFichier=$((nbFichier+1)) fi if [ -x "$file" ] then nbExe=$((nbExe+1)) fi done done echo 'Nb fichier : '$nbFichier echo 'Nb sous-dossier: '$nbDossier echo 'Nb executable : '$nbExe

---

Déjà, à première lecture, ce code fait des opérations qui ne sont pas demandées dans l'exercice mais ne fait pas celles qui ne sont pas demandées.
Ensuite, où rencontres-tu exactement un problème ?

Je te propose d'y aller petit à petit et de commencer par l'exercice suivant:

Exercice 1
Écrire un script bash qui prend 1 argument :
• un chemin de dossier : si celui-ci n'est pas valide (n'est pas un dossier, indice : tester avec -d) écrire un message d'erreur et demander à saisir un chemin jusqu'à ce qu'il soit valide.

Puis tu peux nous poster ton code et, le cas échéant, on t'indiquera si ça peut être amélioré.

re,

ah c'est super sympa les gars :) merci d'avance!! rien que le fait de vous intéressé a mon soucis, ca me fait super plaisir :)

Citation:

---

Envoyé par jack-ft

Pour qu'on puisse t'aider, il faut que tu copies le code et non que tu fasses une copie d'écran (oops! grillé par N_BaH).

Je te propose d'y aller petit à petit et de commencer par l'exercice suivant:

Exercice 1
Écrire un script bash qui prend 1 argument :
• un chemin de dossier : si celui-ci n'est pas valide (n'est pas un dossier, indice : tester avec -d) écrire un message d'erreur et demander à saisir un chemin jusqu'à ce qu'il soit valide.

Puis tu peux nous poster ton code et, le cas échéant, on t'indiquera si ça peut être amélioré.

---

j'ai pensé à instaurer une boucle avec if, genre "si le chemin est valide", ca skip a la suite, sinon avec else "veuillez entrer le bon chemin"

c'est vraiment le fait de restrancrire mes idées qui me pose problemes... :(

(j'avais mal édité mon code sur le chat, sorry)

Citation:

---

Envoyé par Gowan974

j'ai pensé à instaurer une boucle avec if, genre "si le chemin est valide", ca skip a la suite, sinon avec else "veuillez entrer le bon chemin"

---

Le if ne fait pas de boucle, on fait des boucle avec while/until et cela fonctionne à peu près de la même façon que le if sans le else.
Par exemple, si je ne prends que l'exercice simple de jack-ft, décomposons le problème:
-script prends un argument => ici, je récupère le premier argument:

Code:

---

arg1=$1

---

Ensuite, cet argument doit être un répertoire valide ou j'envoie une erreur et je demande à l'utilisateur un nom de répertoire jusqu'à ce que celui-ci soit valide.

Donc techniquement, ici, pas besoin de controler si l'argument 1 existe, car implicitement, si celui-ci n'existe pas, cela ne sera pas un répertoire valide.

Et donc, on a juste un boucle à faire tant que l'argument 1 (ici, on parlera de la variable arg1, puisque je lui ai assigné la valeur de $1 ) n'est pas un répertoire, on envoie une erreur et on demande un nouveau chemin:

Code:

---

1 2 3 4 5 6

until [ -d "$arg1" ] do echo "ERREUR" >&2 echo -n "Chemin: " read arg1 done

---

Contrairement à while, until boucle tant que la commande (ici le test) est faux. si on avait du l'écrire avec while, il aurait fallu vérifier que arg1 ne soit pas un répertoire, donc comme ceci:

Code:

---

1 2 3 4 5 6

while [ ! -d "$arg1" ] do echo "ERREUR" >&2 echo -n "Chemin: " read arg1 done

---

A toi de voir pour la suite...

il n'est pas indiqué que les arguments ont une place définie.
présumons donc qu'ils ont une : nomScript dossier extension nombre.

Citation:

---

un chemin de dossier : si celui-ci n'est pas valide (n'est pas un dossier, indice : tester avec -d) écrire un message d'erreur et demander à saisir un chemin jusqu'à ce qu'il soit valide.

---

Code:

---

1 2 3 4 5

dossier=$1 until test -d "$dossier" do read -r -p 'entrez un dossier valide : ' dossier done

---

Citation:

---

une extension de fichier : si celle-ci n'est pas correcte (ne suit pas une certaine expression régulière sachant que les extensions commencent par un caractère . suivi d'au moins une lettre ou un nombre, indice : tester avec =~) écrire un message d'erreur et demander à saisir une extension jusqu'à ce qu'elle soit valide.

---

pareil : until, mais avec [[ et =~ est une regex (cf. man 7 regex).

Citation:

---

un nombre : si celui-ci n'est pas valide (ne suit pas une certaine expression régulière, indice : tester avec =~) écrire un message d'erreur et demander à saisir un nombre jusqu'à ce qu'il soit valide.

---

pareil, avec une regex qui permet d'identifier un nombre.

enfin, pour compter le nombre de fichiers, j'utilise un tableau dans lequel je mets un Développement des chemins (cf. man bash l'expression en italique)
mais on peut simplement itérer sur ce Développement des chemins, en incrémentant une variable...

si j'ai bien suivi, est ce que si je pars dans ce raisonnement, j'aurai tout mon excercice?

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15

cheminDossier=arg#1 extension=arg#2 nombre=arg#3   tant que cheminDossier n'est pas un chemin vers un dossier demander cheminDossier finTantQue   tantQue extension n'est pas regex_extension demander extension finTantQue   tantQue nombre n'est pas regex_nombre demander nombre finTantQue

---

je préfère until, que je trouve sous-employé.
et ici, plutôt que de tester la négation d'une condition, ça teste sa vérification.

Je peux comprendre, en résumé ca serait plus simple et plus agréable d'utiliser "until" c'est juste que je suis du genre m'éparpiller ca devient brouillon :( je souhaite utilisé While pour réussir à maitriser, ca parait idiot mais je me mêle les pinceaux si facilement :'(

Mais oui, j'ai bien réfléchi, tu as raison, ca sonne mieux de dire jusqu'a oui que tant que pas oui pour cet exo :)

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

pwd cheminDossier=$1 #Argument1 extension=$2 #Argument2 nombre=$3 #Argument3   echo "vous vous trouvez dans "$pwd" " until test -d $1 do read -r -p echo 'entrez un dossier valide : ' cheminDossier done   fi

---

est-ce que le début est bon? j'arrive pas à entrer le reste alors que l'idée est dans ma tête, ca me rend fouuu :')

Bonjour :coucou:

Citation:

---

est-ce que le début est bon?

---

Non. Tu ne sembles pas faire la différence entre les majuscules et les minuscules pour pwd. Et même si la variable d'environnement n'existait pas, comment aurais-tu fait pour afficher le résultat de la commande pwd dans la chaîne de caractères ?

Non. read n'affecte pas de variable.

Non. read a l'option -p mais pas de message.

Non. Y a-t-il la moindre chance que tu sortes de la boucle ?

Attention, tu utilises des commandes que tu ne connais pas sans te documenter sur leur usage :

Code:

---

$ help read | grep -- '^\s\+-p'

---

Code:

---

-p prompt affiche la chaîne PROMPT sans retour à la ligne final[...]

---

on va bientôt pouvoir te faire effacer ton disque dur. :aie:
:lol:

Citation:

---

Envoyé par Gowan974

est-ce que le début est bon?

---

Ç'a le mérite d'être un début!

Je t'invite à:

1. bien reregarder le le post de Disedorgue
2. ainsi que celui de N_BaH
3. puis à corriger ton script en conséquence
4. et, enfin et surtout, bien le tester plusieurs fois avec, comme premier argument:
   
   • plusieurs fois un bon répertoire différent à chaque fois
   • x plusieurs fois un répertoire qui n'existe pas, et avec plusieurs réponses incorrectes
   
   

PS: merci d'avoir remplacé la copie d'écran par du texte!

euh ok... ^^"... tanks quand meme les gens...

Ton algo est bon et résout les 3 premiers points, c'est à dire l'assignation validée de tes 3 paramètres utiles pour la suite de l'exercice.

Pour le premier paramètre, tu as déjà la réponse dans certains posts.

Pour les paramètres 2 et 3 c'est la même chose à la différence près que tu dois utiliser [[ arg#1 =~ ... ]] au lieu de [ -d ... ].

Et au vu des scripts que tu nous montres, je te conseille pleinement de te pencher sur la signification des quotes, guillemets et backquote et dans quels cas on utilise plus l'un que l'autre.

"Son" algo est un copier-coller d'une réponse sur un autre forum. :roll:

Citation:

---

Envoyé par Flodelarab

Non. read n'affecte pas de variable.

---

read effecte la variable REPLY :)

Code:

---

1 2 3 4 5 6 7 8 9 10 11

bashhar@kiss:~$ read mavariable covid-19 bashhar@kiss:~$ echo $mavariable covid-19 bashhar@kiss:~$ echo $REPLY   bashhar@kiss:~$ read covid-20 bashhar@kiss:~$ echo $REPLY covid-20

---