Wyjątek, gdy parametr AddWithValue ma wartość NULL


90

Mam następujący kod do określania parametrów zapytania SQL. Otrzymuję następujący wyjątek, kiedy używam Code 1; ale działa dobrze, gdy używam Code 2. W Code 2mamy czek na null, a więc if..elseblok.

Wyjątek:

Sparametryzowane zapytanie „(@application_ex_id nvarchar (4000)) SELECT E.application_ex_id A” oczekuje parametru „@application_ex_id”, który nie został dostarczony.

Kod 1 :

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

Kod 2 :

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

PYTANIE

  1. Czy możesz wyjaśnić, dlaczego nie można pobrać wartości NULL z wartości logSearch.LogID w kodzie 1 (ale jest w stanie zaakceptować DBNull)?

  2. Czy istnieje lepszy kod do obsługi tego?

Odniesienie :

  1. Przypisz null do SqlParameter
  2. Zwrócony typ danych różni się w zależności od danych w tabeli
  3. Błąd konwersji z bazy danych smallint na C # dopuszcza wartość null int
  4. Jaki jest sens DBNull?

KOD

    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
Co masz na myśli mówiąc lepiej? Kod 2 to właściwy sposób wysyłania wartości null do bazy danych.
Phil Gan,

Odpowiedzi:


149

Irytujące, prawda?

Możesz użyć:

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

Możesz też użyć narzędzia takiego jak „elegancki”, które zrobi za Ciebie cały bałagan.

Na przykład:

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

Mam pokusę, aby dodać metodę, aby uzyskać elegancki IDataReader... naprawdę nie wiem jeszcze, czy jest to dobry pomysł.


1
Myślałem o przedłużeniu Parametersnieruchomości - czy to jest Object?
Phil Gan,

6
@Phil hmmm, tak jest, i rozumiem, co masz na myśli ... możeAddWithValueAndTreatNullTheRightDamnedWay(...)
Marc Gravell

1
@MarcGravell Czy możesz wyjaśnić, dlaczego nie można pobrać wartości NULL z wartości logSearch.LogID w kodzie 1 (ale jest w stanie zaakceptować DBNull)?
LCJ,

18
@Lijo, ponieważ nullwartość parametru oznacza „nie wysyłaj tego parametru”. Podejrzewam, że była to zła decyzja, która po prostu została upieczona. W rzeczywistości myślę, że większość z nich DBNullbyła fundamentalnie złą decyzją, która została upieczona w: stackoverflow.com/a/9632050/23354
Marc Gravell

1
@tylerH z powodu zasad zerowego rzucania coale - które mogą osłabiać w C # 9
Marc Gravell

52

Łatwiej jest po prostu napisać metodę rozszerzenia dla tego, SqlParameterCollectionktóra obsługuje wartości null:

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

Następnie nazywasz to po prostu:

sqlCommand.Parameters.AddWithNullableValue(key, value);

wartość może być int lub int ?, string, bool lub bool ?, DateTime lub Datetime? itp.?
Kiquenet

3
Przeczytałem odpowiedź Marca i pomyślałem "Myślę, że wolałbym po prostu napisać metodę rozszerzającą dla kolekcji Parameters", po czym przewinąłem włos ... (fajną rzeczą w metodzie przedłużania jest to, że mogę znaleźć / zamienić po i wszystkie moje aktualizacje kodu są ukończone)
jleach

1
Świetne rozwiązanie ... Metody rozszerzające muszą być zdefiniowane w klasie statycznej. Instrukcje: wdrażanie i wywoływanie niestandardowej metody rozszerzenia
Chris Catignani

2
Może się mylę (coś w rodzaju nowicjusza w C #), ale nie mógłbyś zrobić tego bardziej zwięźle w ten sposób:return collection.AddWithValue(parameterName, value ?? DBNull.Value);
Tobias Feil

1
@TobiasFeil Tak, ty też możesz to zrobić. To tylko kwestia gustu.
AxiomaticNexus

4

Na wypadek, gdybyś robił to podczas wywoływania procedury składowanej: myślę, że łatwiej będzie to przeczytać, jeśli zadeklarujesz domyślną wartość parametru i dodasz ją tylko wtedy, gdy jest to konieczne.

SQL:

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

DO#:

int? myvalue = initMyValue();
if (myvalue.hasValue) cmd.Parameters.AddWithValue("myparamater", myvalue);

0

jakiś problem, dozwolony z Koniecznie ustawionym SQLDbType

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

gdzie SqlDbType.NVarChar wpisujesz. Koniecznie ustaw typ SQL.

Korzystając z naszej strony potwierdzasz, że przeczytałeś(-aś) i rozumiesz nasze zasady używania plików cookie i zasady ochrony prywatności.
Licensed under cc by-sa 3.0 with attribution required.