Discussion :

[marinette08]

Bonjour,

J'ai un exercice de script shell à faire mais je n'y arrive pas. Pouvez-vous me donner des pistes pour que je puisse le commencer s'il vous plait
Merci

Voici le sujet:

Rédiger un script shell référencé images qui admet en argument un nombre quelconque de
répertoires et qui affiche l'espace disque total occupé, dans chaque répertoire, par les
fichiers de type image au format « point jpg ».
L'espace disque total, occupé par toutes les images de tous les répertoires traités, est
également affiché (dans l'exemple ci-dessus, 41 020 + 1 008 = 42 028).
Exemple :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
$ ./images $HOME/Images/Noel_2009 pluf $HOME/Images/Saint_Maulvis /root
/home/raoul/Images/Noel_2009 contient 41020 ko d'images jpg
pluf : inexistant ou non répertoire
/home/raoul/Images/Saint_Maulvis/ contient 1008 ko d'images jpg
/root : lecture non autorisée
-- Total : 42028 ko
$

On n'oubliera d'opérer toutes les vérifications adéquates et de programmer selon l'esprit
Unix.
Le format d'affichage de l'exemple cidessus devra être respecté.
Indication.
L'utilisation de la commande du peut s'avérer utile. Consulter le manuel. En effet, traiter des
tailles de fichiers avec la commande ls est compliqué car les champs affichés par cette
commande ne sont pas séparés par un espace, mais plusieurs, d'où des difficultés à les
isoler avec le filtre cut.

[BufferBob]

salut,

sans dire que la formulation de l’énoncé est complètement évidente, il est tout de même pas trop mal détaillé, suffisamment pour qu'on comprenne le but de l'exercice sans ambiguïté

un bon début serait donc de répondre à la question : quel est le but de l'exercice concrètement, qu'est-ce qu'il faut faire ? (reformulation)

l'exemple est parlant lui aussi :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

$ ./images $HOME/Images/Noel_2009 pluf $HOME/Images/Saint_Maulvis /root

en plus des directives de l'exercice, on sait donc que le script doit s’appeler images et être invocable directement

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
pluf : inexistant ou non répertoire
/root : lecture non autorisée

on sait également que les erreurs sont traitées, et on sait comment on doit les afficher s'il y en a

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
/home/raoul/Images/Noel_2009 contient 41020 ko d'images jpg
/home/raoul/Images/Saint_Maulvis/ contient 1008 ko d'images jpg

et quand il n'y a pas d'erreur, on sait également comment on doit afficher les résultats

reste à décortiquer le processus, se représenter le bon algorithme, la structure générale du code

• le script vient d'être lancé, le but est de travailler sur les arguments passés en ligne de commande, est-ce qu'on sait les récupérer ?
• assez intuitivement, on va traiter chaque paramètre/répertoire séquentiellement, les uns après les autres, ça implique donc une boucle d'une manière ou d'une autre
• on est dans la boucle, on commence à traiter le premier répertoire... qu'est-ce qu'on en fait ?

arrivé ici, tu dois avoir au moins un bout de code, même non-fonctionnel, sur lequel on pourra éventuellement t'aider plus

[marinette08]

Salut, désolé j'ai beaucoup de difficulté a faire des scripts Shell

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
#! /bin/bash
 
If  (($#==0))
Then  écho " au moins une argument requis " 
  Exit 1 
Fi 
 
For home
Do..
If ..
Then écho "$HOME inexistant ou non répertoire "

[N_BaH]

faut pas dire ça : c'est un constat d'échec improductif.
il est préférable de dire : « je voudrais progresser »

[marinette08]

Oui d'accord mais Je veux progresser vraiment, j'ai juste besoin d'aide

[N_BaH]

je te renvoie à la réponse de BufferBob.

j'ai l'habitude de conseiller d'observer comment on résoud le problème en le faisant "à la main", sans chercher à automatiser, puis de dessiner un arbre de décision éventuellement.

on ne peut malheureusement pas apprendre le maniement du shell à ta place.
on ne fera pas ton travail, mais on corrigera volontiers tes erreurs.

lis les tutos dans les liens qui figurent dans ma signature, et la FAQ shell

[Jipété]

Oui d'accord mais Je veux progresser vraiment, j'ai juste besoin d'aide

Si tu devais faire l'exercice avec un papier, un crayon et une calculatrice, tu ferais comment ? Raconte...

sans dire que la formulation de l’énoncé est complètement évidente, il est tout de même pas trop mal détaillé, suffisamment pour qu'on comprenne le but de l'exercice sans ambiguïté

Euh, y a au moins une ligne mal écrite et donc très ambigüe (nouvelle orthographe) :

On n'oubliera d'opérer toutes les vérifications adéquates et de programmer selon l'esprit Unix.

Au choix :
- On n'oubliera pas d'opérer toutes les vérifications adéquates et de programmer selon l'esprit Unix.
- On oubliera d'opérer toutes les vérifications adéquates et de programmer selon l'esprit Unix.
Bon, honnêtement, la deuxième option je n'y crois pas du tout, mais il est toujours bon, dans ce genre de contexte, de ne laisser aucune place à l'improvisation, à la supposition et à l'estimation.

[marinette08]

J'ai essayer de faire quelque chose
Pouvez me dire si je suis dans la bonne voie ou pas et si il y a des pb me les préciser

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
#!/bin/bash
 
If  (( $#     ))
Then echo "$rep n'existe pas"
Fi
Exit 1 
 
 
For $rep in $(du $1/*.jpg)
Do
  If 
Then echo "$rep contient ... ko d'images jpg"
Else écho " $rep: lecture non autorisé "
Fi 
Donne
Exit 0

[N_BaH]

le B-A BA de la programmation, c'est de disposer de l'environnement adapté au langage utilisé.
or, manifestement, tu n'utilises pas de ligne de commande (shell) pour tester tes commandes !

ce n'est pas une bonne habitude de boucler avec for sur la sortie d'une commande.
si tu avais testé du sur un fichier, tu saurais que sa sortie est composée de plusieurs mots (au sens de la ligne de commande).

tu dois lire un tuto pour débutant avant d'envisager de coder quoi que ce soit !

[disedorgue]

Bonjour,

Sous unix, contrairement à windows, il faut respecter la casse: 2 commandes avec le même nom mais l'une est en minuscule et l'autre en majuscule sont 2 commandes distinctes sous unix.

Après, pour t'aider un minima, pour ton exercice, les commandes suivantes devrait être suffisantes (Il n'y a pas de majuscules dedans!!!):

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

if;then;else;fi

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

test

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

exit

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

while;do;done

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

shift

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

du

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

cut

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

for varx in;do;done

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

((vary+=varx))

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

printf ou echo

Les if avec test pour vérifier le nombre de paramètres et pour vérifier l'existante et l'acces à un des répertoires fournit en paramètre
Le while avec le shift pour boucler sur les paramètres
La boucle for avec le du + le cut pour calculer la taille (l'operation est faite si tu utilises la varx dans le for et vary comme variable qui aura le résultat à la fin de la boucle for)
et le printf ou echo pour afficher les différents messages (dans les if, ou après la boucle for).
Un petit détail: ne pas oublier de réinitialiser la vary à 0 avant d'entrée dans la boucle for.
Il te faudra aussi une autre variable qui portera la somme des différents vary à chaque sortie de boucle for, pour avoir le total (tout répertoire confondu) à la fin de la boucle while

[jack-ft]

Euh... une question bête: pour boucler sur les arguments, y a-t-il un problème à faire un for sur "$@"?

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
#!/bin/bash
 
for arg in "$@"; do
    echo "'${arg}'"
done

Ç'a l'air assez robuste pour supporter les cas limite, comme les chaines vides, les espaces, les retours et l'absence d'arguments:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
$ ./test_args_2.sh "" "a b" "$(echo -e "a\nb")"
''
'a b'
'a
b'

[disedorgue]

Oui, c'est juste par habitude de passer par un shift que j'ai proposé le while+shift et par doute aussi sur la gestion par boucle for

[N_BaH]

des... doutes ?

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
$ (set "" "a b" "$(echo -e "a\nb")"; for i do echo ">$i<"; done)
><
>a b<
>a
b<

si omis, "$@" est implicite; c'est donc la manière de boucler sur les arguments positionnels.

[disedorgue]

Et oui, les mauvaises habitudes perdurent

Les perfs sont largement meilleures via le for au lieu du while+shift (bon, en fait c'est le shift 1 qui est lent quand on a par exemple 65000 paramètres )

En fait, pour faire dans les bons usages si on a un nombre très important de paramètres, serait de les traiter dans une boucle for et lorsque l'on n'en a plus besoin de faire un shift du nombre de paramètres histoire de "libérer la mémoire"