Виняток, коли параметром AddWithValue є NULL


90

У мене є наступний код для вказівки параметрів для запиту 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 у коді 1 (але може прийняти DBNull)?

  2. Чи є кращий код для обробки цього?

Довідково :

  1. Присвоїти null параметру SqlParameter
  2. Повернутий тип даних залежить від даних у таблиці
  3. Помилка перетворення з бази даних smallint у інтерактивний тип C # з нульовим значенням
  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 - це правильний спосіб надсилання нульового значення до бази даних.
Філ Ган

Відповіді:


149

Прикро, чи не так.

Ви можете використовувати:

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

Я маю спокусу додати метод, щоб отримати більше IDataReader... ще не впевнений, чи це гарна ідея.


1
Я думав про продовження Parametersвласності - це Object?
Філ Ган

6
@Phil hmmm, так це, і я розумію, що ти маєш на увазі ... можливоAddWithValueAndTreatNullTheRightDamnedWay(...)
Марк Гравелл

1
@MarcGravell Чи можете ви пояснити, чому він не може взяти значення NULL з logSearch.LogID у коді 1 (але може прийняти DBNull)?
LCJ

18
@Lijo, оскільки nullзначення параметра означає "не надсилати цей параметр". Я підозрюю , що це було погане рішення , що просто був запечений справді, я думаю. Більшість з DBNullбуло принципово погане рішення , яке отримав запечений в: stackoverflow.com/a/9632050/23354
Марк Gravell

1
@tylerH через правила нульового розкладання вугілля - які можуть слабшати в C # 9
Марк Гравелл

52

Мені простіше просто написати метод розширення для того, 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 ?, рядок, bool або bool ?, DateTime або Datetime? тощо?
Кікенет,

3
Я прочитав відповідь Марка і подумав: "Я вважаю за краще просто написати метод розширення для збору параметрів", потім я прокрутив волосся ... (приємно в методі розширення, що я можу зробити одне знаходження / заміну після і всі мої оновлення коду зроблені)
jleach

1
Чудове рішення ... Методи розширення повинні бути визначені в статичному класі. Як: впровадити та викликати власний метод розширення
Кріс Катіньяні,

2
Можливо, я помиляюся (на кшталт новачка на C #), але чи не міг би ти зробити це коротше так:return collection.AddWithValue(parameterName, value ?? DBNull.Value);
Тобіас Фейл,

1
@TobiasFeil Так, ви могли б зробити це теж. Це лише питання смаку.
AxiomaticNexus

4

На всякий випадок, якщо ви робите це під час виклику збереженої процедури: я думаю, це легше читати, якщо ви оголосите значення параметра за замовчуванням і додасте його лише за потреби.

SQL:

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

C #:

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

0

деяка проблема, дозволена з Обов’язково встановленим SQLDbType

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

де SqlDbType.NVarChar ви вводите. Обов’язково встановити тип SQL.

Використовуючи наш веб-сайт, ви визнаєте, що прочитали та зрозуміли наші Політику щодо файлів cookie та Політику конфіденційності.
Licensed under cc by-sa 3.0 with attribution required.