C#でSQL文BETWEENの書き方

Question

int id = 3;
int minAge = 20;
int maxAge = 50;

cmd.CommandText = "INSERT INTO [dbo].[Table2] ([ID], [年齢])" +
"SELECT [ID], [年齢] FROM [dbo].[Table]" +
"WHERE ID = N'" + id + "'";
if(minAge != null && maxAge != null)
{
     cmd.CommandText += "AND 年齢 = N'BETWEEN" + minAge + "AND" + maxAge + "'";
}

上記のような条件でBETWEENの構文を適用したいのですがうまくいきません。
C#では文字列として読み込んでしまうため、エラーが出てくれたらいいのですが・・・
つまり構文が間違っているのかなと思い色々調べてみましたができませんでした。

cmd.CommandText = "INSERT INTO [dbo].[Table2] ([ID], [年齢])" +
"SELECT [ID], [年齢] FROM [dbo].[Table]" +
"WHERE ID = N'" + id + "'";
if(minAge != null && maxAge != null)
{
     cmd.CommandText += "AND 年齢 = N'BETWEEN" + minAge + "'AND'" + maxAge + "'";
}

このように最後の列のANDの両端に'を追加するとエラーが出てしまい実行できません。
{"An expression of non-boolean type specified in a context where a condition is expected, near '50'."}
と出てきます。

どこが間違っているのか教えてください。
intには関係の無い文字化けを防ぐNを記載していますが、これはありでお願いいたします。
よろしくお願いいたします。

Answer 1

直接の回答ではないです。

SQL インジェクション防止とパフォーマンス向上のためクエリは必ずパラメータ化しましょう。

すべての話はそれから（＝パラメータ化してないクエリは議論するに値しない）と言っても過言ではないと個人的には思います。

パラメータ化クエリ
http://surferonwww.info/BlogEngine/post/2012/02/02/Parameterized-query.aspx

パラメータ化には SQL インジェクション防止とパフォーマンス向上以外にも副次的な効用があります。例えば下記：

パラメータ化の副次的な効用
http://surferonwww.info/BlogEngine/post/2016/06/04/use-parameterized-query-to-avoid-unexpected-character-corruption.aspx

今回の質問者さんの問題にも、パラメータ化クエリを使っていれば悩むことはなかったと思います。

Answer 2

まずは実行可能なSQLを書いて、

INSERT INTO [dbo].[Table2] ([ID], [年齢]) SELECT [ID], [年齢] FROM [dbo].[Table] WHERE ID = 1 AND 年齢 BETWEEN 10 AND 20

C#の文字列にして、

"INSERT INTO [dbo].[Table2] ([ID], [年齢]) SELECT [ID], [年齢] FROM [dbo].[Table] WHERE ID = 1 AND 年齢 BETWEEN 10 AND 20"

パラメータクエリなり

"INSERT INTO [dbo].[Table2] ([ID], [年齢]) SELECT [ID], [年齢] FROM [dbo].[Table] WHERE ID = @ID AND 年齢 BETWEEN @MINAGE AND @MAXAGE"

べた書きパラメータに

"INSERT INTO [dbo].[Table2] ([ID], [年齢]) SELECT [ID], [年齢] FROM [dbo].[Table] WHERE ID = " + id + " AND 年齢 BETWEEN " + minAge + " AND " + maxAge

書き換えてみたらどこがおかしいか切り分けできるのではないでしょうか。
+演算子で改行したいのであればその後で。

Answer 3

まずSQLとしてBetween句の使い方おかしいですよね。

参考：SQL BETWEEN演算子

また、文字列連結をしてますが、行末および行頭に空白文字が入っていないため、連結結果がSQLとして文法エラーになる可能性が高いです。

"WHERE ID = N'" + id + "'";

ここの行末に「;」がついてるので、C#としてはここで文字列連結行は終了してますね。

Comments

[退会済みユーザー]

すみません。自分が書いていたコードを質問用に作り直しているのですが手違いがありました。
正しくは編集し直しましたのでもう一度見ていただければ幸いです。
お手数をおかけして申し訳ございません。

Answer 4

; で文字列が閉じてしまっています。

"INSERT INTO [dbo].[Table2] ([ID], [年齢])" +
"SELECT [ID], [年齢] FROM [dbo].[Table]" +
"WHERE ID = N'" + id + "'"; // ←これ
"AND 年齢 = N'BETWEEN" + minAge + "'AND'" + maxAge + "'";

【追記】

余談ですが、この SQL を入れた変数を一度画面やログで出力してみると良いです。識別が繋がってしまっていてちゃんと動いていない、といった物が発見出来ます。あとこれは好き好きですが

  var sql = @"
  SELECT
    *
  FROM
    TABLE1
  ";

と書く方法もあります。(嫌いな人もいます)

【さらに追記】

BETWEEN は BETWEEN ここから AND ここまで という記述にならないといけません。ちなみに年齢(Age)ですので文字列ではないと思われます。なのでなぜ N を使っているか分かりません。

		cmd.CommandText = "INSERT INTO [dbo].[Table2] ([ID], [年齢]) " +
			"SELECT [ID], [年齢] FROM [dbo].[Table] " +
			"WHERE ID = N'" + id + "'";
		if(minAge != null && maxAge != null)
		{
			cmd.CommandText += " AND 年齢 = BETWEEN " + minAge + " AND " + maxAge;
		}

が正しいと思いますが、如何でしょうか。
P.S. コードブロックでは sql でなく csharp をお使い下さい。

Comments

[退会済みユーザー]

すみません。自分が書いていたコードを質問用に作り直しているのですが手違いがありました。
正しくは編集し直しましたのでもう一度見ていただければ幸いです。
お手数をおかけして申し訳ございません。

[退会済みユーザー]

質問用に作り変えているので実際はint型ではなくstring型で行っているためNを付けています。
しかし、言われてみればstring型をint型にキャストしてしまえば同じことですね！
なのでint型にキャストしてmattnさんのコードで試してみましたがやはりダメでした。
{"Invalid column name 'BETWEEN20AND50'."}と出てきました。
cmd.CommandText += " AND 年齢 = 'BETWEEN " + minAge + " AND " + maxAge + "'";
にするとエラーは出ませんが結果を得られませんでした。
何が原因なのかよくわかりません。。。

[mattn]

シングルクォート消して下さい。

[mattn]

あと = も要りません。 年齢 BETWEEN 15 AND 25 となる文字列を作って下さい。

[退会済みユーザー]

cmd.CommandText += "AND 年齢 BETWEEN" + minAge + "AND" + maxAge;
でやると下記のエラーが出てしましました。。
{"An expression of non-boolean type specified in a context where a condition is expected, near 'BETWEEN20AND50'."}
何度もすみません。

[mattn]

えと、回答に書いた内容をちゃんとコピーして使って下さい。空白が消えてしまっていますよ。回答では BETWEEN の後や AND の前後に空白が入っています。落ち着いて、ゆっくり。

[退会済みユーザー]

原因は空白でした。。。。
本当に助かりました。
長々と私にお付き合いしていただきありがとうございました。

[mattn]

ベストアンサーになってしまいましたが、SurferOnWww さんの回答があるべき姿ですのでご検討下さい。