Discussion :

[david71]

Salut,

Je voulais juste savoir s'il existait une fonction toute faite permettant de supprimer tous les blancs d'une chaine, afin d'éviter de réimplémenter cela si cela existe déjà.

Par exemple, retirer les blancs suivants :

ma_chaine__n'est__pas_très_longue

Donner : machainen'estpastrèslongue

Merci pour toute info supplémentaire.

[dev01]

Salut .

regarde du coté de la fonction Trim de la classe string.

[Merlin]

tu sais, "réimplémenter" c'est un bien grand mot, il faut moins de lignes de code pour faire ce que tu veux que ton message pour demander de l'aide...

Code :

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

1
2
3
 
public string DeleteSpace(string laChaine)
{ string tmp; foreach (char c in laChaine) if (c!=' ') tmp+=c; return tmp; }

sinon, en cherchant bien dans la classe string, tu aurais pu voir la fonction Replace().
Pour virer tous les espaces :

Code :

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

machaine = machaine.Replace(" ","");

tout simplement...

[StormimOn]

regarde du coté de la fonction Trim de la classe string.

La méthode Trim remplace les occurences de caractères se situant en début et en fin de chaîne uniquement

Dans le cas présent, la méthode Replace est plus appropriée

Code :

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

1
2
3
 
string maChaine = "ma chaîne n'est pas très longue";
string maNouvelleChaine = maChaine.Replace(" ", string.Empty);

Edit : grilled ^^

[oiffrig]

S'il fait supprimer tous les blancs (donc pas uniquement les espaces), le plus simple est d'utiliser les expressions régulières (namespace System.Text.RegularExpressions)

Code :

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

string nouvelleChaine = Regex.Replace(maChaine, @"\s+", "");

[dev01]

La méthode Trim remplace les occurences de caractères se situant en début et en fin de chaîne uniquement

Dans le cas présent, la méthode Replace est plus appropriée

Code :

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

1
2
3
 
string maChaine = "ma chaîne n'est pas très longue";
string maNouvelleChaine = maChaine.Replace(" ", string.Empty);

Edit : grilled ^^

oups honte sur moi j'ai pas fait gaffe ...

[abelman]

Code :

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

1
2
3
 
public string DeleteSpace(string laChaine)
{ string tmp; foreach (char c in laChaine) if (c!=' ') tmp+=c; return tmp; }

[/code] tout simplement...

Salut,
ça (tmp+=c) c'est un des trucs typiques à éviter car tmp étant un string, faire tmp=... alloue de la mémoire à chaque fois. Pour peu que ta fonction DeleteSpace soit régulièrement sollicitée ça va être lent.

utiliser StringBuilder c'est bien

[david71]

OK merci pour ces informations, qu'est ce qui est plus rapide c'est utiliser les expressions régulières ou la fonction "Replace" (à laquelle je n'ai d'ailleurs pas pensé) ?

[Merlin]

Salut,
ça (tmp+=c) c'est un des trucs typiques à éviter car tmp étant un string, faire tmp=... alloue de la mémoire à chaque fois. Pour peu que ta fonction DeleteSpace soit régulièrement sollicitée ça va être lent.
utiliser StringBuilder c'est bien

Non, il ne faut pas être extrémiste comme ça
Dans l'exemple présent la longueur des chaînes à traiter semble (sans indication contraire) relativement faible. Et dans un tel cas l'utilisation d'un StringBuilder n'est pas forcément à conseiller systématiquement et peut amener à des contre-performances.
Entre les cas où string s'impose et ceux où StringBuilder s'impose, il y a tout un tas de cas limites qui, si on cherche l'optimisation à tout prix, ne peuvent être tranchés qu'avec un profiler en fait.
De fait dès qu'on fait un string += 'bla'; c'est pas toujours rentable d'utiliser un StringBuilder à la place.

[abelman]

Je ne suis pas extrémiste

Je pense quand même que même si la taille de la chaine est petite, si on doit refaire l'opération un certain nombre de fois (typiquement dans une boucle), le StringBuilder est plus performant

Code :

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

1
2
3
4
5
6
7
8
9
10
for (int i=0; i<1000; i++)
{
    mastring += "bla"; // equivaut à mastring = new string(mastring + 'bla');
}
 
// Sb étant un StringBuilder, ce code est plus rapide
for (int i=0; i<1000; i++)
{
    sb.Append("bla"); 
}

Par contre si on le fait juste une fois, effectivement je suis d'accord avec toi

[abelman]

http://www.developpez.net/forums/sho...d.php?p=876846

[bayedemba]

Code :

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

1
2
3
4
5
string drChoisie = " toto ";

drChoisie.Trim();//vide les caractères devant et dérriére la chaine

response.write(drChoisie);// affiche drChoisie = "toto"

[Nathanael Marchand]

C'est peut être pas obligatoire de remonter des sujets de 2006 pour donner la même réponse que dans le message #2 (surtout que celle-ci est erronnée)

[Bluedeep]

utiliser StringBuilder c'est bien

Dans le cas précis, StringBuilder qui est beaucoup plus performante en général que la manip de chaine "à la mimine" n'apporte strictement rien (au contraire).

[Bluedeep]

C'est peut être pas obligatoire de remonter des sujets de 2006 pour donner la même réponse que dans le message #2 (surtout que celle-ci est erronnée)

Oups, pan sur le bec, c'est valable pour moi aussi