Discussion :

[xelm06]

Bonjour,

Je cherche à récupérer des références de normes dans un texte afin de les encadrer de balise HTML <a>, je tente de le faire avec une regex.

Les normes sont de la forme "NF EN xxx", où xxx est le numéro de la norme.

Voici mon code soumis à un exemple:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
String txt = "blablabla NF EN 127 blablabla NF EN 148 blablabla";
Pattern patternNormes = Pattern.compile(".*((NF EN \\d*)).*");
Matcher matcherNormes = patternNormes.matcher(txt);
 
if(matcherNormes.find()){
    matcherNormes.matches();
    for(int i=1;i<matcherNormes.groupCount();i++){
        System.out.println(matcherNormes.group(i));
        txt = txt.replaceAll(matcherNormes.group(i),"<a>"+matcherNormes.group(i)+"</a>");
    }
    System.out.println(txt);
}

A priori la regex que j'utilise n'est pas suffisante puisqu'elle s'arrête à la première occurrence (en partant de la fin) correspondante à mon motif. Car au final seule la seconde norme de l'exemple se retrouve encadrée par les balises HTML <a> (sortie de console) :

Code :

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

1
2
NF EN 148 
blablabla NF EN 127 blablabla <a>NF EN 148</a> blablabla

Y aurait-il un pro des regex qui pourrait me filer un tuyaux pour récupérer les autres occurrences (NF EN 127 dans mon exemple) ?

Par avance, merci pour votre aide

[thelvin]

Hello,

puisque tu utilises find(), les .* au début et à la fin sont inutiles (et c'est à cause d'eux que tu passes directement à la dernières référence). Et l'appel à matches() va tout casser, il annule les infos stockées par find(). Il faut l'enlever, il ne servait à rien de toute façon.

Bien sûr si tu as plusieurs références à trouver, il ne faut pas faire un if sur find(), mais bien entendu une boucle qui trouvera toutes les références au lieu d'une.

[xelm06]

En effet cela simplifie le code, par contre je dois conserver le find(), car dans le cas où le texte n'a pas de référence il me faut éviter une "java.lang.IllegalStateException: No match found".

Voici mon code modifié ci-dessous :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
String txt = "blablabla NF EN 127 blablabla NF EN 148 blablabla";
Pattern patternNormes = Pattern.compile("(NF EN \\d*)");
Matcher matcherNormes = patternNormes.matcher(txt);
 
if(matcherNormes.find()){
    for(int i=0 ; i<matcherNormes.groupCount() ; i++){
        System.out.println(matcherNormes.group(i));
        txt = txt.replaceAll(matcherNormes.group(i),"<a>"+matcherNormes.group(i)+"</a>");
    }
    System.out.println(txt);
}

Cela dit, je ne récupère que la première occurrence, car voici la sortie console :

Code :

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

1
2
NF EN 127
blablabla <a>NF EN 127</a> blablabla NF EN 148 blablabla

On y est presque, une petite idée pour récupérer la suivante ?

[joel.drigo]

Salut,

Comme te l'a dit @Thelvin, il te faut faire une boucle : find() va trouver une occurence qui matche, et il faut refaire un autre find() pour trouver la suivante.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
String txt = "blablabla NF EN 127 blablabla NF EN 148 blablabla";
		Pattern patternNormes = Pattern.compile("(NF EN \\d*)");
		Matcher matcherNormes = patternNormes.matcher(txt);
 
		while(matcherNormes.find()){
		    for(int i=0 ; i<matcherNormes.groupCount() ; i++){
		        System.out.println(matcherNormes.group(i));
		        txt = txt.replaceAll(matcherNormes.group(i),"<a>"+matcherNormes.group(i)+"</a>");
		    } 
		}
 
		System.out.println(txt);

Ta boucle sur les groupes est inutile, puisqu'il n'y a qu'un groupe dans l'expression régulière. Et celui-ci ne sert à rien ici. Ceci fait le job :

Code :

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

1
2
3
4
5
6
7
8
9
10
String txt = "blablabla NF EN 127 blablabla NF EN 148 blablabla";
Pattern patternNormes = Pattern.compile("NF EN \\d*");
Matcher matcherNormes = patternNormes.matcher(txt);
 
while(matcherNormes.find()){
    System.out.println(matcherNormes.group());
    txt = txt.replaceAll(matcherNormes.group(),"<a>"+matcherNormes.group()+"</a>");
}
 
System.out.println(txt);

Tu pourrais procéder plus efficacement comme ça :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
String txt = "blablabla NF EN 127 blablabla NF EN 148 blablabla";
Pattern patternNormes = Pattern.compile("NF EN \\d*");
Matcher matcherNormes = patternNormes.matcher(txt);
 
StringBuffer sb=new StringBuffer();
while(matcherNormes.find()){
    matcherNormes.appendReplacement(sb, "<a>"+matcherNormes.group()+"</a>");
}
matcherNormes.appendTail(sb);
txt=sb.toString();
 
System.out.println(txt);

Voire plus simplement :

Code :

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

1
2
3
4
5
6
7
String txt = "blablabla NF EN 127 blablabla NF EN 148 blablabla";
Pattern patternNormes = Pattern.compile("NF EN \\d*");
Matcher matcherNormes = patternNormes.matcher(txt);
 
txt = matcherNormes.replaceAll("<a>$0</a>");
 
System.out.println(txt);

Voire encore plus simplement :

Code :

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

1
2
3
4
5
String txt = "blablabla NF EN 127 blablabla NF EN 148 blablabla";
 
txt = txt.replaceAll("NF EN \\d*", "<a>$0</a>");
 
System.out.println(txt);

[xelm06]

Merci beaucoup cela fonctionne parfaitement !