Quelles sont les meilleures pratiques pour la conversion de type sécurisé en C#?

Question

Quelles sont les meilleures pratiques pour les conversions de type en C#?

int temp=System.ConvertToInt32(Request.QueryString["Id"]); 
    if (temp!=null) 
     { // logic goes here } 

échoue si Id s'en quelque sorte à être « abc »

S'il vous plaît conseiller l'utilisation des opérateurs ternaires et d'autres énoncés de ligne unique à part si les déclarations d'autre (comme l'utilisation d'opérateurs ternaires seule ligne). Aussi, préférez-vous TryParse sur Convert & pourquoi? Avez-vous dit les gars.

Answer 1

TryParse a l'avantage évident que dans le cas d'échec, il retourne faux au lieu de lancer une exception.

Le modèle standard serait quelque chose comme:

int value; 
if (int.TryParse(Request.QueryString["Id"], out value)) 
{ 
    // Use value 
} 
else 
{ 
    // Do whatever you want on failure 
} 

Maintenant, il est également intéressant compte tenu du fait que vous pouvez donner int.TryParse un IFormatProvider et un NumberStyles - par exemple, vous pouvez bien vouloir préciser CultureInfo.InvariantCulture comme IFormatProvider s'il s'agit vraiment d'un identifiant généré automatiquement (plutôt qu'un identifiant entré par un utilisateur).

Si vous voulez avoir efficacement « valeurs par défaut », vous pouvez écrire une méthode d'aide comme ceci:

public static int? NullableTryParseInt32(string text) 
{ 
    int value; 
    return int.TryParse(text, out value) ? value : (int?) null; 
} 

Vous pouvez ensuite utiliser ce comme ceci:

int value = NullableTryParseInt32(text) ?? 10; 

Ou vous pouvez simplement écrire une méthode qui prend une valeur par défaut, bien sûr :)

Answer 2

utilisez la méthode TryParse de la classe int.

int temp; 
if (int.TryParse(Request.QueryString["Id"], out temp) 
    { // logic goes here } 

Si ID ne contient pas de nombre, TryParse renvoie false.

MISE À JOUR: Changé pour montrer int.TryParse

Answer 3

Quand il s'agit de résoudre n'importe quel problème qui a plusieurs solutions similaires, j'essaie aussi de trouver celui qui s'exprime au lecteur du code que j'essaye d'accomplir le cleæret. Dans mon oppinion cela signifie aller pour le .TryParse dans ce cas particulier.

L'utilisation TryParse indique au lecteur que vous n'êtes pas garanti que l'entrée est valide (si vous étiez j'utiliser au lieu d'analyser) Et puisque vous êtes réellement essaie pour analyser l'entrée comme un int vous pourriez aussi bien laissez le code lu ligne vos intentions

Answer 4

vous avez deux façons de le faire

int i; 
if (Int32.TryParse(Request.QueryString["Id"], out i)) 
{ 
} 

ou vous pouvez faire:

try 
{ 
    Convert.ToInt32(Request.QueryString["Id"]); 
} 
catch (FormatException ex) 
{ 
    // The field Id it's not convertible 
} 
catch (Exception ex) 
{ 
    // It could throw also ArgumentException or OverflowException 
} 

Answer 5

Il semblerait que ceci: http://www.kodefuguru.com/post/2010/06/24/TryParse-vs-Convert.aspx répond à votre question

Answer 6

TryParse serait à l'aide de la meilleure option. Attraper l'exception de la méthode convert est une opération coûteuse.Bien sûr, TryParse n'accepterait que des chaînes tandis que Convert.ToInt32 prendrait l'objet et pourrait effectuer des conversions (désemballage, down-casting de long/double) en dehors de l'analyse syntaxique.

Answer 7

Pour couvrir l'aspect opérateur ternaire de cette question:

Mon conseil sur l'utilisation des opérateurs ternaires est de ne pas les utiliser si vous n'êtes pas déjà familier avec le code en question qu'il lit naturellement à vous . La concision rend le familier plus familier et l'étranger étrange. Lorsque vous avez assez parlé de TryParse ici pour ne plus avoir à y réfléchir, la conversion de if-else en?: Ne sera pas seulement triviale, elle sera automatique. Jusque-là, vous allez juste ajouter à votre propre confusion. Quand je ne suis pas familier avec quelque chose, je descends au code «bébé-parler» au début, apprends la nouvelle chose, et ensuite l'intègre dans mon style normal plus concis après.