无法转换类型为“System.DBNull”的对象 输入“System.String”

Question

我在我的应用程序中收到上述错误。 原始代码

public string GetCustomerNumber(Guid id)
{
     string accountNumber = 
          (string)DBSqlHelperFactory.ExecuteScalar(connectionStringSplendidmyApp, 
                          CommandType.StoredProcedure, 
                          "GetCustomerNumber", 
                          new SqlParameter("@id", id));
     return accountNumber.ToString();
 }

这是我替换为的

public string GetCustomerNumber(Guid id)
{
   object accountNumber =  
          (object)DBSqlHelperFactory.ExecuteScalar(connectionStringSplendidCRM, 
                                CommandType.StoredProcedure, 
                                "spx_GetCustomerNumber", 
                                new SqlParameter("@id", id));
    if (accountNumber is System.DBNull)
    {
       return string.Empty;
    }
    else
    {
       return accountNumber.ToString();
    }
}

是否有更好的方法解决这个问题？

Answer 1

通过一个简单的通用函数，您可以使这变得非常容易。 只需这样做：

return ConvertFromDBVal<string>(accountNumber);

使用该函数：

public static T ConvertFromDBVal<T>(object obj)
{
    if (obj == null || obj == DBNull.Value)
    {
        return default(T); // returns the default value for the type
    }
    else
    {
        return (T)obj;
    }
}

Answer 2

可以使用更短的形式：

return (accountNumber == DBNull.Value) ? string.Empty : accountNumber.ToString()

编辑：还没有注意ExecuteScalar。 如果返回结果中不存在该字段，它确实返回 null。 所以改用：

return (accountNumber == null) ? string.Empty : accountNumber.ToString() 

Answer 3

ExecuteScalar 将返回

• 如果没有结果集，则
• null ，否则返回结果集第一行的第一列，这可能是 DBNull。

如果您知道结果集的第一列是字符串，那么要涵盖所有基础，您需要检查 null 和 DBNull。 类似于：

object accountNumber = ...ExecuteScalar(...);
return (accountNumber == null) ? String.Empty : accountNumber.ToString();

上面的代码依赖于 DBNull.ToString 返回空字符串的事实。

如果 accountNumber 是另一种类型（比如整数），那么您需要更明确：

object accountNumber = ...ExecuteScalar(...);
return (accountNumber == null || Convert.IsDBNull(accountNumber) ?     
         (int) accountNumber : 0;

如果您确定结果集始终至少有一行（例如 SELECT COUNT(*)...），那么您可以跳过对 null 的检查。

在您的情况下，错误消息“无法将类型为‘System.DBNull’的对象转换为类型为‘System.String`”表示结果集的第一列是 DBNUll 值。 这是来自第一行的字符串转换：

string accountNumber = (string) ... ExecuteScalar(...);

Marc_s 的评论说你不需要检查 DBNull.Value 是错误的。

Answer 4

您可以使用 C# 的空合并运算符

return accountNumber ?? string.Empty;

Answer 5

这是我用来转换任何可能是 DBNull.Value 的对象的通用方法：

public static T ConvertDBNull<T>(object value, Func<object, T> conversionFunction)
{
    return conversionFunction(value == DBNull.Value ? null : value);
}

用法：

var result = command.ExecuteScalar();

return result.ConvertDBNull(Convert.ToInt32);

更短：

return command
    .ExecuteScalar()
    .ConvertDBNull(Convert.ToInt32);

Answer 6

还有另一种方法可以解决此问题。 修改一下你的商店流程怎么样？ 通过使用 ISNULL(your field, "") sql 函数，如果返回值为 null，则可以返回空字符串。

然后你就得到了原始版本的干净代码。

Answer 7

我想你可以这样做：

string accountNumber = DBSqlHelperFactory.ExecuteScalar(...) as string;

如果 accountNumber 为空，则意味着它是 DBNull 而不是字符串:)

Answer 8

String.Concat 将 DBNull 和 null 值转换为空字符串。

public string GetCustomerNumber(Guid id)
{
   object accountNumber =  
          (object)DBSqlHelperFactory.ExecuteScalar(connectionStringSplendidCRM, 
                                CommandType.StoredProcedure, 
                                "spx_GetCustomerNumber", 
                                new SqlParameter("@id", id));

    return String.Concat(accountNumber);

 }

然而，我认为你在代码的可理解性上失去了一些东西

Answer 9

由于我得到了一个不为 null 的实例，并且如果与 DBNULL 进行比较，我得到了 Operator '==' can not apply to operands of type 'string' and 'system.dbnull' 例外，
如果我尝试更改为与 NULL 进行比较，它根本不起作用（因为 DBNull 是一个对象），即使这是公认的答案。

我决定简单地使用“is”关键字。
所以结果非常可读：

data = (item is DBNull) ? String.Empty：项目

Answer 10

使用更新的 C# 语法并考虑可空类型的更简洁的方法：

private static T? FromDbNull<T>(object? obj) => 
    obj == null || obj == DBNull.Value ? default : (T)obj;

可以与数据读取器一起使用，如下所示：

        while (reader.Read())
        {
            var newObject = new SomeObject(
                FromDbNull<string?>(reader["nullable_field_1"]),
                FromDbNull<string?>(reader["nullable_field_2"]),
                FromDbNull<string?>(reader["nullable_field_3"]), 
                FromDbNull<double?>(reader["nullable_field_4"])
            );
            
            response.Add(newObject);
        }

Answer 11

因为我自己也遇到了类似的错误，这是谷歌上的第一个点击。
如果您没有将可空属性配置为与您的数据库匹配，则可能会发生相同的错误。

如果您只需要快速& 丑陋的修复，你可以添加：

#nullable 禁用

在模型类的开头禁用。

Answer 12

基于 @rein

public static class DbDataReaderExtensions
{
    public static TObjProp Get<TObj, TObjProp>(
        this DbDataReader reader,
        Expression<Func<TObj, TObjProp>> expression)
    {
        MemberExpression member = expression.Body as MemberExpression;
        string propertyName = member.Member.Name;

        //PropertyInfo propInfo = member.Member as PropertyInfo;

        var recordOrdinal = reader.GetOrdinal(propertyName);
        var obj = reader.GetValue(recordOrdinal);

        if (obj == null || obj == DBNull.Value)
        {
            return default(TObjProp);
        }
        else
        {
            return (TObjProp)obj;
        }
    }
}

给出的答案：

public class MyClass
{
    public bool? IsCheckPassed { get; set; }
}

使用 as：

var test = reader.Get<MyClass, bool?>(o => o.IsCheckPassed);

或者，如果您在异常方法中硬编码类类型：

var test = reader.Get(o => o.IsCheckPassed);

ps 我还没有弄清楚如何在不牺牲代码长度的情况下使 generics 隐式。免费发表评论并提出改进建议

完整示例：

public async Task<MyClass> Test(string connectionString) {
    var result = new MyClass();
    
    await using var con = new SQLiteConnection(connectionString);
    con.Open();

    await using var cmd = con.CreateCommand();
    cmd.CommandText = @$"SELECT Id, IsCheckPassed FROM mytable";
    
    var reader = await cmd.ExecuteReaderAsync();
    while (reader.Read()) {
        // old, not working! Throws exception!
        //bool? isCheckPassed1 = reader.GetBoolean(reader.GetOrdinal("IsCheckPassed"));
        
        // old, working, but too long (also if you have like 20 properties then all the more reasons to refactor..)
        bool? isCheckPassed2 = null;
        bool? isCheckPassed2Temp = reader.GetValue(reader.GetOrdinal("IsCheckPassed"));
        if (isCheckPassed2Temp != null && isCheckPassed2Temp != DBNull.Value)
            isCheckPassed2 = (bool?)isCheckPassed2Temp;
        
        // new
        var isCheckPassed3 = reader.Get<MyClass, bool?>(o => o.IsCheckPassed);
        // repeat for 20 more properties :)
        
        result.IsCheckPassed = isCheckPassed3;
    }
    
    return result;
}

只要表列名称与类的属性名称匹配，解决方案就有效。 并且可能不是生产级性能明智的，因此使用或修改需要您自担风险:)

Answer 13

将其转换为

string s = System.DBNull.value.ToString();