Discussion :

[N_BaH]

Bonjour,

non, c'est pas «ok» pour la boucle for (si les noms de fichiers comportent un caractère composant l'IFS ) :

Code :

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

find ... | while read f; do...; done

ou, vu que tu es en bash

Code :

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

1
2
3
4
5
6
7
8
9
#!/bin/bash
 
shopt -s nullglob globstar
for f in ./**/*.c
do
   sed -i 's/\x0d$//' "$f"
   ./ccomment  <"$f" 1<>"$f"; #heu, ça, j'suis pas sûr !?
   sed -i 's/$/\x0d/' "$f"
done

[disedorgue]

Code :

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

1
2
 
   ./ccomment  <"$f" 1<>"$f"; #heu, ça, j'suis pas sûr !?

Moi non plus, car techniquement, on ne lit pas le dit fichier sur 1 mais sur 0.
Pour moi, si le tampon d'entrée est plus petit que ton fichier, celui-ci sera écrasé avant la lecture complète, mais je me trompe peut-être.
Je conseillerais de passer par un fichier temporaire:

Code :

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

    ./ccomment  <"$f" >"$f.tmp"; mv "$f.tmp" "$f"

ou

Code :

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

    ./ccomment  <"$f" >"$f.tmp" && mv "$f.tmp" "$f"

Dans ce cas, mv ... ne sera éxécuté que si la commande précédente se fait sans erreur.

[_iri_]

Merci pour le blindage supplémentaire au sujet des noms de fichiers.

Ensuite, pour la ligne controversée, il était 3h du mat et je ne voyais plus très clair ! Je n'ai pas trop réfléchi là-dessus, j'ai vu que ça fonctionnait sur différents HelloWorld....

Pour le fichier temporaire, oui, j'avais prévu la seconde méthode. Ce qui m'importait à ce moment là était le contenu de la sortie.

Merci à tous pour vos aides précieuses. Je ne sais pas si je retiendrais tout pour la prochaine fois mais j'ai appris.

Je passe la discussion en résolu. Si toutefois vous aviez des ajouts à faire, n'hésitez pas !

[zipe31]

Pour ceux que ça intéressent, avec l'option shopt -s nullglob globstar du bash, la commande "sed" peut aussi faire dans la récursivité :

Code :

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

sed -i 'syntaxe' ./**/*.c

Testé et approuvé

[N_BaH]

nullglob est alors inutile.

[Flodelarab]

Bon à savoir tout ça.


Pour le descripteur 1<>fichier en lecture-écriture simultanée, je doute. Ça reste un flux.
Quand on lit une ligne, elle appartient déjà au passé.
Quand on écrit, on écrase une phrase que l'on n'a pas encore lue.
Je n'ai pas encore vu de cas qui nécessitait ce genre de canal.

[disedorgue]

Interressant la récursivité du sed, on doit donc pouvoir faire ça sans boucle en un seul sed qui transforme le CRLF, appelle ccomment (via la commande e du sed) et remet en format CRLF.

Je ne peux pas tester dans l'immédiat...

[_iri_]

Je devais convertir, je l'ai donc fait avec ce scipt-ci (que je comprends !) :

Code :

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

1
2
3
4
5
6
7
8
shopt -s nullglob globstar
for f in ./**/*.c ./**/*.h
do
    sed -i 's/\x0d$//' "$f"
    ./ccomment  <"$f" >"$f.tmp" && mv "$f.tmp" "$f"
    sed -i 's/$/\x0d/' "$f"
    echo "$f"
done

Les autres formes proposées sont encore floues.
Vos discussions m'ont incité à retourner lire la littérature de sed.

[N_BaH]

Code BASH :

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

for f in ./**/*.{c,h}

[disedorgue]

Ou tout simplement:

Code :

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

1
2
shopt -s globstar
sed -i "s/\\x0d\$//;H;\${g;s/\\(.*\\)/echo '\1' | .\\/ccomment/e;s/[^\\x0d]\\x0a/\\x0d\\x0a/g}" ./**/*.{c,h}

[zipe31]

Bon je confirme aussi la récursivité pour sed avec l'option globstar de bash

On active l'option ;

Code :

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

$ shopt -s globstar

Arborescence des fichiers ;

Code :

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

1
2
3
4
5
6
7
8
9
10
$ ls -lR ./**/*.c
-rw-r--r-- 1 jp jp 440 août  22 22:39 ./f.c
-rw-r--r-- 1 jp jp 440 août  22 22:39 ./rep1/f.c
-rw-r--r-- 1 jp jp 440 août  22 22:39 ./rep1/sous-repA/f.c
-rw-r--r-- 1 jp jp 440 août  22 22:39 ./rep1/sous-repB/f.c
-rw-r--r-- 1 jp jp 440 août  22 22:39 ./rep1/sous-repC/f.c
-rw-r--r-- 1 jp jp 440 août  22 22:39 ./rep/f.c
-rw-r--r-- 1 jp jp 440 août  22 22:39 ./rep/sous-repA/f.c
-rw-r--r-- 1 jp jp 440 août  22 22:39 ./rep/sous-repB/f.c
-rw-r--r-- 1 jp jp 440 août  22 22:39 ./rep/sous-repC/f.c

On vérifie le format des fichiers (sauce Windows) :

Code :

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

1
2
3
4
5
6
7
8
9
10
$ file ./**/*.c
./f.c:                ASCII C++ program text, with CRLF line terminators
./rep1/f.c:           ASCII C++ program text, with CRLF line terminators
./rep1/sous-repA/f.c: ASCII C++ program text, with CRLF line terminators
./rep1/sous-repB/f.c: ASCII C++ program text, with CRLF line terminators
./rep1/sous-repC/f.c: ASCII C++ program text, with CRLF line terminators
./rep/f.c:            ASCII C++ program text, with CRLF line terminators
./rep/sous-repA/f.c:  ASCII C++ program text, with CRLF line terminators
./rep/sous-repB/f.c:  ASCII C++ program text, with CRLF line terminators
./rep/sous-repC/f.c:  ASCII C++ program text, with CRLF line terminators

La commande (avec sauvegarde) :

Code :

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

$ sed -i.bak "s#\\x0d\$##;s#.*#/bin/echo -E '&' | ccomment#e;s#\$#\\x0d#" ./**/*.c

L'arborescence de contrôle pour la sauvegarde :

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
$ ls -lR ./**/*.c*
-rw-r--r-- 1 jp jp 440 août  22 22:39 ./f.c
-rw-r--r-- 1 jp jp 422 août  22 22:29 ./f.c.bak
-rw-r--r-- 1 jp jp 440 août  22 22:39 ./rep1/f.c
-rw-r--r-- 1 jp jp 422 août  22 22:35 ./rep1/f.c.bak
-rw-r--r-- 1 jp jp 440 août  22 22:39 ./rep1/sous-repA/f.c
-rw-r--r-- 1 jp jp 422 août  22 22:35 ./rep1/sous-repA/f.c.bak
-rw-r--r-- 1 jp jp 440 août  22 22:39 ./rep1/sous-repB/f.c
-rw-r--r-- 1 jp jp 422 août  22 22:35 ./rep1/sous-repB/f.c.bak
-rw-r--r-- 1 jp jp 440 août  22 22:39 ./rep1/sous-repC/f.c
-rw-r--r-- 1 jp jp 422 août  22 22:35 ./rep1/sous-repC/f.c.bak
-rw-r--r-- 1 jp jp 440 août  22 22:39 ./rep/f.c
-rw-r--r-- 1 jp jp 422 août  22 22:35 ./rep/f.c.bak
-rw-r--r-- 1 jp jp 440 août  22 22:39 ./rep/sous-repA/f.c
-rw-r--r-- 1 jp jp 422 août  22 22:35 ./rep/sous-repA/f.c.bak
-rw-r--r-- 1 jp jp 440 août  22 22:39 ./rep/sous-repB/f.c
-rw-r--r-- 1 jp jp 422 août  22 22:35 ./rep/sous-repB/f.c.bak
-rw-r--r-- 1 jp jp 440 août  22 22:39 ./rep/sous-repC/f.c
-rw-r--r-- 1 jp jp 422 août  22 22:35 ./rep/sous-repC/f.c.bak

Le contrôle du format :

Code :

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

1
2
3
4
5
6
7
8
9
10
$ file ./**/*.c
./f.c:                ASCII C program text, with CRLF line terminators
./rep1/f.c:           ASCII C program text, with CRLF line terminators
./rep1/sous-repA/f.c: ASCII C program text, with CRLF line terminators
./rep1/sous-repB/f.c: ASCII C program text, with CRLF line terminators
./rep1/sous-repC/f.c: ASCII C program text, with CRLF line terminators
./rep/f.c:            ASCII C program text, with CRLF line terminators
./rep/sous-repA/f.c:  ASCII C program text, with CRLF line terminators
./rep/sous-repB/f.c:  ASCII C program text, with CRLF line terminators
./rep/sous-repC/f.c:  ASCII C program text, with CRLF line terminators

On vérifie un fichier dans l'arborescence :

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
$ cat rep1/sous-repA/f.c
/* 
  Ceci est un programme de test
  pour remplacer les commentaires C++
  en commentaires C
*/
/* Fonction main du programme */
/* C est aussi la fonction de test */
int main ()
{
    printf("Hello world!\n");  /* ecrit Hello World! dans la console */
    printf("/////Hello world!\n//////");  /* ecrit Hello World! dans la console */
    return 0;  /* valeur retournee par le programme // 0 : succes */
}
/* Fin du code */

Et on contrôle sa sauvegarde :

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
$ cat rep1/sous-repA/f.c.bak 
/* 
  Ceci est un programme de test
  pour remplacer les commentaires C++
  en commentaires C
*/
// Fonction main du programme
// C est aussi la fonction de test
int main ()
{
    printf("Hello world!\n");  // ecrit Hello World! dans la console
    printf("/////Hello world!\n//////");  // ecrit Hello World! dans la console
    return 0;  // valeur retournee par le programme // 0 : succes
}
// Fin du code

Enjoy

[disedorgue]

En voyant ta solution, je me dis que je me suis vraiment compliqué la vie en chargeant dabord la totalité du fichier dans sed avant le passage à ccomment

Par contre, en faisant mes tests avec ce que j'avais sous la main (cygwin), je conseil de remettre en format dos que si c'est utile donc comme je le fais dans ma proposition (s/[^\x0d]$/\x0d/), car ayant compilé ccomment.c avec un gcc sous windows, celui-ci me rajoute lui même le caractere \x0d à la fin de chaque ligne.

[_iri_]

Il y a des trucs vachement puissant qu'un développeur devrait connaître pour se simplifier la vie .... ! En tout cas, je vous ai fait réfléchir !