Невозможно привести объект типа System.DBNull к типу System.String

Я получил указанную выше ошибку в своем приложении. Вот исходный код

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

Есть ли способ лучше?

Можно использовать более короткую форму:

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

РЕДАКТИРОВАТЬ: Не обращал внимания на ExecuteScalar. Он действительно возвращает null, если поле отсутствует в возвращаемом результате. Поэтому используйте вместо этого:

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

С простой универсальной функцией вы можете сделать это очень легко. Просто сделай это:

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

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, неверен.

Вы можете использовать оператор объединения NULL в C #

return accountNumber ?? string.Empty;

Есть еще один способ обойти эту проблему. Как насчет изменения процедуры вашего магазина? используя функцию sql ISNULL (ваше поле, ""), вы можете вернуть пустую строку, если возвращаемое значение равно null.

Тогда у вас будет чистый код в виде оригинальной версии.

Это общий метод, который я использую для преобразования любого объекта, который может быть 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);

Полагаю, вы можете сделать это так:

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

Если accountNumber имеет значение null, это означает, что это DBNull, а не строка :)

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

 }

Однако я думаю, что вы теряете понятность кода.

Поскольку у меня есть экземпляр, который не равен нулю, и если я сравнил с DBNULL, я получил Operator '==' cannot be applied to operands of type 'string' and 'system.dbnull' исключение, и если я попытался изменить, чтобы сравнить с NULL, он просто не сработал (поскольку DBNull является объектом), даже если это принятый ответ.

Я решил просто использовать ключевое слово «есть». Так что результат очень читабельный:

data = (item is DBNull) ? String.Empty : item

Я использую расширение, чтобы устранить эту проблему, которая может быть или не быть тем, что вам нужно.

Это выглядит так:

public static class Extensions
{

    public String TrimString(this object item)
    {
        return String.Format("{0}", item).Trim();
    }

}

Примечание:

Это расширение не возвращает nullзначений! Если элемент равен nullили DBNull.Value , он вернет пустую строку.

Использование:

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

Преобразуйте это как

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