Каковы наилучшие методы безопасного преобразования типов в C#?

Question

Каковы наилучшие методы преобразования типов в C#?

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

Это терпит неудачу, если Id какой-то образом оказывается «а»

Пожалуйста, советы использования троичных операторов и других заявлений однолинейных кроме если другого заявления (например, используя одну строки тройных операторов). Кроме того, вы, ребята, предпочитаете TryParse over Convert &, почему так? Попросите своих товарищей по стилю.

Answer 1

TryParse имеет очевидное преимущество, что в случае отказа он будет возвращать ложь вместо того, чтобы бросать исключение.

Стандартная модель будет что-то вроде:

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

Теперь, это также стоит иметь в виду, что вы можете дать int.TryParse в IFormatProvider и NumberStyles - например, вы можете захотеть указать CultureInfo.InvariantCulture в качестве IFormatProvider, если это действительно означает автоматический идентификатор (а не введенный пользователем).

Если вы хотите эффективно иметь «значения по умолчанию» можно написать вспомогательный метод как это:

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

Вы можете использовать это как так:

int value = NullableTryParseInt32(text) ?? 10; 

Или вы могли бы просто написать метод, который принимает значение по умолчанию, конечно :)

Answer 2

использовать метод TryParse для класса int.

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

Если идентификатор не содержит числа, TryParse вернет false.

UPDATE: Изменено показать int.TryParse

Answer 3

Когда дело доходит до решения любой проблемы, которая имеет несколько подобных решений, я также пытаюсь найти тот, который выражает читателю кода, который я пытаюсь выполнить. В моем oppinion это означает, что вы собираетесь использовать .TryParse в этом конкретном случае.

Использование TryParse говорит читателю, что вы не гарантированы, что вход активен (если вы были бы я использовать вместо разбора) А так как вы на самом деле пытается разобрать вход как межды вы можете также пусть считанного код Line ваших намерений

Answer 4

у вас есть два способа сделать это вы можете сделать

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

или:

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

Казалось бы, что это: http://www.kodefuguru.com/post/2010/06/24/TryParse-vs-Convert.aspx отвечает на ваш вопрос

Answer 6

Использование TryParse будет лучшим вариантом. Исключение исключения из метода конвертации - дорогостоящая операция.Конечно, TryParse принимал бы только строки, в то время как Convert.ToInt32 будет принимать объект и может выполнять преобразования (распаковывание, сбрасывание из длинного/двойного), кроме разбора.

Answer 7

Для покрытия троичной аспект оператора этого вопроса:

Мой совет по использованию тройных операторов не использовать их, если вы еще не так хорошо знакомы с кодом в вопросе, что он читает, естественно, к вам , Конкретность делает знакомых более знакомыми и странными, незнакомыми.

Если вы достаточно подробно остановились на обсуждении TryParse, вам не нужно даже думать об этом до конца, преобразование из if-else в?: Не будет просто тривиальным, оно будет автоматическим. До тех пор ты просто собираешься добавить свое собственное замешательство.

Когда я незнакома с чем-то, сначала сбрасываю код «говорить о ребенке», изучаю новую вещь, а затем интегрирую ее в свой обычный более сжатый стиль.