Impossible de convertir l'objet de type 'System.DBNull' en type 'System.String'.

Question

J'ai obtenu l'erreur ci-dessus dans mon application. Voici le code original

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

Je l'ai remplacé par

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

Y a-t-il un meilleur moyen de contourner ce problème ?

Answer 1

Avec une simple fonction générique, vous pouvez rendre cela très facile. Il suffit de faire ceci :

return ConvertFromDBVal<string>(accountNumber);

en utilisant la fonction :

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

Un formulaire plus court peut être utilisé :

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

EDIT : Je n'ai pas prêté attention à ExecuteScalar. Il retourne vraiment null si le champ est absent du résultat de retour. Donc utilisez-le à la place :

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

Answer 3

ExecuteScalar retournera

• null s'il n'y a pas d'ensemble de résultats
• sinon la première colonne de la première ligne du jeu de résultats, qui peut être DBNull.

Si vous savez que la première colonne du jeu de résultats est une chaîne de caractères, alors, pour couvrir toutes les bases, vous devez vérifier à la fois les null et les DBNull. Quelque chose comme :

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

Le code ci-dessus repose sur le fait que DBNull.ToString renvoie une chaîne vide.

Si accountNumber était d'un autre type (par exemple un nombre entier), il faudrait être plus explicite :

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

Si vous êtes sûr que votre jeu de résultats contiendra toujours au moins une ligne (par exemple, SELECT COUNT(*)...), vous pouvez ignorer la vérification de null.

Dans votre cas, le message d'erreur "Unable to cast object of type 'System.DBNull' to type 'System.String`" indique que la première colonne de votre jeu de résultats est une valeur DBNUll. Ceci est dû au cast en string sur la première ligne :

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

Le commentaire de Marc_s selon lequel il n'est pas nécessaire de vérifier la présence de DBNull.Value est faux.

Answer 4

Vous pouvez utiliser l'opérateur de coalescence de nullité de C#

return accountNumber ?? string.Empty;

Answer 5

Il existe un autre moyen de contourner ce problème. En utilisant la fonction sql ISNULL(votre champ, ""), vous pouvez retourner une chaîne vide si la valeur de retour est nulle.

Vous avez alors votre code propre comme version originale.