C# DataGridviewに配置したCheckBoxを使って、チェックを入れた項目をデータベースから一括削除したい。

Question

ASP.NET Visual Studio C# .NETframework// DataGridviewにCheckBoxを配置し、チェックを入れた項目を一括削除したい。

開発環境（Windows, ASP.NET Webアプリケーション(.NET Framework4.7.2) Web Forms, Visual Studio2019, C#）

VisualStudioの.NETFrameworkでC#を用いてWebアプリケーションを作成しています。
作成するものとしては作業内容のデータ（工数など）を一覧表示するもので、機能は登録・編集・削除の三つ、画面数は1つだけです。

データベースはVisualStudioに内蔵されているSQL serverを使っています。
GridviewのテンプレートにツールボックスからCheckBoxを配置し、IDは"DeleteCheck"としてあります。

データベースのデータをGridviewで一覧表示する画面で、CheckBoxにチェックをいれ、ボタンを押すと
チェックの入っているデータがデータベースから削除され、項番を振り直して残りのデータを再表示すると
いう処理を行いたいです。

データベースの内容としては
CREATE TABLE [dbo].[Kaisyu_Data] (
[Data_No] INT NOT NULL,
[Task] NVARCHAR (50) NULL,
[Difficulty] NCHAR (10) NULL,
～～～～～
CONSTRAINT [PK_Kaisyu_Data] PRIMARY KEY CLUSTERED ([Data_No] ASC)
);
といったかたちになっています。

GridViewのIDはGridView1です。

ここでお聞きしたいのは、チェックボックスを使った一括削除に関して
以下のコードで正常に一括削除が行われない理由と改善方法です。

処理を実行した際、チェックボックスを入れたものがすべて削除されずに残ってしまいます。
エラーメッセージ等は出てきません。

以下に問題の例とソースコードを記入します。

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

例)
a
b
c
d
e
のデータのうち、b,c,dにチェックをいれ削除ボタンを押下する。→a,eだけが残るはずだがa,c,eとなりcも残る。
データベースが増えてもだいたい真ん中あたりのデータが削除されずに残る。

a
b
c
のデータのうち、b,cにチェックをいれた場合にはb,cが削除されaのみが残り結果としては正常

a
b
c
のデータのうちbのみにチェックをいれると、aが削除される場合がある。

エラーメッセージの表示や例外ではじかれることはない。

該当のソースコード

  protected void DeleteButton_Click(object sender, EventArgs e)
        {
            foreach (GridViewRow row in GridView1.Rows)
            {
                 CheckBox cb = row.FindControl("DeleteCheck") as CheckBox;
                    if (cb != null && cb.Checked) 
                    {
                        int productID =Convert.ToInt32(GridView1.DataKeys[row.RowIndex].Value);

                        SqlConnection con = new SqlConnection();
                        con.ConnectionString = @"接続文字列";
                        con.Open();

                            try
                            {
                                string sql = @" DELETE FROM Kaisyu_Data WHERE Data_No = " + productID + ";" + "UPDATE Kaisyu_Data SET Data_No -= 1 WHERE Data_No >= " + productID + ";";

                                SqlCommand com = new SqlCommand(sql, con);
                                com.ExecuteNonQuery();
                            }
                            catch (SqlException exc)
                            {
                            System.Diagnostics.Debug.WriteLine(exc.Message);
                            }
                            finally
                            {
                        　　 con.Close();
                    　　　　 }   
                    }
            }
            Response.Redirect("List.aspx");
        }

補足情報

Comments

[退会済みユーザー]

ここは https://teratail.com/help/question-tips に書いてあるように、「質問と回答を通してお互いに知識や情報を交換・共有する場所」「お互いの好意の上に成り立っている助け合いの場」です。

自分の知りたいこと以外は聞く耳持たないとか、そういう指摘を受けてスレッドを https://teratail.com/questions/277443 消去したりする人はここには向いてないと思いますよ。

[退会済みユーザー]

しつこいです。

[退会済みユーザー]

スレッドにノイズを増やす人は許せないので徹底的にやります。覚悟しておいてください。

[退会済みユーザー]

ただあなたの意志にそぐわなかっただけで、ノイズも増やしてなければ悪いこともしていません。
別の人に回答してもらえば済む話なのに、自分の思い通りにならないからといって難癖をつけ、利用を妨害しているのはあなたです。あなたが満足するためのものではありません。他の解答も見させてもらいましたが、あなたは自分の考えの押し付けがすごく、相手が受け入れなければ機嫌をそこねるということを繰り返しており、今回も同じです。
1位だかなんだか知りませんが立場を利用して全て思い通りにしようとするのではなく、気に入らないなら放っておいてください。

[退会済みユーザー]

> ノイズも増やしてなければ悪いこともしていません。

↓ こう言われているのですけど？　「過去に投稿した質問と同じ内容の質問」をするということが Tratail にノイズを増やして悪いことをしているとは認識できませんか？

複数のユーザーから「過去に投稿した質問と同じ内容の質問」という意見がありました
teratailでは閲覧数を増やす目的などにより、過去に投稿した質問と同じ内容の質問を新たに投稿することは推奨していません。
質問について新たにわかったことや試したことがあれば、すでにある質問を編集してください。

> 気に入らないなら放っておいてください。

前のスレッドで撤退する（＝放っておく）と書いたのは認識してます？　放っておいたのに「過去に投稿した質問と同じ内容の質問」をするというのはどういうこと？

[退会済みユーザー]

挙句の果てにはノイズスレッドを 2 つ残して逃げていった・・・

Answer 1

以下のコードで正常に一括削除が行われない理由と改善方法です。

理由はクエリで UPDATE ... しているところでしょう。

string sql = @" DELETE FROM Kaisyu_Data WHERE Data_No = " + productID + ";" + 
             "UPDATE Kaisyu_Data SET Data_No -= 1 WHERE Data_No >= " + productID + ";";

Data_No の連番を維持したかった？

改善方法は UPDATE ... を削除することでしょう。

主キーはレコードを一意に識別する以上のものではないと認識し、連番を維持するなどということはそもそも考えもしないのが良いと思います。