Quel est le problème avec ToLowerInvariant() ?

Question

J'ai la ligne de code suivante :

var connectionString = configItems.
                Find(item => item.Name.ToLowerInvariant() == "connectionstring");

L'analyse du code de VS 2010 m'indique ce qui suit :

Avertissement 7 CA1308 : Microsoft.Globalization : Dans la méthode ... remplacer l'appel à 'string.ToLowerInvariant()' par String.ToUpperInvariant().

Est-ce que cela signifie ToUpperInvariant() est plus fiable ?

Answer 1

Google donne un indice pointant vers CA1308 : Normalisation des chaînes de caractères en majuscules

C'est écrit :

Les chaînes de caractères doivent être normalisées en majuscules. Un petit groupe de caractères, lorsqu'ils sont convertis en minuscules, ne peut pas faire d'aller-retour. Faire un aller-retour signifie convertir les caractères d'une locale à une autre locale qui représente les données de caractères différemment, et ensuite retrouver avec précision les caractères originaux à partir des caractères convertis.

Donc, oui - ToUpper est plus fiable que ToLower.

À l'avenir, je vous suggère de commencer par googler - je le fais pour tous les avertissements de FxCop qui me sont lancés ;) Cela aide beaucoup de lire la documentation correspondante ;)

Answer 2

En plus de ce que dit TomTom, .net est optimisé pour la comparaison de chaînes de caractères en majuscules. L'utilisation de l'invariant supérieur est donc théoriquement plus rapide que l'invariant inférieur.

C'est en effet ce qui est indiqué dans le CLR via C# comme le soulignent les commentaires. Le lien suivant cite cette partie du livre. Je ne suis pas sûr que ce soit vraiment vrai, car on ne trouve rien sur MSDN à ce sujet. Le guide de comparaison des chaînes de caractères sur msdn mentionne que toupperinvariant et tolowerinvariant sont égaux et ne préfère pas le premier.

Comparaisons de chaînes en C#