Discussion :

[seb.49]

Suite au post que j'ai vu hier (Utiliser string.ToLower() c'est mal !), je me suis amusé a compararer quelques méthode du framework afin de sélectionner les plus rapide et les mettre dans mon code.

Voici mes premiers résultats :

Code :

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

if (val.ToUpper().Contains("une".ToUpper()) == false)

VS

Code :

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

if (val.IndexOf("une",0,StringComparison.InvariantCultureIgnoreCase) == -1)

Code :

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

cs = cs + "test";

VS

Code :

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

csb.Append("test");

(csb étant un StringBuilder et cs une simple string. La boucle est exécuter 3000 fois)

Code :

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

if (val == "")

VS

Code :

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

if (string.IsNullOrEmpty(val))

Code :

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

string a = "toto " + val + " toto";

VS

Code :

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

string a = string.Format("toto {0} toto", val);

Pour le coup la contaténation est plus rapide que le string.Format, pas très logique je trouve. A creuser...

Pour Neptune et Nip

Code :

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

  if ((val == null) || (val == ""))

VS

Code :

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

if (string.IsNullOrEmpty(val))

Code :

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

  if ((val == null) || (val.Length == 0))

VS

Code :

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

if (string.IsNullOrEmpty(val))

testé 1 000 fois et 3 000 000 de fois à chaque fois sinon le vainqueur était toujours différent

Plus de détail ici http://dotnetperls.com/Content/IsNul...y-Samples.aspx

Code :

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

1
2
3
4
5
6
7
8
9
10
 
DateTime dtDateTime;
try
{
dtDateTime = DateTime.Parse(val); //val étant une string contenant "01/01/2008"
}
catch
{
dtDateTime = DateTime.MinValue;
}

VS

Code :

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

1
2
3
 
DateTime dtDateTime;
DateTime.TryParse(val, out dtDateTime);

Code :

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

1
2
3
4
5
6
7
8
9
  DateTime dtDateTime;
try
{
dtDateTime = DateTime.Parse(val); //val étant une string contenant "01/01/a008"
}
catch
{
dtDateTime = DateTime.MinValue;
}

VS

Code :

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

1
2
3
 
DateTime dtDateTime;
DateTime.TryParse(val, out dtDateTime);

Et vous, vous avez aussi des méthode à préconiser par rapport à d'autres ?

[dev01]

Bonjour.

c'est pas mal ça.

Pour le stringBuilder, c'est quelque chose d'assez connu, et ça s'explique facilement. A chaque fois que tu fait maChaine + " " + uneChaine, il y a 3 réallocations de chaine de caractères avec les recopies qui vont bien.

Dans les bonnes pratiques il y a aussi l'utilisation de string.IsNullOrEmpty(maChaine) à la place de maChaine == ""

[seb.49]

Merci, je rajoute ça dans mon post

[zoubidaman]

J'ai fait un post sur mon blog qui compare le temps d'exécution des différentes fonctions pour concaténer des chaines de caractères...

Je n'ai pas les même résultats. J'ai eu comme conclusion qu'il vaut mieux utiliser StringBuilder quand on veut concaténer au moins 9 chaines de caractères...

[seb.49]

Je constate pas cela, c'est toujours StringBuilder qui gagne chez moi.

D'autres on essayé ?

[dev01]

Je n'ai pas les même résultats. J'ai eu comme conclusion qu'il vaut mieux utiliser StringBuilder quand on veut concaténer au moins 9 chaines de caractères...

???
pas vu ce genre de chose non plus.

Dans les astuces également il y a l'utilisation (mais pas trop intensif quand même) de string.Format("toto {0} totot", maChaine); plutot que a = "toto " + maChaine + " totot";
Normalement ça donne de meilleur résultat.

[seb.49]

???
Dans les astuces également il y a l'utilisation (mais pas trop intensif quand même) de string.Format("toto {0} totot", maChaine); plutot que a = "toto " + maChaine + " totot";
Normalement ça donne de meilleur résultat.

Normalement ? J'ai éditer mon post avec ton test et c'est la concaténation qui est plus rapide par rapport au string.Format.

[neptune]

Pour le coup la contaténation est plus rapide que le string.Format, pas très logique je trouve. A creuser...

C'est contradictoire avec une constation précédente, à savoir qu'un StringBuilder est plus performant qu'un concaténation. Hors, String.Format() utilise un StringBuilder en interne... Il faudrait comparer avec un formatage pour en être certain.

[Nip]

Pour le stringbuilder c'est interessant de l'utiliser a partir de 5 concatenations: http://www.chinhdo.com/20070929/stringbuilder-part-2/

Sinon je n'ai pas verifie la difference de perfs mais myString.Length == 0 est plus efficace que isnullorempty pour verifier si la chaine est vide (ce qui est logique puisque isnullorempty verifie 2 conditions contre une pour length mais meme si tu fais str == null || str.Length == 0, ca devrait etre plus rapide). Mais je suis extremement surpris par les differences de perfs: tu aurais fait str == null || str == "" pour comparer a IsNullOrEmpty j'aurais compris une telle difference mais la c'est etonnant. Tu veux pas mettre le code que tu utilises?

[neptune]

Sinon je n'ai pas verifie la difference de perfs mais myString.Length == 0 est plus efficace que isnullorempty pour verifier si la chaine est vide (ce qui est logique puisque isnullorempty verifie 2 conditions contre une pour length mais meme si tu fais str == null || str.Length == 0, ca devrait etre plus rapide).

Je ne suis pas du tout d'accord avec toi. Tu ne peux tester la longueur avant de t'être assuré que la string n'est pas nulle.

Et puis, voici ce que fait exactement le framework, je pense qu'on ne peut faire mieux en performance pour tester si une chaine est vide.

Code :

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

1
2
3
4
5
6
7
8
public static bool IsNullOrEmpty(string value)
{
    if (value != null)
    {
        return (value.Length == 0);
    }
    return true;
}

[Nip]

Je ne suis pas du tout d'accord avec toi. Tu ne peux tester la longueur avant de t'être assuré que la string n'est pas nulle.

Tu peux ne pas etre d'accord etant donne que je parle explicitement de chaine vide. Quand je lis

Code :

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

if (val == "")

il n'y a la aucune verification de la nullite. Dans ce cas precis val.Length == 0 est de loin la meilleure option.

Mais si myString est null tu plantes. Bien sur tu a peut être un try catch mais la les performances sont a la ramasse.

Totalement d'accord; si il y a necessite de tester la nullite alors la meilleure solution est le IsNullOrEmpty.

[davcha]

D'une manière générale, si on a besoin de faire de nombreux appels à la reflexion (en dehors de GetType() qui est optimisé), il vaut mieux faire appel à des méthodes dynamiques.

[seb.49]

myString.Length == 0 est plus efficace que isnullorempty

Mais si myString est null tu plantes. Bien sur tu a peut être un try catch mais la les performances sont a la ramasse.

Regarde les test de conversion de date dans la post initial

Tu veux pas mettre le code que tu utilises?

http://www.coldwire.net/blogs/toutou...de-code-c.aspx