Использование TryParse для установки значений свойств объекта

Question

Я в настоящее время реорганизую код для замены Convert.To's TryParse.

Я натолкнулся на следующий бит кода, который создает и присваивает свойство объекту.

List<Person> list = new List<Person>(); 

foreach (DataRow row in dt.Rows) 
{ 
    var p = new Person{ RecordID = Convert.ToInt32(row["ContactID"]) }; 

    list.Add(p); 
} 

То, что я придумал, как замена:

var p = new Person { RecordID = Int32.TryParse(row["ContactID"].ToString(), out RecordID) ? RecordID : RecordID }; 

Любые мысли, мнения, альтернативы тому, что я сделал?

Answer 1

Напишите метод расширения.

public static Int32? ParseInt32(this string str) { 
    Int32 k; 
    if(Int32.TryParse(str, out k)) 
     return k; 
    return null; 
} 

Answer 2

Я хотел бы использовать альтернативную реализацию TryParse, которая возвращает int?:

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

Тогда вы можете написать:

var p = new Person { RecordID = Helpers.TryParseInt32(row["ContactID"].ToString()) ?? 0 }; 

(Или использовать другое значение по умолчанию, если вы хотите - в любом случае это будет видно в вашем коде.)

Answer 3

Предлагаю отделить Часть TryParse из инициализатора. Это будет более читаемо.

int recordId; 
Int32.TryParse(row["ContactID"].ToString(), out recordID) 

foreach (DataRow row in dt.Rows) 
{ 
    var p = new Person{ RecordID = recordId }; 
    list.Add(p); 
}