Não é possível lançar o objeto do tipo 'System.DBNull' para o tipo 'System.String`

Recebi o erro acima em meu aplicativo. Aqui está o código original

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

Eu substituí com

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

Existe uma maneira melhor de contornar isso?

Uma forma mais curta pode ser usada:

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

EDIT: Não prestei atenção ao ExecuteScalar. Ele realmente retorna nulo se o campo estiver ausente no resultado de retorno. Portanto, use:

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

Com uma função genérica simples, você pode tornar isso muito fácil. Basta fazer isso:

return ConvertFromDBVal<string>(accountNumber);

usando a função:

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 irá retornar

• nulo se não houver conjunto de resultados
• caso contrário, a primeira coluna da primeira linha do conjunto de resultados, que pode ser DBNull.

Se você sabe que a primeira coluna do conjunto de resultados é uma string, para cobrir todas as bases, você precisa verificar se há nulo e DBNull. Algo como:

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

O código acima se baseia no fato de que DBNull.ToString retorna uma string vazia.

Se accountNumber fosse outro tipo (digamos, inteiro), você precisaria ser mais explícito:

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

Se você tiver certeza de que seu conjunto de resultados sempre terá pelo menos uma linha (por exemplo, SELECT COUNT (*) ...), você pode ignorar a verificação de nulo.

No seu caso, a mensagem de erro "Não foi possível lançar o objeto do tipo 'System.DBNull' para o tipo 'System.String`" indica que a primeira coluna do conjunto de resultados é um valor DBNUll. Isso é do elenco para a string na primeira linha:

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

O comentário de Marc_s de que você não precisa verificar DBNull.Value está errado.

Você pode usar o operador de coalescência nula do C #

return accountNumber ?? string.Empty;

Existe outra maneira de contornar esse problema. Que tal modificar seu procedimento de loja? usando ISNULL (seu campo, "") a função sql, você pode retornar uma string vazia se o valor de retorno for nulo.

Então você tem seu código limpo como versão original.

Este é o método genérico que uso para converter qualquer objeto que possa ser um DBNull.Value:

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

uso:

var result = command.ExecuteScalar();

return result.ConvertDBNull(Convert.ToInt32);

mais curta:

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

Suponho que você pode fazer assim:

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

Se accountNumber for null, significa que não era DBNull string :)

String.Concat transforma os valores DBNull e nulos em uma string vazia.

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

    return String.Concat(accountNumber);

 }

No entanto, acho que você perde algo na compreensão do código

Já que eu obtive uma instância que não é nula e se eu comparei com DBNULL eu tive exceção Operator '==' cannot be applied to operands of type 'string' and 'system.dbnull', e se eu tentei mudar para comparar com NULL, simplesmente não funcionou (já que DBNull é um objeto) mesmo essa resposta aceita.

Decidi simplesmente usar a palavra-chave 'é'. Portanto, o resultado é muito legível:

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

Eu uso uma extensão para eliminar esse problema para mim, que pode ou não ser o que você está procurando.

É assim:

public static class Extensions
{

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

}

Nota:

Esta extensão não retorna nullvalores! Se o item for nullou DBNull.Value , ele retornará uma String vazia.

Uso:

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

Converta como

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