タイプ「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()

編集：ExecuteScalarに注意を払っていません。返される結果にフィールドがない場合は、実際にはnullを返します。代わりに使用してください：

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;

結果セットに常に少なくとも1つの行があることが確実である場合（例：SELECT COUNT（*）...）、nullのチェックをスキップできます。

あなたの場合、エラーメッセージ「タイプ 'System.DBNull'のオブジェクトをタイプ 'System.String`にキャストできません」は、結果セットの最初の列がDBNUll値であることを示しています。これは、1行目のキャストから文字列への変換です。

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

Marc_sの、DBNull.Valueをチェックする必要がないというコメントは間違っています。

C＃のnull合体演算子を使用できます

return accountNumber ?? string.Empty;

この問題を回避する別の方法があります。ストアの手順を変更してみませんか？ISNULL（your field、 ""）SQL関数を使用すると、戻り値がnullの場合に空の文字列を返すことができます。

次に、元のバージョンとしてクリーンなコードがあります。

これは、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);

 }

しかし、私はあなたがコードの理解可能性で何かを失うと思います

nullではないインスタンスを取得したため、DBNULLと比較すると例外が発生しOperator '==' cannot be applied to operands of type 'string' and 'system.dbnull'、NULLと比較するように変更しようとしても、（DBNullはオブジェクトであるため）動作しませんでした。

「is」キーワードを使用することにしました。したがって、結果は非常に読みやすくなっています。

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