質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.50%
C#

C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

SQL Server

SQL Serverはマイクロソフトのリレーショナルデータベース管理システムです。データマイニングや多次元解析など、ビジネスインテリジェンスのための機能が備わっています。

Visual Studio

Microsoft Visual StudioはMicrosoftによる統合開発環境(IDE)です。多種多様なプログラミング言語に対応しています。

SQL

SQL(Structured Query Language)は、リレーショナルデータベース管理システム (RDBMS)のデータベース言語です。大きく分けて、データ定義言語(DDL)、データ操作言語(DML)、データ制御言語(DCL)の3つで構成されており、プログラム上でSQL文を生成して、RDBMSに命令を出し、RDBに必要なデータを格納できます。また、格納したデータを引き出すことも可能です。

ASP.NET

ASP.NETは動的なWebサイトやWebアプリケーション、そしてWebサービスを構築出来るようにする為、Microsoftによって開発されたウェブアプリケーション開発フレームワークです。

Q&A

1回答

2927閲覧

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

退会済みユーザー

退会済みユーザー

総合スコア0

C#

C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

SQL Server

SQL Serverはマイクロソフトのリレーショナルデータベース管理システムです。データマイニングや多次元解析など、ビジネスインテリジェンスのための機能が備わっています。

Visual Studio

Microsoft Visual StudioはMicrosoftによる統合開発環境(IDE)です。多種多様なプログラミング言語に対応しています。

SQL

SQL(Structured Query Language)は、リレーショナルデータベース管理システム (RDBMS)のデータベース言語です。大きく分けて、データ定義言語(DDL)、データ操作言語(DML)、データ制御言語(DCL)の3つで構成されており、プログラム上でSQL文を生成して、RDBMSに命令を出し、RDBに必要なデータを格納できます。また、格納したデータを引き出すことも可能です。

ASP.NET

ASP.NETは動的なWebサイトやWebアプリケーション、そしてWebサービスを構築出来るようにする為、Microsoftによって開発されたウェブアプリケーション開発フレームワークです。

0グッド

0クリップ

投稿2020/07/14 04:36

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が削除される場合がある。

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

該当のソースコード

C#

1 protected void DeleteButton_Click(object sender, EventArgs e) 2 { 3 foreach (GridViewRow row in GridView1.Rows) 4 { 5 CheckBox cb = row.FindControl("DeleteCheck") as CheckBox; 6 if (cb != null && cb.Checked) 7 { 8 int productID =Convert.ToInt32(GridView1.DataKeys[row.RowIndex].Value); 9 10 SqlConnection con = new SqlConnection(); 11 con.ConnectionString = @"接続文字列"; 12 con.Open(); 13 14 try 15 { 16 string sql = @" DELETE FROM Kaisyu_Data WHERE Data_No = " + productID + ";" + "UPDATE Kaisyu_Data SET Data_No -= 1 WHERE Data_No >= " + productID + ";"; 17 18 SqlCommand com = new SqlCommand(sql, con); 19 com.ExecuteNonQuery(); 20 } 21 catch (SqlException exc) 22 { 23 System.Diagnostics.Debug.WriteLine(exc.Message); 24 } 25 finally 26 { 27    con.Close(); 28      } 29 } 30 } 31 Response.Redirect("List.aspx"); 32 }

補足情報

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

退会済みユーザー

退会済みユーザー

2020/07/14 05:20

ここは https://teratail.com/help/question-tips に書いてあるように、「質問と回答を通してお互いに知識や情報を交換・共有する場所」「お互いの好意の上に成り立っている助け合いの場」です。 自分の知りたいこと以外は聞く耳持たないとか、そういう指摘を受けてスレッドを https://teratail.com/questions/277443 消去したりする人はここには向いてないと思いますよ。
退会済みユーザー

退会済みユーザー

2020/07/14 05:24

しつこいです。
退会済みユーザー

退会済みユーザー

2020/07/14 05:26

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

退会済みユーザー

2020/07/14 06:59

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

退会済みユーザー

2020/07/14 07:22

> ノイズも増やしてなければ悪いこともしていません。 ↓ こう言われているのですけど? 「過去に投稿した質問と同じ内容の質問」をするということが Tratail にノイズを増やして悪いことをしているとは認識できませんか? 複数のユーザーから「過去に投稿した質問と同じ内容の質問」という意見がありました teratailでは閲覧数を増やす目的などにより、過去に投稿した質問と同じ内容の質問を新たに投稿することは推奨していません。 質問について新たにわかったことや試したことがあれば、すでにある質問を編集してください。 > 気に入らないなら放っておいてください。 前のスレッドで撤退する(=放っておく)と書いたのは認識してます? 放っておいたのに「過去に投稿した質問と同じ内容の質問」をするというのはどういうこと?
退会済みユーザー

退会済みユーザー

2020/07/14 07:26

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

回答1

0

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

理由はクエリで 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 ... を削除することでしょう。

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

投稿2020/07/15 03:35

退会済みユーザー

退会済みユーザー

総合スコア0

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

まだベストアンサーが選ばれていません

会員登録して回答してみよう

アカウントをお持ちの方は

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問