ジェネリック型がnullableの場合はnullを返します

Question

出力がDBNullであるかどうかをチェックし、ジェネリック型がnullの場合はnullを返す関数を記述しています。そうでなければ、単にエラーをスローします。

アップデート：私はdefault(T)リターンがすべてのNULL可能フィールドにNULLかどうかわからないのですが、すべての推奨

public T Get<T>(string key) 
{ 
    int ordinal = reader.GetOrdinal(key); 
    if (reader.IsDBNull(ordinal)) 
    { 
     if (typeof(T).GetTypeInfo().IsValueType && Nullable.GetUnderlyingType(typeof(T)) == null) // isn't a nullable field 
      throw new InvalidCastException(); 
     else return default(T); 
    } 
    return reader.GetFieldValue<T>(ordinal); 
} 

を追加しました。今はnullを返すように他の方法がある場合は？

Answer 1

はい、default(T)はあなたが正常に動作する必要があります持っているので、何を、実際にSomeType?/Nullable<SomeType>あるすべてのTためnull様価値の正しい種類です。

あなたはしかしstringとbyte[]のようなものを検討する必要があります - これらはものデータベースから来ることができるし、nullにすることができます。

値が<decimal>の場合、誰かが単に間違いを犯したときに何をするべきかを考えて、<int>を求めることもできます。その場合、InvalidCastExceptionが発生します。例外を使用するよりも適切かもしれないIDataReaderにIsDBNull()メソッドがあります。最後に

：例外をキャッチして処理する、コンピュータリソースの面ではthrow;throw e;へ

Answer 2

を好むが、非常に高価な作業です。あなたのメソッドは、ロジックフローの一部として例外を使用しています。これは、あなたが解決したい問題を見て間違った方法です。

また、特にInvalidCastException例外をキャッチしています。この例外は、null参照をキャストしようとする以外の場合に発生する可能性があります。潜在的に有効なエラーを隠しているか誤解しています。あなたに何時間もの不満を抱かせて、髪を引っ張るようなデバッグが必要なものです。

以下は元の方法と同じ動作をしますが、統合エラー処理中にスタックを巻き戻すオーバーヘッドはありません。

public T Get<T>(string key) 
{ 
    if(reader.IsDbNull(reader.GetOrdinal(key))) 
    { 
     //IF YOU SPECIFICALLY WANT TO THROW AN ERROR IF A VALUE TYPE 
     //if (typeof(T).IsValueType) 
     //{ throw new InvalidCastException(); } 
     return default(T); 
    } 

    return reader.GetFieldValue<T>(reader.GetOrdinal(key)); 
}