csvファイルの行ごとの読み込み、sqlへの書き込みのループ時のエラー

Question

csvファイルから一行読み取り、sqlへ書き込むのを繰り返したいのですが、
一行目は通るのですが、二回目のcmd.ExecuteNonQuery();を実行時にエラーが出てしまいます。

原因と解決策を教えていただけませんか？
よろしくお願いいたします。

コード

 class Ccsv
    {
        string csvLine;
        string[] csvData;
        //csvの読み込み
        public void Readcsv(string csvpass, string serverInformation)
        {
            if (File.Exists(csvpass))
            {
                StreamReader csvFile = new StreamReader(csvpass, Encoding.GetEncoding("UTF-8"));
                string constr = serverInformation;
                var con = new SqlConnection(constr);
                var cmd = con.CreateCommand();

                con.Open();
                try
                {
                    do
                    {
                        //csvファイルの読み込み
                        csvLine = csvFile.ReadLine();
                        csvData = csvLine.Split(',');

                        //sqlへ書き出し
                        cmd.CommandText = @"INSERT INTO ExcelTest (品名, 品番) VALUES (@itemName, @itemNum)";
                        cmd.Parameters.Add(new SqlParameter("@itemName", csvData[2]));             //品名
                        cmd.Parameters.Add(new SqlParameter("@itemNum", int.Parse(csvData[3])));   //品番
                        cmd.ExecuteNonQuery();
                    } while (csvFile.Peek() != -1);
                }
                finally
                {
                    con.Close();
                    csvFile.Close();
                }
            }
        }
    }

エラー文

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

Comments

[退会済みユーザー]

開発環境（OS, Visual Studio, .NET のバージョン、DB サーバは何かとそのエディションとバージョン）を書いてください。

どの行でエラーが出るのか書いてください。

Answer 1

CSV は 2 行以上あって、ループの中で、

cmd.Parameters.Add(new SqlParameter("@itemName", csvData[2]));             //品名
cmd.Parameters.Add(new SqlParameter("@itemNum", int.Parse(csvData[3])));   //品番

を繰り返しているからだと思います。

パラメータを追加するのはループの外に出して（そこでは値は追加しない）、ループの中でパラメータの Value プロパティに値を設定するようにしたらどうなりますか？

【追記】

cmd.CommandText の設定もループの外に出してください。

Comments

[FM88mhmh]

ありがとうございます。

sqlへの書き込みの部分をループの外に出して、
csvの変数は配列の配列にいれてsqlにINSERTしてみます。

Answer 2

２回目のループ時に前のループでパラメータがすでに追加されていますので同一の名前で追加できません。

パラメータをクリアするなりしてください。

Comments

[FM88mhmh]

sqlへの書き込みの部分をループから出して書き換えてみます
ありがとうございます

Answer 3

cmd.Parametersに対して
１ループ目に@itemNameと@itemNumを追加して、
２ループ目に@itemNameと@itemNumを追加しようとするから
２ループ目に同名のパラメータは既に追加されていますというエラーメッセージですよね？
そのままの意味なので、クリアして追加し直すか、既にある@itemNameのValueを変更するかでいいのではないでしょうか。

Comments

[FM88mhmh]

”@~”はC#の変数とは別物なんでしょうか？
```
b = a[i];
c[i] = b;
```
の”b”のような使い方はできないんですね。