Remplacer // par /* */

Bonjour,

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

Code:

---

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

---

ou, vu que tu es en bash

Code:

---

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

---

Citation:

---

Envoyé par N_BaH

Code:

---

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:

---

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

---

ou

Code:

---

./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.

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 !

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:

---

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

---

Testé et approuvé ;)

:ccool: nullglob est alors inutile.

Bon à savoir tout ça. :ccool:


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.

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...

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

Code:

---

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.

Code:

---

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

---

Ou tout simplement:

Code:

---

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

---

:aie::aie::aie:

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

On active l'option ;

Code:

---

$ shopt -s globstar

---

Arborescence des fichiers ;

Code:

---

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:

---

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:

---

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

---

L'arborescence de contrôle pour la sauvegarde :

Code:

---

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:

---

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:

---

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:

---

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 ;)

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 :ccool:

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.

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 ! :ccool: