ข้อยกเว้นเมื่อพารามิเตอร์ AddWithValue เป็น NULL


92

ฉันมีรหัสต่อไปนี้สำหรับระบุพารามิเตอร์สำหรับแบบสอบถาม SQL ฉันได้รับข้อยกเว้นดังต่อไปนี้เมื่อฉันใช้Code 1; Code 2แต่ทำงานได้ดีเมื่อฉันใช้ ในCode 2เรามีการตรวจสอบค่าว่างและด้วยเหตุนี้if..elseบล็อก

ข้อยกเว้น:

แบบสอบถามที่กำหนดพารามิเตอร์ '(@application_ex_id nvarchar (4000)) SELECT E.application_ex_id A' คาดว่าพารามิเตอร์ '@application_ex_id' ซึ่งไม่ได้ให้มา

รหัส 1 :

command.Parameters.AddWithValue("@application_ex_id", logSearch.LogID);

รหัส 2 :

if (logSearch.LogID != null)
{
         command.Parameters.AddWithValue("@application_ex_id", logSearch.LogID);
}
else
{
        command.Parameters.AddWithValue("@application_ex_id", DBNull.Value );
}

คำถาม

  1. คุณช่วยอธิบายได้ไหมว่าทำไมจึงไม่สามารถใช้ค่า NULL จากค่า logSearch.LogID ใน Code 1 (แต่สามารถรับ DBNull ได้)

  2. มีรหัสที่ดีกว่าในการจัดการสิ่งนี้หรือไม่?

อ้างอิง :

  1. กำหนด null ให้กับ SqlParameter
  2. ประเภทข้อมูลที่ส่งคืนจะแตกต่างกันไปตามข้อมูลในตาราง
  3. ข้อผิดพลาดในการแปลงจากฐานข้อมูล smallint เป็น C # nullable int
  4. ประเด็นของ DBNull คืออะไร?

รหัส

    public Collection<Log> GetLogs(LogSearch logSearch)
    {
        Collection<Log> logs = new Collection<Log>();

        using (SqlConnection connection = new SqlConnection(connectionString))
        {
            connection.Open();

            string commandText = @"SELECT  *
                FROM Application_Ex E 
                WHERE  (E.application_ex_id = @application_ex_id OR @application_ex_id IS NULL)";

            using (SqlCommand command = new SqlCommand(commandText, connection))
            {
                command.CommandType = System.Data.CommandType.Text;

                //Parameter value setting
                //command.Parameters.AddWithValue("@application_ex_id", logSearch.LogID);
                if (logSearch.LogID != null)
                {
                    command.Parameters.AddWithValue("@application_ex_id", logSearch.LogID);
                }
                else
                {
                    command.Parameters.AddWithValue("@application_ex_id", DBNull.Value );
                }

                using (SqlDataReader reader = command.ExecuteReader())
                {
                    if (reader.HasRows)
                    {
                        Collection<Object> entityList = new Collection<Object>();
                        entityList.Add(new Log());

                        ArrayList records = EntityDataMappingHelper.SelectRecords(entityList, reader);

                        for (int i = 0; i < records.Count; i++)
                        {
                            Log log = new Log();
                            Dictionary<string, object> currentRecord = (Dictionary<string, object>)records[i];
                            EntityDataMappingHelper.FillEntityFromRecord(log, currentRecord);
                            logs.Add(log);
                        }
                    }

                    //reader.Close();
                }
            }
        }

        return logs;
    }

3
คุณหมายถึงอะไรดีกว่า? รหัส 2 เป็นวิธีที่ถูกต้องในการส่งค่า null ไปยังฐานข้อมูล
Phil Gan

คำตอบ:


152

น่ารำคาญไม่ใช่เหรอ

คุณสามารถใช้ได้:

command.Parameters.AddWithValue("@application_ex_id",
       ((object)logSearch.LogID) ?? DBNull.Value);

หรือใช้เครื่องมืออย่าง "dapper" ซึ่งจะทำให้คุณวุ่นวาย

ตัวอย่างเช่น:

var data = conn.Query<SomeType>(commandText,
      new { application_ex_id = logSearch.LogID }).ToList();

ฉันอยากจะเพิ่มวิธีการเพื่อ dapper เพื่อให้ได้IDataReader... ยังไม่แน่ใจจริงๆว่าเป็นความคิดที่ดีหรือไม่


1
ฉันกำลังคิดว่าจะขยายParametersทรัพย์สิน - นั่นคือObject?
Phil Gan

6
@Phil อืมใช่แล้วฉันเห็นว่าคุณหมายถึงอะไร ... อาจจะAddWithValueAndTreatNullTheRightDamnedWay(...)
Marc Gravell

1
@MarcGravell คุณช่วยอธิบายได้ไหมว่าเหตุใดจึงไม่สามารถใช้ NULL จากค่า logSearch.LogID ในรหัส 1 (แต่สามารถรับ DBNull ได้)
LCJ

18
@Lijo เนื่องจากnullในค่าพารามิเตอร์หมายความว่า "อย่าส่งพารามิเตอร์นี้" ผมสงสัยว่ามันคือการตัดสินใจที่ไม่ดีที่ได้ก็อบในในความเป็นจริงผมคิดว่า. ส่วนใหญ่ของDBNullคือการตัดสินใจที่เลวร้ายพื้นฐานที่ได้อบใน: stackoverflow.com/a/9632050/23354
Marc Gravell

1
@tylerH เนื่องจากกฎการร่ายโคลที่เป็นโมฆะซึ่งอาจจะลดลงใน C # 9
Marc Gravell

53

ฉันคิดว่ามันง่ายกว่าที่จะเขียนวิธีการขยายสำหรับSqlParameterCollectionที่จัดการกับค่าว่าง:

public static SqlParameter AddWithNullableValue(
    this SqlParameterCollection collection,
    string parameterName,
    object value)
{
    if(value == null)
        return collection.AddWithValue(parameterName, DBNull.Value);
    else
        return collection.AddWithValue(parameterName, value);
}

จากนั้นคุณก็เรียกมันว่า:

sqlCommand.Parameters.AddWithNullableValue(key, value);

ค่าสามารถเป็นint หรือ int?, string, bool หรือ bool?, DateTime หรือ Datetime? ฯลฯ ?
Kiquenet

3
ฉันอ่านคำตอบของ Marc และคิดว่า "ฉันคิดว่าฉันอยากจะเขียนวิธีการขยายสำหรับคอลเลกชันพารามิเตอร์" จากนั้นฉันก็เลื่อนเส้นผมลง ... (สิ่งที่ดีเกี่ยวกับวิธีการขยายคือฉันสามารถทำการค้นหา / แทนที่เพียงครั้งเดียว หลังจากและอัปเดตรหัสทั้งหมดของฉันเสร็จสิ้น)
jleach

1
ทางออกที่ดี ... ต้องกำหนดวิธีการขยายในคลาสแบบคงที่ วิธีการ: ใช้งานและเรียกใช้วิธีการขยายแบบกำหนดเอง
Chris Catignani

2
บางทีฉันอาจจะเข้าใจผิด (เป็นมือใหม่ C #) แต่คุณทำมันให้กระชับกว่านี้ไม่ได้:return collection.AddWithValue(parameterName, value ?? DBNull.Value);
Tobias Feil

1
@TobiasFeil ใช่คุณก็ทำได้เช่นกัน เป็นเพียงเรื่องของรสนิยม
AxiomaticNexus

4

ในกรณีที่คุณกำลังทำสิ่งนี้ในขณะที่เรียกขั้นตอนการจัดเก็บ: ฉันคิดว่ามันง่ายกว่าที่จะอ่านถ้าคุณประกาศค่าเริ่มต้นในพารามิเตอร์และเพิ่มเมื่อจำเป็นเท่านั้น

SQL:

DECLARE PROCEDURE myprocedure
    @myparameter [int] = NULL
AS BEGIN

ค#:


0

ปัญหาบางอย่างได้รับอนุญาตด้วยการตั้งค่า SQLDbType ที่จำเป็น

command.Parameters.Add("@Name", SqlDbType.NVarChar);
command.Parameters.Value=DBNull.Value

โดยที่คุณพิมพ์ SqlDbType.NVarChar ตั้งค่าประเภท SQL ที่จำเป็น

โดยการใช้ไซต์ของเรา หมายความว่าคุณได้อ่านและทำความเข้าใจนโยบายคุกกี้และนโยบายความเป็นส่วนตัวของเราแล้ว
Licensed under cc by-sa 3.0 with attribution required.