AddWithValueパラメーターがNULLの場合の例外


89

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. コード1のlogSearch.LogID値からNULLを取得できない(ただしDBNullを受け入れることができる)理由を説明してください。

  2. これを処理するためのより良いコードはありますか?

参照

  1. SqlParameterにnullを割り当てます
  2. 返されるデータ型は、テーブルのデータによって異なります
  3. データベースsmallintからC#nullableintへの変換エラー
  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

回答:


148

迷惑ですね。

次を使用できます。

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 hmmm、そうです、そして私はあなたが何を意味するのかわかります...多分AddWithValueAndTreatNullTheRightDamnedWay(...)
MarcGravell

1
@MarcGravellコード1のlogSearch.LogID値からNULLを取得できない(ただしDBNullを受け入れることができる)理由を説明できますか?
LCJ 2012年

18
@Lijonullは、パラメーター値で「このパラメーターを送信しない」ことを意味するためです。私は単にで焼いてしまったことを悪い決断だった疑いが実際には、私は思います。ほとんどのはDBNullで焼いてしまった根本的に悪い決断だった:stackoverflow.com/a/9632050/23354
マルクGravell

1
@tylerHは、C#9で弱体化する可能性のあるヌルコールキャストルールのため
MarcGravell

52

SqlParameterCollectionnull値を処理するの拡張メソッドを作成する方が簡単だと思います。

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 2015

3
マークの答えを読んで、「Parametersコレクションの拡張メソッドを作成したい」と思った後、髪の毛を下にスクロールしました...(拡張メソッドの良いところは、1回の検索/置換ができることです。すべてのコード更新が完了した後)
jleach 2016年

1
優れたソリューション...拡張メソッドは静的クラスで定義する必要があります。方法:カスタム拡張メソッドを実装して呼び出す
Chris Catignani 2017年

2
たぶん私は間違っています(C#初心者のようなものです)が、次のようにもっと簡潔に行うことはできませんでした:return collection.AddWithValue(parameterName, value ?? DBNull.Value);
TobiasFeil19年

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タイプを設定します。

弊社のサイトを使用することにより、あなたは弊社のクッキーポリシーおよびプライバシーポリシーを読み、理解したものとみなされます。
Licensed under cc by-sa 3.0 with attribution required.