Discussion :

[Beuss]

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 :

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

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 :

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

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...

[adiGuba]

Salut,


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

a++

[Beuss]

à ç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

[adiGuba]

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 :

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

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 :

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

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

[Beuss]

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 :

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

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;
    }

[®om]

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