Impossibile eseguire il cast di un oggetto di tipo "System.DBNull" per digitare "System.String"

Ho ricevuto l'errore di cui sopra nella mia app. Ecco il codice originale

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

Ho sostituito con

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

C'è un modo migliore per aggirare questo?

È possibile utilizzare una forma più breve:

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

EDIT: Non ho prestato attenzione a ExecuteScalar. Restituisce davvero null se il campo è assente nel risultato restituito. Quindi usa invece:

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

Con una semplice funzione generica puoi renderlo molto facile. Basta fare questo:

return ConvertFromDBVal<string>(accountNumber);

utilizzando la funzione:

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 tornerà

• null se non è stato impostato alcun risultato
• altrimenti la prima colonna della prima riga del gruppo di risultati, che potrebbe essere DBNull.

Se sai che la prima colonna del gruppo di risultati è una stringa, per coprire tutte le basi devi controllare sia null che DBNull. Qualcosa di simile a:

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

Il codice precedente si basa sul fatto che DBNull.ToString restituisce una stringa vuota.

Se accountNumber fosse un altro tipo (diciamo intero), allora dovresti essere più esplicito:

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

Se sai per certo che il tuo gruppo di risultati avrà sempre almeno una riga (ad es. SELEZIONA COUNT (*) ...), puoi saltare il controllo per null.

Nel tuo caso, il messaggio di errore "Impossibile eseguire il cast dell'oggetto di tipo" System.DBNull "per digitare" System.String "" indica che la prima colonna del set di risultati è un valore DBNUll. Questo è dal cast alla stringa sulla prima riga:

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

Il commento di Marc_s che non è necessario controllare per DBNull.Value è sbagliato.

È possibile utilizzare l'operatore di coalescenza null di C #

return accountNumber ?? string.Empty;

C'è un altro modo per risolvere questo problema. Che ne dici di modificare la procedura del tuo negozio? utilizzando la funzione sql ISNULL (il tuo campo ""), puoi restituire una stringa vuota se il valore restituito è null.

Quindi hai il tuo codice pulito come versione originale.

Questo è il metodo generico che utilizzo per convertire qualsiasi oggetto che potrebbe essere un DBNull.Value:

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

utilizzo:

var result = command.ExecuteScalar();

return result.ConvertDBNull(Convert.ToInt32);

più breve:

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

Suppongo che tu possa farlo in questo modo:

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

Se accountNumber è null significa che era DBNull non stringa :)

String.Concat trasforma DBNull e valori null in una stringa vuota.

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

    return String.Concat(accountNumber);

 }

Tuttavia, penso che tu perda qualcosa sulla comprensibilità del codice

Dato che ho un'istanza che non è nulla e se ho confrontato con DBNULL ho ottenuto Operator '==' cannot be applied to operands of type 'string' and 'system.dbnull' , e se ho provato a cambiare per confrontare con NULL, semplicemente non ha funzionato (poiché DBNull è un oggetto) anche questa è la risposta accettata.

Ho deciso di utilizzare semplicemente la parola chiave "is". Quindi il risultato è molto leggibile:

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

Uso un'estensione per eliminare questo problema per me, che può essere o meno quello che stai cercando.

Funziona così:

public static class Extensions
{

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

}

Nota:

Questa estensione non restituisce nullvalori! Se l'elemento è nullo DBNull.Value , restituirà una stringa vuota.

Uso:

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

Converti come

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