Discussion :

[Rodrigue]

Bonjour,

Je souhaiterais détecter les commentaires multilignes (/* ... */) contenus dans des fichiers C++: .h, .cpp.

Pour ce faire, j'utilise boost::regex. J'ai déjà trouvé une expression rationnelle (ou régulière comme vous souhaitez ) qui détecte les commentaires contenus sur une seule ligne.

Code :

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

"/\\*([[:punct:]]|[[:alnum:]]|[[:blank:]])*\\*/"

Mais pas moyen de lui faire reconnaître les multilignes, j'ai bien essayé de remplacer [:blank:] par [:space:], voir même :

Code :

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

"/\\*.*\\*/"

mais le problème c'est qu'il ne s'arrête pas au premier */ mais au dernier.

Voici l'exemple simple sur lequel je travaille:

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
20
21
22
23
24
25
26
/* Check to see if a file is a PNG file using png_sig_cmp().  png_sig_cmp()
 * returns zero if the image is a PNG and nonzero if it isn't a PNG.
 *
 * The function check_if_png() shown here, but not used, returns nonzero (true)
 * if the file can be opened and is a PNG, 0 (false) otherwise.
 *
 * ...
 */
#define PNG_BYTES_TO_CHECK 4
int check_if_png(char *file_name, FILE **fp)
{
   char buf[PNG_BYTES_TO_CHECK];
 
   /* Open the prospective PNG file. */
   if ((*fp = fopen(file_name, "rb")) == NULL)
      return 0;
 
   /* Read in some of the signature bytes */
   if (fread(buf, 1, PNG_BYTES_TO_CHECK, *fp) != PNG_BYTES_TO_CHECK)
      return 0;
 
   /* Compare the first PNG_BYTES_TO_CHECK bytes of the signature.
      Return nonzero (true) if they match */
 
   return(!png_sig_cmp(buf, (png_size_t)0, PNG_BYTES_TO_CHECK));
}

Le résultat est donc:

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
20
21
22
23
/* Check to see if a file is a PNG file using png_sig_cmp().  png_sig_cmp()
 * returns zero if the image is a PNG and nonzero if it isn't a PNG.
 *
 * The function check_if_png() shown here, but not used, returns nonzero (true)
 * if the file can be opened and is a PNG, 0 (false) otherwise.
 *
 * ...
 */
#define PNG_BYTES_TO_CHECK 4
int check_if_png(char *file_name, FILE **fp)
{
   char buf[PNG_BYTES_TO_CHECK];
 
   /* Open the prospective PNG file. */
   if ((*fp = fopen(file_name, "rb")) == NULL)
      return 0;
 
   /* Read in some of the signature bytes */
   if (fread(buf, 1, PNG_BYTES_TO_CHECK, *fp) != PNG_BYTES_TO_CHECK)
      return 0;
 
   /* Compare the first PNG_BYTES_TO_CHECK bytes of the signature.
      Return nonzero (true) if they match */

J'utilise comme option de recherche:

Code :

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

1
2
3
boost::match_flag_type flags = boost::match_default;
 
   while(boost::regex_search(start, end, what, expression, flags))

Merci pour votre aide!

[screetch]

le regexp matche la plus grande occurrence de ton expression

or

Code :

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

/*  */ */

a deux patterns qui "matchent" la regexp :

Code :

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

/*  */

et

Code :

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

/*  */ */

donc il prend la plus longue occurrence!

je ne sais pas si il y a un moyen de dire a boost de ne pas poursuivre la recherche lorsqu'une serie match le pattern (quel langage barbare desole) mais sinon il faut que tu reconnaisses le pattern qui commence par /*, ne contient pas */ et se termine par */. c'est chiant car */ tient sur deux caracteres, c'est donc plus difficile a reconnaitre

[Mat007]

Salut,

Essaye par exemple :

Code :

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

"/\\*(~[\\*])*\\*(~[\\*/](~[\\*])*\\*|\\*)*/"

Source : The JavaCC FAQ, la note 4.
J'ai transcrit en regex standard dans une chaîne C viteuf donc c'est peut-être pas exactement ça, en tous cas c'est beaucoup plus compliqué que ce que tu proposes pour obtenir une expression correcte.

MAT.

[Rodrigue]

Merci pour le lien
J'ai essayé et ça ne fonctionne pas
JAVACC:

Code :

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

"/*" (~["*"])* "*" (~["*","/"] (~["*"])* "*" | "*")* "/"

Si je traduis (je pense que ~ est l'opérateur d'exclusion ^

Code :

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

"/\\*(^[*])*\*(^[*/](^[*])*\*|\*)*/"

[Mat007]

Les * dans doivent être échappés par \\ en tous cas.
Après peut-être que ^\\* suffit et qu'il n'y a pas besoin de ^[\\*], je ne sais plus

MAT.

[Rodrigue]

J'ai trouvé !

Code :

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

"/\\*([^*]|(\\*+([^*/])))*\\*+/"

Cette expression prend n'importe quel type de commentaire

Maintenant le problème, ce sont les commentaires //.
J'ai encore le même problème quand je fais:

Code :

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

//.*$

Ce n'est pas logique comme écriture...