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

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

4 голосов
/ 12 ноября 2008

В настоящее время я выполняю рефакторинг кода для замены 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 };

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

Ответы [ 4 ]

6 голосов
/ 12 ноября 2008

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

public static Int32? ParseInt32(this string str) {
    Int32 k;
    if(Int32.TryParse(str, out k))
        return k;
    return null;
}
1 голос
/ 12 ноября 2008

Я бы использовал альтернативную реализацию 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 };

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

0 голосов
/ 08 января 2019 
private static void TryToDecimal(string str, Action<decimal> action)
{
        if (decimal.TryParse(str, out decimal ret))
        {
            action(ret);
        }
        else
        {
            //do something you want
        }
}

TryToDecimal(strList[5], (x) => { st.LastTradePrice = x; });
TryToDecimal(strList[3], (x) => { st.LastClosedPrice = x; });
TryToDecimal(strList[6], (x) => { st.TopPrice = x; });
TryToDecimal(strList[7], (x) => { st.BottomPrice = x; });
TryToDecimal(strList[10], (x) => { st.PriceChange = x; });
0 голосов
/ 12 ноября 2008

Я предлагаю отделить часть 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);
}
...