Mit TryParse für Objekteigenschaft Werte Einstellung

Question

Ich bin derzeit Code Refactoring ist zu ersetzen Convert.To TryParse.

Ich habe über das folgende Stück Code kommen, die eine Eigenschaft auf ein Objekt erstellen und zuweisen.

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

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

     list.Add(p);
}

Was ich komme mit als Ersatz ist:

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

Alle Gedanken, Meinungen, Alternativen zu dem, was ich getan habe?

Answer 1

Schreiben Sie eine Erweiterungsmethode.

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

Answer 2

würde ich eine alternative Implementierung TryParse verwenden, die einen int? zurückgibt:

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

Dann können Sie schreiben:

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

(oder einen anderen Standardwert verwenden, wenn Sie wollen -. Oder so ist es in Ihrem Code sichtbar sein wird)

Answer 3

Ich schlage vor, den TryParse Teil von initializer getrennt. Es wird besser lesbar.

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

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

Answer 4

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; });