regExp ième occurence du masque

Bonjour,
J'ai une chaîne de caractère comportant certains signes de ponctuation et je voudrai la reformater de la façon suivante : mots espaces Signe de ponctuation espaces mots -> mots Signe espace mots

Bonjour , comment ça va ? Bien ; et vous?
->
Bonjour, comment ça va? Bien; et vous?

Le problème est que mon code me renvoie
Bonjour, comment ça va, Bien, et vous?
Toutes les occurrences de mon masque sont remplacées par le premier signe trouvé : ici la virgule.

Si vous voyez où est mon erreur…

Code:

---

1 2 3

p = Pattern.compile("([ ])+([.!?;,])([ ])+"); m = p.matcher(texte); if(m.find()){texte = m.replaceAll(m.group(2)+" ");}

---

Merci

Salut,


Tu utilises mal les méthodes find() et group(). Ces méthodes permettent de traiter chacun des éléments trouvés, et doivent être utilisé dans une boucle...

Ici lorsque tu fais le find(), cela recherche la première occurence correspondante à ta regexp (la première virgule) puis tu effectues donc un replaceAll() avec la chaine ", " "en dur"...

Pour remplacer toutes les occurences en utilisant la valeur du group(), tu dois utiliser une boucle, par exemple :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15

String texte = "Bonjour , comment ça va ? Bien ; et vous?";   Pattern p = Pattern.compile("([ ])+([.!?;,])([ ])+"); Matcher m = p.matcher(texte); if (m.find()) { StringBuffer sb = new StringBuffer(); do { // Pour chaque regexp trouvé on effectue un remplacement : m.appendReplacement(sb, m.group(2) + " "); } while (m.find()); // Puis on rajoute la fin du texte (eventuellement) m.appendTail(sb);   texte = sb.toString(); }

---

Mais bon en général il suffit d'utiliser directement replaceAll() et les indices via $1, $2, etc...

Code:

---

1 2 3 4 5

String texte = "Bonjour , comment ça va ? Bien ; et vous?";   Pattern p = Pattern.compile("([ ])+([.!?;,])([ ])+"); Matcher m = p.matcher(texte); texte = m.replaceAll("$2 ");

---

Ou plus directement :

Code:

---

1 2 3

String texte = "Bonjour , comment ça va ? Bien ; et vous?";   texte = texte.replaceAll("([ ])+([.!?;,])([ ])+", "$2 ");

---

Plus d'info : :faq: Comment remplacer une sous-chaîne en la reportant dans la chaîne de remplacement ?


Au passage tu peux simplifier ton expression en évitant de définir un groupe pour les espaces : " +([.!?;,]) +" (et du coup il faut utiliser $1 ou m.group(1) pour obtenir le signe de ponctuation).


a++

Merci a toi, je n'ai pas saisi que find et group marchaient comme ça.