Wirf eine Null in etwas?

Question

Ich hatte heute eine interessante Diskussion mit einem Kollegen. Wir diskutierten zwei Codeabschnitte in C#.

Code Snippet 1:

if(!reader.IsDBNull(2)) 
{ 
    long? variable1 = reader.GetInt64(2) 
} 

Code Snippet 2:

long variable1 = reader.IsDBNull(2) ? (long?) null : reader.GetInt64(2) 

Frage ist: ist es eine gute Praxis null in einen nullable lange zu werfen? Oder würden Sie lieber die traditionelle if-Anweisung verwenden, um zu vermeiden, dass null lang genug NULL ist.

Answer 1

Die Ausdrücke (type?)null, default(type?) und new Nullable<type>() am Ende kompiliert wird in die gleichen Opcodes Refactoring würde:

 long? x = (long?)null; 
     long? y = default(long?); 
     long? z = new Nullable<long>(); 

wird verwandelte sich in:

IL_0001: ldloca.s x 
    IL_0003: initobj valuetype [mscorlib]System.Nullable`1<int64> 
    IL_0009: ldloca.s y 
    IL_000b: initobj valuetype [mscorlib]System.Nullable`1<int64> 
    IL_0011: ldloca.s z 
    IL_0013: initobj valuetype [mscorlib]System.Nullable`1<int64> 

Mit anderen Worten, wenn Sie mit Nullable-Typen arbeiten, Sie kostenlos zu benutzen sind je nachdem, welche Version Sie am besten gefällt. Beachten Sie jedoch, dass Sie versuchen sollten, Arithmetik mit nullbaren Typen zu vermeiden. Wenn Sie einen nullfähigen Wert aus einem bedingten Ausdruck zurückgeben möchten, müssen beide möglichen Ergebnisse nullwertfähig sein, wenn einer von ihnen null sein kann. Jeder andere Weg könnte in diesem Fall eine Ausnahme verursachen.

Answer 2

ziehe ich es nicht null Wert werfen (es mir seltsam aussieht):

long? variable1 = reader.IsDBNull(2) ? null : (long?)reader.GetInt64(2); 

Weitere Optionen:

long? variable1 = reader.IsDBNull(2) ? default(long?) : reader.GetInt64(2); 
long? variable1 = reader.IsDBNull(2) ? (long?)null : reader.GetInt64(2); 
long? variable1 = reader.IsDBNull(2) ? new Nullable<long>() : reader.GetInt64(2); 
long? variable1 = reader.IsDBNull(2) ? new long?() : reader.GetInt64(2); 
long? variable1 = reader.IsDBNull(2) ? null : new long?(reader.GetInt64(2)); 

Es ist nur die Frage des Geschmacks. Ich denke, die erste Option ist lesbarer als andere.

UPDATE: Stellen Sie sich auch einige Erweiterungsmethoden Schreiben Sie den Code deutlicher zu machen:

public static class DataReaderExtensions 
{ 
    public static long? GetNullableInt64(this IDataReader reader, int index) 
    { 
     if (reader.IsDBNull(index)) 
      return null; 

     return reader.GetInt64(index); 
    } 
} 

In diesem Fall, dass Sie nicht verwenden ternären Operator (kein Casting NULLABLE), und Lesewerte von Lesern mehr sehen recht:

long? variable1 = reader.GetNullableInt64(2); 

Answer 3

Snippet 2 ist in meinem Fall wert, wie im Fall von null du wird 0 erhalten, die für long

ein vollständig gültigen Wert

Answer 4

Statt

(long?) null 

Verwendung

default(long?) 

ich über Code wie

long? variable1 = reader.IsDBNull(2) ? default(long?) : reader.GetInt64(2)