Неможливо передати об’єкт типу "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()

EDIT: Я не звертав уваги на ExecuteScalar. Це дійсно повертає нуль, якщо поле відсутнє в результаті повернення. Тому використовуйте замість цього:

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 є неправильним.

Ви можете використовувати оператор нульового згортання C #

return accountNumber ?? string.Empty;

Є ще один спосіб вирішити це питання. Як щодо модифікації процедури вашого магазину? використовуючи ISNULL (ваше поле, "") sql функцію, ви можете повернути порожній рядок, якщо значення повернення є нульовим.

Тоді у вас є чистий код як оригінальна версія.

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