Remplacement de sous-chaines

Bonjour,
voilà mon problème :
J'effectue de la substitution de sous-chaines de façon dynamique. L'idée de base est la suivante :

Code:

---

1 2 3 4 5

String content = ....;   for(int id = 0 ; id < 10 ; id++) { content = content.replaceAll("Une expression faisant intervenir id", "une autre expression faisant intervenir id"); }

---

Ce qui d'après moi n'est pas super efficace puisque j'ai une nouvelle instance de String créée à chaque tour de boucle (sachant que les ID sont en fait récupérés en base et que donc il ne s'agit pas d'une boucle "simple")
J'ai tenté ma chance avec du StringBuffer et les matchers mais le problème est qu'il faut resetter le matcher à chaque fois vu que je modifie le StringBuffer, ce qui est encore plus long que l'utilisation d'une chaine (cela dit je n'ai pas de données au niveau de l'impact mémoire
Pour info la version StringBuffer a l'aspect suivant :

Code:

---

1 2 3 4 5 6 7 8 9

for(int id = 0 ; id < 10 ; id++) { String imageName = id + ".gif"; Pattern p = Pattern.compile("regex faisant intervenir ID"); Matcher m = p.matcher(sb); while(m.find()) { sb.replace(m.start(), m.end(), "<img src=\"" + imageName + '"'); m.reset(); } }

---

Elle produit bien le même résultat mais est légèrement plus lente...

Salut,


A quoi ressemble ton "regex faisant intervenir ID" ? Il y a peut-être moyen de faire plus simple...

a++

à ça (de tête et sans despécialiser) : "<img src="[^"]servlet\?idImage=" + id + '"'
Mes URL sont variables(avecou sans http, avec ou sans le nom d'hote, etc), donc je vois pas plus simple

Puisque id semble correspondre à un numérique tu peux facilement uniformiser ton pattern afin de ne faire qu'un seul passage. Tu sera quand même obligé d'utiliser un StringBuffer temporaire en plus de ta chaine mais cela te garantit de bonne performance pour un peu de mémoire en plus.

Par exemple :

Code:

---

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

public static String transform(String str) { Pattern p = Pattern.compile("<img src=\"[^\"]*servlet\\?idImage=([0-9]+)\""); Matcher m = p.matcher(str); if (m.find()) { // ceci permet de ne pas créer de StringBuffer temporaire // si la chaine ne comporte aucun élément correspondant StringBuffer buffer = new StringBuffer(); do { String imageName = m.group(1) + ".gif"; m.appendReplacement(buffer, "<img src=\"" + imageName + '"'); } while(m.find()); m.appendTail(buffer); str = buffer.toString(); } return str; }

---

Que tu utilises comme cela :

Code:

---

1 2	String s = // ta chaine s = transform(s);

---

a++

PS : si ton id n'est pas vraiment un numérique tu dois surement pouvoir adapter la regexp ;)

id est bien un numérique
et de tête (toujours), ce que tu proposes est exactement l'implémentation de replaceAll dans le JDK 5.0 (de tête hein, je l'ai pas sous les yeux)...
Je testerai je te dirai si c'est plus rapide...
Edit : en utilisant un seul stringbuffer je peux peut être diminuer l'impact mémoire :

Code:

---

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

public static String transform(String str) { Pattern p = Pattern.compile("<img src=\"[^\"]*servlet\\?idImage=([0-9]+)\""); Matcher m = p.matcher(str); StringBuffer buffer = new StringBuffer(str.length); if (m.find()) { do { String imageName = m.group(1) + ".gif"; m.appendReplacement(buffer, "<img src=\"" + imageName + '"'); } while(m.find()); m.appendTail(buffer); str = buffer.toString(); buffer.clear(); } return str; }

---

Et tu peux utiliser StringBuilder, il est à StringBuffer ce que ArrayList est à Vector :)

Sauf que appendReplacement et appendTail n'a pas été modifié pour fonctionner avec un AbstractStringBuilder et ne fonctionne qu'avec les StringBuffer (ce qui est un peu stupide)

hello,
t'as trouvé la soluce? parce que je suis interressé.

@++

Pour l'instant je suis sur autre chose mais normalement normalement le derneir code donné devrait fonctionner...