如果泛型类型可为空，则返回null

Question

我正在编写一个检查输出是否为DBNull的函数，如果泛型类型为可为空，则返回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);
}

但是我不确定default(T)是否为每个可为空的字段返回null。 如果现在还有其他方法可以使它返回null？

Answer 1

是的， default(T)是正确的那种null ，每样值T ，其实是一个SomeType? / Nullable<SomeType> ，所以您所拥有的应该可以正常工作。

但是，您可能要考虑诸如string和byte[]类的东西-它们也可以来自数据库，并且可以为null。

您可能还想考虑如果某人简单地犯了一个错误并在值是<decimal>时要求输入<int>的错误。 那会导致InvalidCastException 。 IDataReader上有一个IsDBNull()方法可能比使用异常更合适。

最后：喜欢throw; throw e;

Answer 2

就计算机资源而言，捕获和处理异常是一项极其昂贵的任务。 您的方法将异常用作其逻辑流程的一部分。 这是查看您要解决的问题的错误方法。

此外，您正在专门捕获InvalidCastException异常。 除了尝试强制转换为空引用以外，在其他情况下也可能发生此异常。 您正在隐藏/歪曲潜在的有效错误； 这可能会使您花费大量时间进行令人沮丧的拔毛调试。

以下应该为您提供与原始方法相同的行为，但不会在集成错误处理过程中释放堆栈的开销。

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