Không thể truyền đối tượng kiểu 'System.DBNull' thành kiểu 'System.String`

Tôi gặp lỗi ở trên trong ứng dụng của mình. Đây là mã gốc

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

Tôi đã thay thế bằng

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ó cách nào tốt hơn để giải quyết vấn đề này không?

Có thể sử dụng mẫu ngắn hơn:

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

CHỈNH SỬA: Chưa chú ý đến ExecuteScalar. Nó thực sự trả về null nếu trường không có trong kết quả trả về. Vì vậy, hãy sử dụng thay thế:

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

Với một hàm chung đơn giản, bạn có thể thực hiện điều này rất dễ dàng. Chỉ cần làm điều này:

return ConvertFromDBVal<string>(accountNumber);

sử dụng chức năng:

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 sẽ trả về

• null nếu không có tập hợp kết quả
• nếu không thì cột đầu tiên của hàng đầu tiên của tập kết quả, có thể là DBNull.

Nếu bạn biết rằng cột đầu tiên của tập kết quả là một chuỗi, thì để bao gồm tất cả các cơ sở, bạn cần kiểm tra cả null và DBNull. Cái gì đó như:

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

Đoạn mã trên dựa trên thực tế là DBNull.ToString trả về một chuỗi rỗng.

Nếu accountNumber là một loại khác (giả sử là số nguyên), thì bạn cần phải rõ ràng hơn:

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

Nếu bạn biết chắc rằng tập kết quả của mình sẽ luôn có ít nhất một hàng (ví dụ: CHỌN COUNT (*) ...), thì bạn có thể bỏ qua việc kiểm tra giá trị rỗng.

Trong trường hợp của bạn, thông báo lỗi "Không thể truyền đối tượng kiểu 'System.DBNull' sang loại 'System.String`" cho biết cột đầu tiên của tập kết quả của bạn là giá trị DBNUll. Đây là từ ép kiểu thành chuỗi trên dòng đầu tiên:

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

Nhận xét của Marc_s rằng bạn không cần kiểm tra DBNull.Value là sai.

Bạn có thể sử dụng toán tử liên kết null của C #

return accountNumber ?? string.Empty;

Có một cách khác để giải quyết vấn đề này. Làm thế nào về việc sửa đổi thủ tục cửa hàng của bạn? bằng cách sử dụng hàm sql ISNULL (trường của bạn, ""), bạn có thể trả về chuỗi trống nếu giá trị trả về là null.

Sau đó, bạn có mã sạch của mình như phiên bản gốc.

Đây là phương pháp chung mà tôi sử dụng để chuyển đổi bất kỳ đối tượng nào có thể là DBNull.

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

sử dụng:

var result = command.ExecuteScalar();

return result.ConvertDBNull(Convert.ToInt32);

ngắn hơn:

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

Tôi cho rằng bạn có thể làm như thế này:

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

Nếu accountNumber là null nghĩa là nó không phải là DBNull string :)

String.Concat biến đổi các giá trị DBNull và null thành một chuỗi rỗng.

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

    return String.Concat(accountNumber);

 }

Tuy nhiên, tôi nghĩ rằng bạn mất một số thứ về khả năng hiểu mã

Vì tôi nhận được một phiên bản không phải là null và nếu tôi so sánh với DBNULL, tôi đã nhận được lỗi Operator '==' cannot be applied to operands of type 'string' and 'system.dbnull'và nếu tôi cố gắng thay đổi để so sánh với NULL, nó chỉ đơn giản là không hoạt động (vì DBNull là một đối tượng) ngay cả khi đó là câu trả lời được chấp nhận.

Tôi quyết định chỉ sử dụng từ khóa 'là'. Vì vậy, kết quả rất dễ đọc:

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

Tôi sử dụng tiện ích mở rộng để loại bỏ vấn đề này đối với tôi, có thể có hoặc không phải những gì bạn đang theo đuổi.

Nó diễn ra như thế này:

public static class Extensions
{

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

}

Ghi chú:

Phần mở rộng này không trả về nullgiá trị! Nếu mục là nullhoặc DBNull.Value , nó sẽ trả về một Chuỗi trống.

Sử dụng:

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

Chuyển đổi nó giống như

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