Discussion :

[HurtMarley]

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 :

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

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

Merci

[adiGuba]

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 :

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

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 :

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

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 :

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

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

Plus d'info : 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++

[HurtMarley]

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