ไม่สามารถแคสต์ออบเจ็กต์ประเภท "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;

หากคุณแน่ใจว่าชุดผลลัพธ์ของคุณจะมีอย่างน้อยหนึ่งแถวเสมอ (เช่น SELECT COUNT (*) ... ) คุณสามารถข้ามการตรวจสอบค่าว่างได้

ในกรณีของคุณข้อความแสดงข้อผิดพลาด "Unable to cast object of type" System.DBNull "to type" System.String` "ระบุว่าคอลัมน์แรกของชุดผลลัพธ์ของคุณเป็นค่า DBNUll นี่คือจากนักแสดงไปจนถึงสตริงในบรรทัดแรก:

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

ความคิดเห็นของ Marc_s ที่คุณไม่จำเป็นต้องตรวจสอบ DBNull ค่าผิด

คุณสามารถใช้ตัวดำเนินการรวมค่าว่างของ C #

return accountNumber ?? string.Empty;

มีอีกวิธีหนึ่งในการแก้ไขปัญหานี้ วิธีแก้ไขขั้นตอนการจัดเก็บของคุณ โดยใช้ฟังก์ชัน ISNULL (ฟิลด์ของคุณ "") sql คุณสามารถส่งคืนสตริงว่างได้หากค่าที่ส่งคืนเป็นค่าว่าง

จากนั้นคุณมีรหัสที่สะอาดเป็นเวอร์ชันดั้งเดิม

นี่เป็นวิธีการทั่วไปที่ฉันใช้เพื่อแปลงอ็อบเจ็กต์ใด ๆ ที่อาจเป็น 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 เป็นโมฆะหมายความว่า 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);

 }

อย่างไรก็ตามฉันคิดว่าคุณสูญเสียบางอย่างในการเข้าใจโค้ด

เนื่องจากฉันมีอินสแตนซ์ที่ไม่เป็นโมฆะและถ้าฉันเทียบกับ DBNULL ฉันได้ Operator '==' cannot be applied to operands of type 'string' and 'system.dbnull' exeption และถ้าฉันพยายามเปลี่ยนเพื่อเปรียบเทียบกับ 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();