[C#]SQLParameterの変数（@xx）をインクリメントしたい

Question

前提・実現したいこと

SQLサーバとC#にて自社金属加工業で使うソフトの開発を行っています。
素人が作った切り貼りだらけの読みにくいコードで申し訳ありません。

問題となっている処理の流れは以下のとおりです。
１．条件に合致する社名を抽出し、リストへ詰め込む
２．１に合致する件数分３の処理をループさせる
３．１で抽出した社名を条件に合致する品目を抽出しリストへ詰め込む（ここで問題発生）
４．リストの内容を元に後処理を実行（未実装）

発生している問題・エラーメッセージ

System.Data.SqlClient.SqlException: '変数名 '@results' は既に宣言されています。変数名は、クエリ バッチまたはストアド プロシージャ内で一意にしてください。'

該当のソースコード

        private void button4_Click(object sender, EventArgs e)
        {
            String nyuryokudate = dateTimePicker2.Text;

            {
                var results = new List<string>();
                string constr = @"接続…";
                using (var connection = new SqlConnection(constr))
                using (var command = connection.CreateCommand())

                    try
                    {
                        connection.Open();
                        command.CommandText = "select 会社 from zairyoTBL where 入力日 = @nyuryokudate and フラグ is null group by 会社";
                        command.Parameters.Add(new SqlParameter("@nyuryokudate", nyuryokudate));
                        SqlDataReader sdr = command.ExecuteReader();
                        while (sdr.Read() == true)
                        {
                            results.Add(sdr["会社"].ToString());
                        }
                        sdr.Close();
                    }
                    catch (Exception exception)
                    {
                        Console.WriteLine(exception.Message);
                        return;
                    }
                    finally
                    {
                        //コネクションクローズ
                        connection.Close();
                    }
                int count = results.Count();

                var results2 = new List<string>();
                string constr2 = @"接続…";
                using (var connection2 = new SqlConnection(constr2))
                using (var command2 = connection2.CreateCommand())

                    try
                    {
                        //コネクションオープン
                        connection2.Open();
                        //SQL生成
                        command2.Parameters.Add(new SqlParameter("@nyuryokudate2", nyuryokudate));

                        for (int i=0;i<=count;++i)
                        {
                            command2.Parameters.Add(new SqlParameter("@results", results[i]));//ここの@resultsがループの際、重複しているエラーが出ている。
                            command2.CommandText = "select 材質１,径１,長さ１,数量１,単価１,硬度１,その他１,数量２,単価２,会社 from zairyoTBL where 入力日=@nyuryokudate2 and 会社=@results and フラグ is null";
                            SqlDataReader sdr2 = command2.ExecuteReader();
                            while (sdr2.Read() == true)
                            {
                                results2.Add(sdr2["材質１"].ToString());
                                results2.Add(sdr2["径１"].ToString());
                                results2.Add(sdr2["長さ１"].ToString());
                                results2.Add(sdr2["数量１"].ToString());
                                results2.Add(sdr2["単価１"].ToString());
                                results2.Add(sdr2["硬度１"].ToString());
                                results2.Add(sdr2["その他１"].ToString());
                                results2.Add(sdr2["数量２"].ToString());
                                results2.Add(sdr2["単価２"].ToString());
                                results2.Add(sdr2["会社"].ToString());
                            }
                            Console.WriteLine("[{0}]", string.Join(", ", results2));
                            sdr2.Close();

                        }
                    }
                    catch (Exception exception)
                    {
                        Console.WriteLine(exception.Message);
                        return;
                    }
                    finally
                    {
                        //コネクションクローズ
                        connection2.Close();
                    }

試したこと

ループの際のSqlParameterの変数をインクリメントできれば良いと思い
SqlParameter("@results"+[i],results[i])
としてみたりしたのですが、文法が違うようでうまくいきませんでした。

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Answer 1

発生している問題、

System.Data.SqlClient.SqlException: '変数名 '@results' は既に宣言されています。

に対する直接の解決策のレスではなくて（それはすでに回答されているようですので）、こうした方がよさそうという提案ですが・・・

１．条件に合致する社名を抽出し、リストへ詰め込む
２．１に合致する件数分３の処理をループさせる
３．１で抽出した社名を条件に合致する品目を抽出しリストへ詰め込む（ここで問題発生）

上の 1 の条件というのは zairyoTBL テーブルの「入力日」と「フラグ」ですよね。

ということは、要するに、zairyoTBL テーブルから「入力日」と「フラグ」が特定の条件（where 入力日 = @nyuryokudate and フラグ is null）でレコードを抽出し、抽出したレコードを「会社」ごとに並べてリスト（List<string> 型のオブジェクト）に詰め込みたいのですよね？

そうであれば、上の 1 ～ 3 のステップを踏む必要はなくて、SELECT クエリを以下のようにして一発で希望のレコードは抽出できるのではないですか？

select 材質１,径１, ... ,会社 from zairyoTBL where 入力日=@nyuryokudate2 and フラグ is null order by 会社

私の思い違いがあれば指摘ください。

Comments

[superhiro]

>抽出したレコードを「会社」ごとに並べてリスト（List<string> 型のオブジェクト）に詰め込みたいのですよね？
はい。Listに詰め込む目的としては、次の処理で会社ごとの明細をSQLで抽出し、加工して会社ごとに注文書みたいな形で出力したいために、条件となる社名が必要かと思い一次処理として行いました。
もっと良いやり方があるのかもしれませんが、持っている知識でできるやり方で作ってみました。

[退会済みユーザー]

> 持っている知識でできるやり方で作ってみました。

私が提案したやり方は、上のレスの「要するに」というところの理解が正しければという条件付きですが、今質問に書かれているよりはるかに簡単に、もちろん持っている知識で、必要なレコードを抽出できると思うのですが。何か条件に合わないのですかね？

[superhiro]

私が難しく考えているだけかもしれませんが、会社ごとのデータに後処理で成形する際、SQL結果がその会社分のデータになっていないとだめなんじゃないかと思っているためです。提案頂いた内容ですと、会社ごとに並びますが、複数の会社分のデータが帰ってきてしまいます。すると、後処理でまた会社ごとに分割しないといけないのではないでしょうか。

[退会済みユーザー]

上のレスの「会社ごとに注文書」の例で言うと、実際は質問のコードの for ループの中の最後に注文書を作成するコードがあって、ループが一回回るたびに（即ち会社ごとに）注文書を作りたいということだったようですね。そこが読めてなかったです。すみません。

[superhiro]

はい、そのようにしたいと思っています。
だた、今調べていたらListの中身をフィルタして抽出できるようですので、ご提示頂いた方法でList化して、条件を与えて会社ごとに抽出もできそうです。
ありがとうございました。

Answer 2

command2.Parameters.Add(new SqlParameter("@results", results[i]));
これは「@results」という名前のパラメータを新しく作成しcommand2のパラメータに追加するということです。
ですので、すでに前のループで「@results」という名前のパラメータは追加されているので「同じ名前のパラメータは追加できません」とエラーとなります。

ではどうやってパラメータの値を変更するかですが

command.Parameters["@results"].Value = i;

というような形で値を代入できます。

これはおせっかいですが、
「インクリメント」とは一般的に+1していくことを指す言葉で、値を変更するという意味ではないです。
インクリメント(ウィキペディア)

Comments

[superhiro]

プログラミングを始めたばかりで用語もままならず、伝わりづらい表現もあったかと思いますが、ご回答頂きありがとうございます。

ループ1周目のSQLを
```C#
command2.CommandText = "select 材質１,径１,長さ１,数量１,単価１,硬度１,その他１,数量２,単価２,会社 from zairyoTBL where 入力日=@nyuryokudate2 and 会社='results[0]' and フラグ is null";
```
2周目のSQLを
```C#
command2.CommandText = "select 材質１,径１,長さ１,数量１,単価１,硬度１,その他１,数量２,単価２,会社 from zairyoTBL where 入力日=@nyuryokudate2 and 会社='results[1]' and フラグ is null";
```

のようにしたいと思い、@resultsの名前に連番をつけて重複しないようにし、それに対してresults[0]やresults[1]を代入すれば良いかなぁと思っていました。
そのため、連番の意味でインクリメントと表現していました。

さて、回答いただいた内容ですが、
```C#
command.Parameters["@results"].Value = i;
```
これは、@resultsのパラメータをiにする。という意味でしょうか？

```C#
command2.Parameters.Add(new SqlParameter("@results", results[i]));
command2.Parameters["@results"].Value = i;
command2.CommandText = "select 材質１,径１,長さ１,数量１,単価１,硬度１,その他１,数量２,単価２,会社 from zairyoTBL where 入力日=@nyuryokudate2 and 会社=@results and フラグ is null";
```
これで、上述のような処理になるのかがあまり理解できていません。
処理は先に進みましたが、別のエラーが出て対応しているところです。

[YAmaGNZ]

まず、新しいパラメータを追加するのはループの外にしなければ、2重定義となってしまいます。
これについては対応されましたでしょうか？

また、私の回答はそのままコピペしても意味がありません。
あくまで@resultsというパラメータの値をiにするという例です。
実際にどのような値にするかはご自身で決定してください。
また、Valueプロパティについては
SqlParameter.Value プロパティ（https://docs.microsoft.com/ja-jp/dotnet/api/system.data.sqlclient.sqlparameter.value?view=netframework-4.8）をご覧ください。

[superhiro]

Addをループの外に出して、パラメーター値の変更内容をresults[i]にすることで思っていた処理にすることができました。ただ、予想していた出力結果になっていなかったので、その他改修したいと思います。
貴重なお時間ありがとうございました。