Nullcheck before int.TryParse

Question

У меня есть код, в котором многие строки анализируются на целые значения.

string item = null; //or a value 
int result; 
if (!string.IsNullOrEmpty(item) && int.TryParse(item, out result)) 
{ 
    //do stuff 
} 

Нужно ли каждый раз проверять IsNullOrEmpty? Если это null или пустой, синтаксический разбор должен завершиться неудачей.

Answer 1

Нет, String.IsNullOrEmpty избыточен здесь, потому что Int32.TryParse обрабатывает этот случай, возвращая false. Так что это более кратким:

int result; 
if (int.TryParse(item, out result)) 
{ 
    //do stuff 
} 

MSDN:

преобразование не выполняется, если параметр s является нулевым или String.Empty, является не правильный формат, или представляет собой число меньше чем MinValue или больше MaxValue.

Answer 2

Int.TryParse возвращает логическое значение (true, если это удастся), так что вы можете написать:

string item = null; //or a value 
int result; 
if (int.TryParse(item, out result)) 
{ 
    //do stuff 
} 

Answer 3

Нет необходимости проверять наличие пустого или пустого места, если вы используете .TryParse для преобразования. Поскольку возвращаемое значение будет ложным, если преобразование завершилось неудачно. Так безопасно использовать так:

string item = null; //or a value 
int result; 
if(int.TryParse(item, out result)) 
{ 
    //do stuff 
} 
else 
{ 
    // you can show conversion failure message here 
    // or can proceed with value 0 
} 

Дополнительное примечание: Если вы обрабатываете преобразование с помощью int.Parse, то было бы здорово, чтобы проверить string.IsNullOrEmpty поскольку null причины ArgumentNullException и пустая строка причины FormatException