Nie można rzutować obiektu typu „System.DBNull” na typ „System.String”

Otrzymałem powyższy błąd w mojej aplikacji. Oto oryginalny kod

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

Zastąpiłem

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

Czy jest lepszy sposób obejścia tego?

Można użyć krótszej formy:

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

EDYCJA: Nie zwróciłem uwagi na ExecuteScalar. W rzeczywistości zwraca wartość null, jeśli w wyniku zwracanym nie ma pola. Zamiast tego użyj:

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

Dzięki prostej funkcji ogólnej możesz to bardzo ułatwić. Po prostu zrób to:

return ConvertFromDBVal<string>(accountNumber);

za pomocą funkcji:

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 zwróci

• null, jeśli nie ma zestawu wyników
• w przeciwnym razie pierwsza kolumna pierwszego wiersza zestawu wyników, która może mieć wartość DBNull.

Jeśli wiesz, że pierwsza kolumna zestawu wyników jest łańcuchem, to aby objąć wszystkie zasady, musisz sprawdzić zarówno null, jak i DBNull. Coś jak:

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

Powyższy kod opiera się na fakcie, że DBNull.ToString zwraca pusty ciąg.

Gdyby numer konta był innego typu (powiedzmy liczbą całkowitą), musiałbyś być bardziej precyzyjny:

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

Jeśli wiesz na pewno, że Twój zbiór wyników zawsze będzie zawierał co najmniej jeden wiersz (np. SELECT COUNT (*) ...), możesz pominąć sprawdzanie wartości null.

W Twoim przypadku komunikat o błędzie „Nie można rzutować obiektu typu„ System.DBNull ”na typ„ System.String ”oznacza, że ​​pierwsza kolumna zestawu wyników jest wartością DBNUll. To jest od rzutu do łańcucha w pierwszej linii:

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

Komentarz Marc_s, że nie musisz sprawdzać DBNull.Value, jest błędny.

Można użyć operatora łączenia wartości null w języku C #

return accountNumber ?? string.Empty;

Istnieje inny sposób obejścia tego problemu. Co powiesz na zmodyfikowanie procedury sklepu? używając funkcji ISNULL (twoje pole, "") sql, możesz zwrócić pusty ciąg, jeśli wartość zwracana jest równa null.

Następnie masz czysty kod jako oryginalną wersję.

Oto ogólna metoda, której używam do konwersji dowolnego obiektu, który może być DBNull.Value:

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

stosowanie:

var result = command.ExecuteScalar();

return result.ConvertDBNull(Convert.ToInt32);

krótszy:

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

Przypuszczam, że możesz to zrobić w ten sposób:

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

Jeśli accountNumber ma wartość null, oznacza to, że był to DBNull, a nie łańcuch :)

String.Concat przekształca wartości DBNull i null na pusty ciąg.

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

    return String.Concat(accountNumber);

 }

Myślę jednak, że tracisz coś na zrozumiałości kodu

Odkąd mam instancję, która nie jest null i jeśli porównuję z DBNULL, otrzymałem Operator '==' cannot be applied to operands of type 'string' and 'system.dbnull' wyjątek, a gdybym spróbował zmienić, aby porównać do NULL, po prostu nie działało (ponieważ DBNull jest obiektem), nawet to jest akceptowana odpowiedź.

Postanowiłem po prostu użyć słowa kluczowego „is”. Wynik jest więc bardzo czytelny:

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

Używam rozszerzenia, aby wyeliminować ten problem za mnie, który może, ale nie musi, być tym, czego szukasz.

To wygląda tak:

public static class Extensions
{

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

}

Uwaga:

To rozszerzenie nie zwraca nullwartości! Jeśli elementem jest nulllub DBNull.Value , zwróci pusty ciąg.

Stosowanie:

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

Konwertuj jak

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