Un prog en bourne shell

Bonjour à tous,
Mon problème n'a pas trop sa place sur ce forum, car c'est un programme en script shell que je souhaite réaliser.
Pour me placer dans le contexte, je suis une formation à distance de programmeur et pour l'un des devoir que je dois rendre, il m'est demandé d'écrire en Bourne shell.

La Question qui m'est demandée est la suivante: Écrire un programme en Bourne shell qui rangera dans le répertoire courant les fichiers en fonction de leur extension. Le programme créera les dossiers qui porteront le nom de l’extension du fichier.
par ex:

Code:

---

1 2	$ls tuc1.c truc2.c cherche.pas dossier

---

on lance notre programme et celui-ci range tout bien

Code:

---

1 2	$ls c pas dossier

---

J'ai déjà creusé un peu la question, j'ai quelques pistes mais rien de bien concret. Si l'un d'entre vous pouvais me venir en aide, je lui serai très reconnaissant.

Merci

Les Quelques pistes que j'explore sont une boucle for, demande l'arborescence du répertoire courant. Le programme doit récupérer le nom des extension des fichiers, et créer à l'aide d'un bon vieux mkdir les répertoires portant le nom de l'extension de fichier. Ces-dernier qui devront être déplacé dans ceux-ci. grace a un mv *.ext ./ext
Voilà j'ai quelques commandes de base, Mais je ne vois pas trop comment syntaxer tout ça.
J'ai penser à un piège qui pourrait exister:
- Création du répertoire alors qu'il existe déjà et par conséquent, suppression du fichier qui était dedans.
Merci de votre aide



j'ai récupéré ce script que j'ai bidouillé et qui marche pas

Code:

---

1 2 3 4 5 6 7 8 9 10 11

for i in /home/user/Dropbox/EDUCATEL/PRATIQUE_UNIX\/dev4/; do test ${i##*.} = 'sql' then do_truc() mkdir 'sql' elif test ${i##*.} = 'gnagna' do_machin() mv *.gnagna 'sql' fi done

---

Bonjour,

Citation:

---

Création du répertoire alors qu'il existe déjà et par conséquent, suppression du fichier qui était dedans.

---

non, si le répertoire existe déjà, cela affichera une erreur, et cela ne fera rien.

Tu pourrais nous exposer clairement ton algorithme.

et, lis un tuto (notamment celui indiqué dans ma signature, pour commencer), au lieu d'essayer de repiquer des scripts récupérés à droite à gauche.

Merci pour ces liens, qui vont m'être fort instructifs je pense. Du coup en effet je reprends mon problème au départ et je pars d'un Algorithme. Je travaille dessus
La suite à venir...

J'ai avancé un peu sur mon problème,
j'ai trouvé la commande pour récupérer l'extension des fichiers dans le répertoire courant.

Code:

---

1 2	ls -C | sed 's/.*\.//g'

---

Mon hic maintenant, Je n'arrive pas à stocker le résultat obtenu dans une autre variable. Comment faire ? Faut-il un pipe ?
J'ai besoin des caractères de l'extension afin de créer un répertoire du même nom.
Merci à tous, bon début de semaine

Bonjour,

Je pense que lister les fichiers est un peu plus délicat dans le sens où ta commande va aussi lister les dossier que tu pourrais avoir créé avec une précédente instance.

Ensuite reste le problème des fichiers qui n'ont pas d'extension car cette information reste optionnelle.

Je te conseille donc plutôt la commande suivante :

Code:

---

1 2	ls -l | grep ^- | awk '{print $9}' | grep '\.' | sed 's/.*\.//g'

---

Traduction : je liste en format long pour avoir un max d'infos, le premier grep me permet de sélectionner uniquement les entrées commençant par '-' (fichiers conventionnels) puis le awk transforme le flux en n'extrayant que la colonne 9 (nom du fichier); là dessus, le second grep exclut tout fichier qui ne comporte pas de '.' dans son nom => pas d'extension et enfin, on applique ton sed pour ne récupérer que l'extension.

Après pour répondre plus directement à ta question : pour exploiter le résultat de cette commande, il faut l'affecter à une variable :

Code:

---

1 2	EXTENSIONS=$(ls -l | grep ^- | awk '{print $9}' | grep '\.' | sed 's/.*\.//g')

---

Dans les tutos fournis par N_BaH, tu trouveras ensuite comment faire un for in do done

@+

ON N'UTILISE PAS ls DANS LES SCRIPTS !!!

eh, mandraxx,
encore un cut et tu pouvais prétendre concourir pour la pire ligne de commande. grrr.
grep, sed, et awk peuvent tous les trois établir des correspondances avec un motif, donc un seul devrait suffire.
mais de toutes façons, on n'utilise pas ls dans les scripts, justement pour éviter ce genre de code, et si les fichiers ont des espaces dans leurs noms, le champ $9 ne renverra pas le nom complet.

donc, boucle for !

Citation:

---

Envoyé par N_BaH

ON N'UTILISE PAS ls DANS LES SCRIPTS !!!
...
de toutes façons, on n'utilise pas ls dans les scripts, justement pour éviter ce genre de code, et si les fichiers ont des espaces dans leurs noms, le champ $9 ne renverra pas le nom complet. donc, boucle for !

---

Oui, tu as tout à fait raison, mais je ne voulais pas décourager un débutant qui a trouvé un début de solution (c'est forgeant qu'on devient forgeron comme on dit) et juste mettre en évidence les difficultés que cela engendre pour conduire aux bonnes pratiques. :ccool:

Citation:

---

Envoyé par N_BaH

eh, mandraxx,
encore un cut et tu pouvais prétendre concourir pour la pire ligne de commande. grrr.
grep, sed, et awk peuvent tous les trois établir des correspondances avec un motif, donc un seul devrait suffire.

---

lol :pastaper:
J'avoue que j'ai eu la flemme de chercher le motif complexe qui fera tout ça d'un seul coup vu qu'on n'en a pas besoin si on n'utilise pas ls:calim2:

@+

Cool Merci Mandraxx
Je vais manger un bout et tester tout ça

Mais si c'est pas très correct d'utiliser la commande ls dans un script, qu’utilise-t-on ?

J'ai un peu avancé sur mon script, par contre il n'est pas opérationnel. rrrh

Code:

---

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

#!/bin/bash     EXTENTIONS=$(ls -C | sed 's/.*\.//g') # affecte le nom des extensions des fichier du dossier à la variable EXTENTIONS   echo $EXTENTIONS #optionnel if [ -f $EXTENTIONS ]; then #Si le dossier existe mv *.$EXTENTIONS $EXTENTIONS #alors déplacé le fichier d'extention *.ext dans dossier extle elif #sinon mkdir $EXTENTIONS #crée le dossier portant le nom de l'extension mv *.$EXTENTIONS $EXTENTIONS #déplacé le fichier d'extention *.ext dans dossier extle fi

---

Je n'ai pas reussi à intéger le code que tu m'a laissé Mandraxx. Par contre je m'en suis inspiré pour déclarer et affecter ma variable.EXTENSIONS. Mais ça marche pas :(

Citation:

---

Mais si c'est pas très correct d'utiliser la commande ls dans un script, qu’utilise-t-on ?

---

on utilise le Développement des chemins, dans ton cas avec une boucle for.
et, pour extraire les extensions, on utilise le Remplacement des paramètres, comme dans le script que tu voulais adapter.

attention -f, teste si un fichier est un fichier régulier, pas si un fichier est un dossier (regarde help test).

voila un scriptt qui est la correction de mon devoir, mais il ne fonctionne pas, pas ?

Code:

---

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

#/bin/sh for fichier in 'ls *.*' do if [-f $fichier –a –r $fichier] then suffixe='echo $fichier  cut –f2 –f' if! [-d $suffixe] then echo le repertoire $suffixe a ete cree mkdir $suffixe fi mv $fichier $suffixe echo le fichier $fichier a ete deplace dans $ (suffixe). fi done

---

quand je lance mon rpogramme le terminal me repond:
user@K73E:~/Dropbox/EDUCATEL/PRATIQUE_UNIX/dev4$ sh question3
[: 14: ls: unexpected operator

Merci de votre aide

Bonjour,

Je suppose que tu as recopié ce script qui a du t'être fourni imprimé sur papier non :mrgreen: ?

La substitution pour exécuter une commande ne se fait pas avec une quote mais avec le caractère produit par AltGr+7 (je ne me rappelle jamais de son petit nom :oops:).

Après, si je comprends bien ce code, je dirais qu'il faut un pipe à la place du &61679; (bizarre ça :?) et un -d'.' à la place du deuxième -f de cut.

Bon, dès que tu as quelque chose qui marche, il faut essayer d'aller plus loin pour progresser : suis les conseils de N_BaH et essaye de modifier ton script pour utiliser le Développement des chemins et là, tu auras un résultat robuste.

Astuce : la plupart des shells proposent une option de debug bash -x fichier.sh, tu peux l'utiliser pour suivre le comportement du code et ainsi comprendre comment réagit le système.

@+

Et il faut aussi laisser un espace apres le [ et un avant le ] dans les tests.

c'est une correction qui vient de où? parce que le for i in `ls ...`, c'est loin d'être une bonne habitude...

en effet il y a eu un problème dans le copier coller  correspond bien à un pipe "|"
J'ai placé des espaces entre les crochets du test.
Mais le programme ne se lance toujours pas, le terminal m’écrit:

Citation:

---

"user@K73E:~/Dropbox/EDUCATEL/PRATIQUE_UNIX/dev4$ sh question3

---

[: 14: ls: unexpected operator"

Code:

---

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

#/bin/sh for fichier in 'ls *.*' do if [ -f $fichier –a –r $fichier ] then suffixe='echo $fichier | cut –f2 –f' if ![-d $suffixe] then echo le repertoire $suffixe a ete cree mkdir $suffixe fi mv $fichier $suffixe echo le fichier $fichier a ete deplace dans $suffixe. fi done

---

Citation:

---

Envoyé par sk8trasher

en effet ya eut un probleme dans le copier coller  correspond bien à un pipe "|"
J'ai placé des espaces entre les crochets du test.
Mais le programme ne se lance toujours pas, le terminal m’écrit: "user@K73E:~/Dropbox/EDUCATEL/PRATIQUE_UNIX/dev4$ sh question3
[: 14: ls: unexpected operator"

Code:

---

1 2 3 4 5 6

#/bin/sh for fichier in 'ls *.*' do if [ -f $fichier –a –r $fichier ] then suffixe='echo $fichier | cut –f2 –f' ...

---

---

On te l'a dit: l'expansion des commandes c'est le backquotte et non la quotte simple !!!

Code:

---

1 2 3 4 5 6

#/bin/sh for fichier in `ls *.*` do if [ -f $fichier –a –r $fichier ] then suffixe=`echo $fichier | cut –f2 –f` ...

---

C'est ce qui arrive quand on recopie bêtement un source sans avoir appris la syntaxe.

Accessoirement tu auras tout avantage à utiliser la nouvelle écriture $(...) qui permet d'imbriquer plus facilement des expansions dans d'autres expansions

Code:

---

1 2 3 4 5 6

#/bin/sh for fichier in $(ls *.*) do if [ -f $fichier –a –r $fichier ] then suffixe=$(echo $fichier | cut –f2 –f) ...

---

PS: je ne vois pas trop ce qu'est sensé faire le cut -f2 -f (sans rien derrière)...

Citation:

---

Envoyé par sk8trasher

J'ai placé des espaces entre les crochets du test.

Code:

---

1 2 3 4 5 6

#/bin/sh for ... do if [ -f $fichier –a –r $fichier ] then ... if ![-d $suffixe]

---

---

Ouais, ça se voit :roll: !!!

Bon, allez, je le tape parce que ça m'énerve trop ce "corrigé" totalement foiré. On ne sait pas qui des deux (entre le prof ou l'élève) est le pire...

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23

#/bin/sh # Boucle sur tous les fichiers ayant un "." dans leur nom for fichier in *.* do # Si le fichier est de type régulier et qu'on peut le lire if [ -f "$fichier" -a -r "$fichier" ] then # Récupération de la chaine contenue après le "." suffixe=`echo "$fichier" | cut -f2 -d.`   # Si le dossier correspondant à cette chaine n'existe pas if [ ! -d "$suffixe" ] then # On le crée (putain si par malheur un truc ayant le même nom existe... mais je ne suis pas là non plus pour tout faire !!!) mkdir "$suffixe" echo "le repertoire $suffixe a été créé" fi   # On déplace le fichier dans le dossier (tant pis si ce n'est pas un dossier) mv "$fichier" "$suffixe" echo "le fichier $fichier a été déplacé dans $suffixe." fi done

---

Voilà. Tu n'as plus qu'à faire ctrl-c+ctrl-v. Eventuellement apprendre tes leçons serait un plus mais c'est toi qui voit...

Citation:

---

Envoyé par N_BaH

on utilise le Développement des chemins, dans ton cas avec une boucle for.

---

On utilise "find -maxdepth 1" :mouarf:

Code:

---

apprendre tes leçons serait un plus

---

bof, bof. Si les leçons sont au niveau du corrigé... :cry:

Merci pour ces corrections, je n'arrive pas à lancer le programme pour autant.En revanche cette source me parait plus juste que la version que j'ai posté.
Quand je lance le programme celui ci me répond: " unexpected operator" pour chaque fichier à déplacer. Il ne crée pas non plus les répertoire.

Je suis preneur si vous avez des sites références sur la programmation en Bourne shell.
En faite je pense pas que ce soit la correction, c'est juste une aide, voilà pourquoi des syntaxes peuvent vous paraître aberrante.

Je continue à chercher une solutions. affaire à suivre...

Citation:

---

Envoyé par sk8trasher

Merci pour ces corrections, je n'arrive pas à lancer le programme pour autant.
Quand je lance le programme celui ci me répond: " unexpected operator" pour chaque fichier à déplacer. Il ne crée pas non plus les répertoire.

---

Une erreur due au copier/coller. J'ai tapé cet exemple en utilisant certaines lignes (qui me semblaient bonnes) du tien alors qu'il était probablement lui-même issu d'un traitement de texte style Word. Certains caractères ont été mal écris. Ils semblent bons visuellement mais ne le sont pas.
Reprends chaque ligne "test" et réécris les tirets (-a -r etc...). Tu les réécris réellement => tu les effaces et tu les remets !!!

De mon coté j'ai repassé mon code dans mon post précédent. Tu peux de nouveau le copier/coller, cette fois il marchera.

Citation:

---

Envoyé par sk8trasher

En revanche cette source me parait plus juste que la version que j'ai posté.

---

C'est exactement la même à ces détails près que
1) il y a des commentaires (tu verras que ce n'est jamais du temps perdu que d'en mettre)
2) la syntaxe est respectée (espaces quand il faut, etc)
3) les commandes utilisées sont écrites avec les bons arguments (-d et non -f etc)

Citation:

---

Envoyé par sk8trasher

Je suis preneur si vous avez des sites références sur la programmation en Bourne shell.

---

http://fr.lang.free.fr/cours/Shell_v2.0.pdf

Citation:

---

Envoyé par Sve@r

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23

#/bin/sh # Boucle sur tous les fichiers ayant un "." dans leur nom for fichier in *.* do # Si le fichier est de type régulier et qu'on peut le lire if [ -f "$fichier" -a -r "$fichier" ] then # Récupération de la chaine contenue après le "." suffixe=`echo "$fichier" | cut -f2 -d.` # Si le dossier correspondant à cette chaine n'existe pas if [ ! -d "$suffixe" ] then # On le crée (putain si par malheur un truc ayant le même nom existe... mais je ne suis pas là non plus pour tout faire !!!) mkdir "$suffixe" echo "le repertoire $suffixe a été créé" fi # On déplace le fichier dans le dossier (tant pis si ce n'est pas un dossier) mv "$fichier" "$suffixe" echo "le fichier $fichier a été déplacé dans $suffixe." fi done

---

Voilà. Tu n'as plus qu'à faire ctrl-c+ctrl-v. Eventuellement apprendre tes leçons serait un plus mais c'est toi qui voit...

---

Juste pour le fun... ;)
Petite remarque 1: un fichier sans extension (sans point), comme "foo", n'est pas trié.
Petite remarque 2: un fichier contenant deux points consécutifs, comme "foo..txt", ou finissant par un point, comme "foo.", provoque deux erreurs (mkdir et mv) et n'est pas classé.
Petite remarque 3: un fichier contenant deux points non consécutifs, comme "foo.tar.gz", est classé dans "tar" (et non dans "tar.gz" ni "gz")
:mouarf:
de la difficulté à traiter tous les cas...
)jack(

Citation:

---

Envoyé par jack-ft

Juste pour le fun... ;)
Petite remarque 1: un fichier sans extension (sans point), comme "foo", n'est pas trié.
Petite remarque 2: un fichier contenant deux points consécutifs, comme "foo..txt", ou finissant par un point, comme "foo.", provoque deux erreurs (mkdir et mv) et n'est pas classé.
Petite remarque 3: un fichier contenant deux points non consécutifs, comme "foo.tar.gz", est classé dans "tar" (et non dans "tar.gz" ni "gz")
:mouarf:
de la difficulté à traiter tous les cas...
)jack(

---

Bien vu. :ccool:
Bon, régler le 3 est le plus facile: on remplace cut -f2 -d. par cut -f2- -d.
Le point 1, lui, ne se pose pas car l'énoncé parle d'un script fait pour traiter les fichiers ayant une extension.
Quand au point 2 bah pour le point terminal je pense qu'on peut faire un test sur la chaine "$suffixe" qui, dans ce cas, sera nulle. Et s'il y a 2 (ou plus) points d'affilée je pense qu'en les fusionnant en un seul via sed placé en amont du cut ça devrait le faire mais bon, j'ai plus trop envie de m'y coller ce soir :aie:

Cool Merci, le script réalise exactement ce que je souhaitais. Je vais le scruter de plus près histoire de comprendre toutes ces subtilités.