Исключение, когда параметр AddWithValue имеет значение NULL

92

У меня есть следующий код для указания параметров для SQL-запроса. Я получаю следующее исключение, когда использую Code 1: но работает нормально, когда пользуюсь Code 2. У Code 2нас есть проверка на null и, следовательно, на 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 # 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;
    }
LCJ
источник
3
Что вы имеете в виду под словом «лучше»? Код 2 - это правильный способ отправить нулевое значение в базу данных.
Фил Ган

Ответы:

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?
Фил Ган
6
@Phil хммм, да, и я понимаю, что вы имеете в виду ... может бытьAddWithValueAndTreatNullTheRightDamnedWay(...)
Марк Гравелл
1
@MarcGravell Не могли бы вы объяснить, почему он не может принять NULL из значения logSearch.LogID в коде 1 (но может принять DBNull)?
LCJ
18
@Lijo, потому nullчто значение параметра означает «не отправлять этот параметр». Я подозреваю, что это было плохое решение, которое просто прижилось. На самом деле, я думаю, что большая часть из них DBNullбыла в корне неправильным решением, которое было принято: stackoverflow.com/a/9632050/23354
Марк Грейвелл
1
@tylerH из-за правил приведения типа null coale - которые могут ослабевать в C # 9
Марк Грейвелл
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
Я прочитал ответ Марка и подумал: «Думаю, мне лучше просто написать метод расширения для коллекции параметров», затем я прокрутил волосы вниз ... (приятная вещь в методе расширения заключается в том, что я могу выполнить один поиск / замену после и все обновления моего кода сделаны)
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 #:

z00l
источник
0

какая-то проблема, разрешенная с обязательно установленным SQLDbType

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

где SqlDbType.NVarChar вы вводите. Обязательно задайте тип SQL.

user1599225
источник