[Regex] Expressions régulières: sous groupe

Version imprimable

bonjour,
j'ai le code suivant: (s est une variable string, contenu est une string)
Code:

1 2 3 4 5 6 7 8 Pattern regex = Pattern.compile("(\\W("+s+")\\W)"); Matcher match = regex.matcher(contenu); boolean b = match.find(); if (b == true) { String matchFound = match.group(); String matchColored =""+matchFound+""; contenu = contenu.replaceAll(matchFound, matchColored); }
ça fonctionne très bien sauf si j'ai une variable s par exemple slt et que dans mon texte, j'ai slt(bonjour). J'obtiens
Code:

1 2 3 Autre erreur : java.util.regex.PatternSyntaxException: Unclosed group near index 5 slt(
donc j'ai voulu faire:
Code:

1 2 3 4 5 6 7 8 Pattern regex = Pattern.compile("(\\W("+s+")\\W)"); Matcher match = regex.matcher(contenu); boolean b = match.find(); if (b == true) { String matchFound = match.group(2); String matchColored =""+matchFound+""; contenu = contenu.replaceAll(matchFound, matchColored); }
et ça marche sauf que maintenant, si j'ai une variable ct et que dans mon texte, j'ai action par exemple, alors ça va me repérer le mot action car il contient ct. or moi, je ne veux que ct. C'est pour ça que j'avais mis les \\W. Donc comment faire
- pour ne surligner que le mot exact et pas un mot qui en inclus un autre
- ne pas tenir compte des caractères non mot d'avant et d'après ?
Merci

05/07/2005, 13h23
barbu0055

Dans le deuxieme exemple moi il ne trouve pas action.

slt,
en fait qd je dis qu'il trouve action (je me suis peut etre mal exprimée) c'est que les lettres ct de action vont etre surlignées aussi. Or moi je ne veux pas car le seul mot du texte que je veux surligner est ct, et pas les lettres ct inclus dans un autre mot. Si tu fais le code suivant, tu auras les lettres ct de action, surlignées.
Code:

1 2 3 4 5 6 7 8 9 10 11 12 s="ct"; contenu ="action ct ect"; Pattern regex = Pattern.compile("(\\W("+s+")\\W)"); Matcher match = regex.matcher(contenu); boolean b = match.find(); if (b == true) { String matchFound = match.group(2); String matchColored =""+matchFound+""; contenu = contenu.replaceAll(matchFound, matchColored); System.out.println("contenu="+contenu); } zone_texte.setText(contenu);
avec le println, tu obtiens

Code:

contenu=action ct ect

05/07/2005, 14h17
barbu0055

J avais bien compris ce que tu voulais. Mais en fait j'avais fait un autre test qui marchait. Parce qu'en fait dans ton code ce qui pause problème ce n'est pas ton expression régulière mais plutôt le replaceAll.

C'est d'ailleurs également lui qui t'envoie l'exception du premier cas.

Voila comment tu peux faire, c un peu plus lourd mais je ne c pas si tu as le choix :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
String contenu = "action ct ect";
    	String s = "ct";
    	Pattern regex = Pattern.compile("(\\W("+s+")\\W)");
    	Matcher match = regex.matcher(contenu);
    	boolean b = match.find();
    	System.out.println(contenu);
 
    	while (b) {
    	         String matchFound = match.group(2);
    	         int i = match.start();
    	         String matchColored ="<font size=\"4\" style bgcolor=\"#33CCFF\">"+matchFound+"</font>";
    	         contenu = contenu.substring(0,i)+
				 contenu.substring(i).replaceFirst(matchFound, matchColored);
    	         b = match.find();
    	}
 
        System.out.println(contenu);

Je suis peut etre aller un peu vite.
Ce que g fait ne suffit pas, ca marche dans le cas ou il ne le trouve qu'une fois :) ce qui est un peu trop restrictif je pense.
Mais ce coup ci ca doit tre bon :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
String contenu = "action ct e ct a";
    	String s = "ct";
    	Pattern regex = Pattern.compile("(\\W("+s+")\\W)");
    	Matcher match = regex.matcher(contenu);
    	System.out.println(contenu);
    	String res = "";
    	int oldMatch=0;
    	int newMatch=0;
    	while (match.find()) {
    	         String matchFound = match.group(2);
    	         newMatch = match.start();
    	         res += contenu.substring(oldMatch,newMatch);
    	         oldMatch = match.end();
    	         res += "<font size=\"4\" style bgcolor=\"#33CCFF\">"+
				 	contenu.substring(newMatch,oldMatch)+"</font>";
    	}
    	res += contenu.substring(oldMatch,contenu.length());
        System.out.println(res);

j'ai testé ton code et il fonctionne bien dans l'exemple donné. Le probleme :arf: c'est que je n'avais pas tout mis dans le premier message. En fait, ma regex et mon replaceAll sont dans une boucle while. Cette boucle permet d'obtenir la variable s. Donc avec les substrings, j'ai des indexoutofboundsexception car à chaque tour de while, je me retrouve avec un oldmatch qui est plus grand que le new match. :koi:
j'ai donc essayé de réinitialiser les old et new match à 0 à chaque nouveau tour de la boucle while. Mais de toute façon il y a le pb de res car à chaque fois on fait res = res+... donc je me retrouve avec un tres tres grand texte, ou on a plusieurs fois la meme chose. Cette nuit j'ai dormi :zzz: que 3 heures, c'est peut etre pour ça que je vois pas comment faire :roll: Je mets qd meme le code:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 while(rs.next()){ oldMatch=0; newMatch=0; String s = rs.getString("name"); Pattern regex = Pattern.compile("(\\W("+s+")\\W)"); Matcher match = regex.matcher(contenu); while (match.find()) { String matchFound = match.group(2); newMatch = match.start(); res += contenu.substring(oldMatch,newMatch); oldMatch = match.end(); res += ""+ contenu.substring(newMatch,oldMatch)+""; } } res += contenu.substring(oldMatch,contenu.length()); zone_texte.setText(res); }

Si g bien compris tu as plusieurs recherche successive à faire dans le même corps de texte.
Dans ce cas à la fain de chaque boucle while tu ajoute

Code:

contenu = res;

c tout, ce qui donne:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 while(rs.next()){ res = ""; oldMatch=0; newMatch=0; String s = rs.getString("name"); Pattern regex = Pattern.compile("(\\W("+s+")\\W)"); Matcher match = regex.matcher(contenu); while (match.find()) { String matchFound = match.group(2); newMatch = match.start(); res += contenu.substring(oldMatch,newMatch); oldMatch = match.end(); res += ""+ contenu.substring(newMatch,oldMatch)+""; } } res += contenu.substring(oldMatch,contenu.length()); contenu = res; zone_texte.setText(res); }
Mais ca risque d'etre dangereux, parce que les recherches suivantes vont se faire sur un coprs de texte déjà modifié. Pour éviter cela tu pourrais mettre toutes tes recherche dans un seule expresion régulière en liant les mot avec un "|", à savoir faire un disjonction de tous les mots :

Code:

\w(mot1)\w|\w(mot2)\w ...

ca se fait facilement en faisant une simple boucle sur ton rs.next() en mettant les rs.getString("name") bout à bout.

slt,
dsl g pas pu répondre + tot.
j'ai essayé de faire toutes les recherches en une. ça fonctionne partiellement. J'obtiens bien tous les mots que je voulais surligner et je n'ai plus le pb des parentheses (qd j'avais "slt(" et que je cherchais "slt"). Mais apparemment, il y a une analyse du texte caracteres par caracteres. Qd je fais un println des old match et new match, ça affiche:
old=0, new =0,
old=0, new=1,
old=1, new=2,
old=2, new=3...
et si je fais un println de res, j'ai:
Code:

1 2 ... been...
je voulais aussi que soient surlignés seulement les mots de la base, sans les caracteres non mots (\W) qui précèdent et qui suivent. Donc avt ds mon code, je mettais match.group(2). Mais maintenant que j'ai une expression qui regroupe tout, je ne sais pas comment faire. Quels chiffres mettre car il y a plusieurs groupes à récupérer? et je ne sais pas lesquels. Je suppose que ce sera un multiple de i. Donc pour savoir ceux qui m'interessent, j'ai testé:
Code:

1 2 3 4 5 6 while (match.find()) { for(int i=1; i<match.groupCount()+1;i++){ if(match.group(i)!=null){ System.out.println("i= "+i+" match= "+match.group(i)); } }
mais ça ne rentre jamais dans le if car tous les match.group st null et je ne sais pas pourquoi.

voici le code en entier:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 if (cb == e.getItem()){ ResultSet rs = statement.executeQuery("select * from ..."); while(rs.next()){ String s = rs.getString("name"); tout = tout +"(\\W"+"("+ s +")"+"\\W)"+"|"; } Pattern regex = Pattern.compile(tout); Matcher match = regex.matcher(contenu); while (match.find()) { String matchFound = match.group(); //je ne sais pas quel nb mettre newMatch = match.start(); System.out.println("old = "+oldMatch); System.out.println("new = "+newMatch); res += contenu.substring(oldMatch,newMatch); oldMatch = match.end(); res += ""+contenu.substring(newMatch,oldMatch)+""; } res += contenu.substring(oldMatch,contenu.length()); contenu = res; zone_texte.setText(res); }
il y a aussi ds le text pane: html> head> /head> body> qui apparaissent au début du texte et /body> /html> qui apparaissent à la fin du texte car les balises ne st pas bien ouvertes ou fermées et je suppose que ça a un rapport avec le fait que il y ait des font caracteres par caractères.

voila je sais pas si je suis tres clair (hesite pas à le dire ;)
et encore merci pour toute ton aide depuis le début.

Code:

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
27
28
if (cb == e.getItem()){
ResultSet rs = statement.executeQuery("select * from ...");
 
      String tout = "(\\W(";
      while(rs.next()){
         String s = rs.getString("name");
         if (rs.isLast())
               tout += "s|";
         else
               tout += "s";
      }
      tout += ")\\W)";
 
      Pattern regex = Pattern.compile(tout);
      Matcher match = regex.matcher(contenu);
      while (match.find()) {
         String matchFound = match.group(); //je ne sais pas quel nb mettre
         newMatch = match.start()+1;
         System.out.println("old = "+oldMatch);
         System.out.println("new = "+newMatch);
         res += contenu.substring(oldMatch,newMatch);
         oldMatch = match.end()-1;
         res += "<font size=\"4\" style bgcolor=\"#33CCFF\">"+contenu.substring(newMatch,oldMatch)+"</font>";
      }
      res += contenu.substring(oldMatch,contenu.length());
      contenu = res;
      zone_texte.setText(res);
}

G modifié un peu l expression régulière pour n'avoir qu'un sous-groupe parce que ta boucle "for" risque d'etre assez lourde, et g supprimer les caractère autour du mot trouvé.
Par contre dans mes tests il ne fait pas du caractère par caractère, alors ca vient peut etre de ton resultset qui contient un espace ou qqchose comme ca (imprime l'expression pour le savoir).
Voila mon test :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
int newMatch = 0;
    	int oldMatch = 0;
    	String res = "";
    	String contenu = "-ct--af--ct(bonjout)   ct     af     vbg";
    	Pattern regex = Pattern.compile("(\\W(ct|af)\\W)");
        Matcher match = regex.matcher(contenu);
        while (match.find()) {
           String matchFound = match.group(); //je ne sais pas quel nb mettre
           newMatch = match.start()+1;
           System.out.println("old = "+oldMatch);
           System.out.println("new = "+newMatch);
           res += contenu.substring(oldMatch,newMatch);
           oldMatch = match.end()-1;
           res += "<font size=\"4\" style bgcolor=\"#33CCFF\">"+contenu.substring(newMatch,oldMatch)+"</font>";
        }
        res += contenu.substring(oldMatch,contenu.length());
        contenu = res;
        System.out.println(res);

07/07/2005, 12h48
billynirvana

C'est un alignement de chaine de caractères que tu veux faire???

style:

ALPHONSE
TELEPHONE

dis moi car j'ai déjà fait cet algorithme, et il marche :P
07/07/2005, 14h08
debdev

:lahola:
bah je sais plus koi dire, à part :merci:
encore une fois, tu as résolu mon problème :yaisse2:
t'es un boss en java :bravo: et notamment sur les regex :ccool:
alors sincèrement merci pour le temps que tu passes pour m'aider.
a bientot :coucou: (car j'en doute pas une seconde, je vais bientot bloquer sur un autre probleme :roll: )
bonne apres midi
07/07/2005, 14h10
debdev

slt billynirvana, je viens de voir ta réponse. non, ce n'était pas un alignement que je faisais, mais merci d'avoir proposé ton aide. barbu0055 a trouvé une solution.