Discussion :

[Rakken]

Bonjour,

Voilà, je me posais une question qui me semble bête, mais je voudrais savoir quelle est la méthode "propre" pour faire un substring à partir d'un StringBuffer, mais en récupérant un StringBuffer et pas un String.
Voila un code simplifié du problème :

Code :

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

1
2
3
4
5
6
 
StringBuffer ret = new StringBuffer();
for (maClasse elemCourant: listeElement) {
   ret.append(elemCourant.getLibelle()).append(", ");
}
// ret.substring(0, ret.length() - 2);

A l'endoit du commentaire, je voudrais supprimer la dernière virgule, mais dans la mesure ou j'ai d'autres traitements à faire sur mon StringBuffer, je voudrai garder un StringBuffer et pas passer en String.
J'imagine que je pourrais recréer un StringBuffer à partir de mon String obtenu avec substring, mais je trouve ça relativement idiot en fait.
Bref, quelle est la méthode élégante ?

Merci d'avance !

[MigouW]

Pourquoi ne pas utiliser les methodes delete(int start,int end) ou deleteCharAt(int location) ?

[bulbo]

Alors pour supprimer la dernière virgule .. cela serait pas plus élégant de tester pour ne pas la mettre ? Et surement plus efficace.

Maintenant pour le StringBuffer (Note que sun recommande StringBuilder) je vois plusieurs approche.

delete(start, end)
deleteCharAt(index)

et enfin subSequence qui te retourne un CharSequence que tu peux re-utiliser ensuite.

Bulbo

[bhamp0]

Code :

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

1
2
3
4
5
6
StringBuffer ret = new StringBuffer();
String sep = "";
for (maClasse elemCourant: listeElement) {
   ret.append(sep).append(elemCourant.getLibelle());
   sep = ", ";
}

[Rakken]

Delete... Et en plus, j'ai parcouru la doc.
On va dire qu'on est vendredi et que je suis fatigué.

En tout cas merci !
(me disait bien aussi que c'était pas possible que ca existe pas)

[Rakken]

bhamp0 > Oui, c'est ce que j'avais fait à la place (enfin, une variante avec un booleen), mais la question me turlupinait quand même.

Autre question du coup, c'est plus rentable de faire une assignation d'un String dans une boucle ou de supprimer ce qui a été concaténé en trop ?

[bhamp0]

bhamp0 > Oui, c'est ce que j'avais fait à la place (enfin, une variante avec un booleen), mais la question me turlupinait quand même.

La façon que je donne permet d'éviter d'utiliser un boolean + un test ; ça "automatise" le traitement.

Autre question du coup, c'est plus rentable de faire une assignation d'un String dans une boucle ou de supprimer ce qui a été concaténé en trop ?

Rentable dans quel sens ? Mémoire ou temps ?
Ca risque d'être kif kif, mais j'aime pas l'idée de devoir traiter des cas particuliers.
La solution à coup de "length - 2" t'oblige à vérifier si ta liste n'était pas vide au départ (un "setLength(-2)" doit provoquer une erreur, je pense). Donc la ~meilleure~ solution à mon sens est celle qui te permet de ne pas avoir à gérer de cas particuliers à coup de "if then else".

[adiGuba]

Alors pour supprimer la dernière virgule .. cela serait pas plus élégant de tester pour ne pas la mettre ? Et surement plus efficace.

Pas si sûr...


Le fait de supprimer les caractères en trop est à la fois simple et efficace (à noter que dans le cas des derniers caractères, on peut se contenter d'un setLength()) :

Code :

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

1
2
3
4
	for (String s : list) {
		sb.append(s).append(", ");
	}
	sb.setLength(sb.length()-2);

Il y a plusieurs moyen de ne pas mettre le dernier séparateur, mais cela implique plusieurs vérifications préalable pas forcément aussi lisible ni clair :

Code :

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

1
2
3
4
5
6
	if (!list.isEmpty()) {
		sb.append(list.get(0));
		for (String s : list.subList(1, list.size())) {
			sb.append(", ").append(s);
		}
	}

Le tout pour un gain qui doit être vraiment minime...


a++

PS : Sinon +1 pour le StringBuilder depuis Java 5

[bhamp0]

Pas si sûr...
(...)

Le premier code que tu donnes oblige aussi des vérifications préalables. Le code exact doit être :

Code :

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

1
2
3
4
5
6
if(!list.isEmpty()) {
     for (String s : list) {
          sb.append(s).append(", ");
     }
     sb.setLength(sb.length()-2);
}

[adiGuba]

Oui en effet

a++