Czy stan "Jeśli" jest lepszy niż? i odlewanie

Question

Mam następujące dwa podejścia dla tej samej funkcjonalności - jedna z warunkiem "if", a druga z "?? i odlewania”Które podejście jest lepsze Dlaczego

Kod:.?

Int16? reportID2 = null; 
    //Other code 

    //Approach 1 
    if (reportID2 == null) 
    { 
     command.Parameters.AddWithValue("@report_type_code", DBNull.Value); 
    } 
    else 
    { 
    command.Parameters.AddWithValue("@report_type_code", reportID2); 
    } 

    //Approach 2 
    command.Parameters.AddWithValue("@report_type_code", ((object) reportID2) ?? DBNull.Value); 

UPDATE

Na podstawie odpowiedzi, są następujące korzyści ??

1. Zwiększona czytelność
2. Zmniejszona ilość rozgałęzień w przepływie programu (zredukowany cykl złożoność omatyczna)

Uwaga: Koszt odlewania jako obiekt jest nieistotny.

mającej

1. Null-Coallescing Operator - Why Casting?

Answer 1

Zawsze używam null-coalescing operator w takich przypadkach:

command.Parameters.AddWithValue("@name", value ?? DBNull.Value); 

command.ExecuteScalar() as int? ?? -1; 

itp

Zwiększa kodu reada zdolność, zmniejsza głębokość rozgałęzienia. Został również stworzony specjalnie do scenariuszy związanych z bazą danych, takich jak ADO.NET.

Answer 2

Operator koalescencji zerowej (??) jest lepszym rozwiązaniem, ponieważ działa tak samo jak początkowy blok, ale w pojedynczej, łatwej do odczytania linii. Dzięki temu kod jest bardziej czytelny i łatwiejszy w utrzymaniu.

Jest to jeden z wielu przykładów cukru syntaktycznego, czyli instrukcji kodu, które są "skrótami" do reprezentowania powszechnie używanego pomysłu. Innym tego przykładem jest i++, ponieważ zastępuje on i = i + 1. Jest czystszy i prostszy, podobnie jak ??.

Answer 3

W twoim przykładzie podejście 2 jest lepsze. Ty should not repeat yourself, a apprach 1 ma kod i nazwę parametru dwa razy. Jeśli chcesz zmienić nazwę paramater, powinieneś to zrobić w dwóch miejscach, a to jest kłopot.

Prawdziwy kod porównać to jest to:

object value = DBNull.Value; 
if (reportID2 != null) 
{ 
    value = reportID2; 
} 
command.Parameters.AddWithValue("@report_type_code", value); 

Jeśli używasz tego czy operator ?? to kwestia osobistych preferencji. Myślę, że if jest bardziej przejrzysty, szczególnie dlatego, że w przypadku operatora koalescencji potrzebny jest nawias i rzut.

Answer 4

Wolałbym operatora ??. Chociaż zwięzłość nie zawsze prowadzi do lepszej czytelności, w tym przypadku jest tak, ponieważ jako czytelnik nie trzeba porównywać tego, co jest równe, a co różni się między dwoma wierszami if i else. Ponadto wyeliminowałeś duplikowanie kodu (co zawsze jest dobre!). Rozważ przypadek zmiany nazwy pola bazy danych o nazwie @report_type_code.Następnie musisz to zmienić w jednym miejscu.