削除フラグ WHERE句について

Question

現在C#で住所録を作成しており、削除フラグを使用して論理削除するという課題に悩んでいます。
テーブルカラムにはDEL_FLG作成済みです。
削除ボタン(DelBtn)をクリックすると確認メッセージ表示後にDataGridViewのその行を削除するという仕様です。
削除ボタンクリックイベント内削除処理の"UPDATE mst_address SET DEL_FLG = 1"のWHERE句の指定の仕方がわからず、現在は削除ボタンをクリックすると全てのデータのDEL_FLGが1になってしまいます。WHERE句で削除ボタンをクリックした行を指定するにはどうしたら良いでしょうか？

テーブルは以下になります。
主キーはSEQ_NOとなっており、非表示の状態です。
一行ごとに削除ボタンがあるDataGridViewで、ボタンを押下した行を削除するという仕様です。
よろしくお願い致します。

CREATE TABLE `mst_address` (
 `SEQ_NO` int(11) NOT NULL AUTO_INCREMENT COMMENT '管理番号',
 `NAME1` varchar(10) NOT NULL COMMENT '氏名1',
 `NAME2` varchar(10) NOT NULL COMMENT '氏名2',
 `ZIP1` varchar(3) NOT NULL COMMENT '郵便番号1',
 `ZIP2` varchar(4) NOT NULL COMMENT '郵便番号2',
 `ADDRESS3` varchar(100) NOT NULL COMMENT '住所3',
 `ADDRESS1` varchar(100) DEFAULT NULL COMMENT '住所1',
 `ADDRESS2` varchar(100) DEFAULT NULL COMMENT '住所2',
 `TEL` varchar(20) DEFAULT NULL COMMENT '電話番号',
 `INS_USER` varchar(10) NOT NULL COMMENT '登録者',
 `INS_DATETIME` datetime NOT NULL COMMENT '登録日',
 `UPD_USER` varchar(10) NOT NULL COMMENT '更新者',
 `UPD_DATETIME` datetime NOT NULL COMMENT '更新日',
 `DEL_FLG` bit(1) NOT NULL COMMENT '削除フラグ',
 PRIMARY KEY (`SEQ_NO`)
) ENGINE=InnoDB AUTO_INCREMENT=10 DEFAULT CHARSET=utf8

追記
主キーSEQ_NOを追加(テーブルには非表示)し、WHERE句に指定した修正後ソースコードです。
dataGridView1.CurrentRow.Cells[1].ValueでSEQ_NOの値が取得できることは確認しています。

namespace AddressList
{
    public partial class AddressList : Form
    {
        public AddressList()
        {
            InitializeComponent();
        }

        public void 住所一覧_Load(object sender, EventArgs e)
        {
            // MySQLへの接続
            using (MySqlCommand command = new MySqlCommand())
            {
                MySqlConnection conn = new MySqlConnection();
                conn.ConnectionString = "server=サーバ名; user id=ID; Password=PW; database=address_note";
                
                //DBとの接続オープン
                conn.Open();
               
                //データを保存するテーブル作成
                DataTable dt = new DataTable();

                //SQL文と接続情報を引数に、データアダプターを作成
                MySqlDataAdapter da = new MySqlDataAdapter("SELECT **SEQ_NO,** NAME1,NAME2,ZIP1,ZIP2,ADDRESS1,ADDRESS2,ADDRESS3,TEL FROM mst_address WHERE DEL_FLG = 0", conn);


                //SQL文で指定したデータをDataTableに格納
                da.Fill(dt);

                //表示させる
                dataGridView1.DataSource = dt;

                //DBとの接続クローズ
                conn.Close();

                
               ** dataGridView1.Columns[1].Visible = false;       //SEQ_NOを非表示にする**
                //横幅指定・削除ボタン配置
                dataGridView1.Columns[2].Width = 50;
                dataGridView1.Columns[3].Width = 45;
                dataGridView1.Columns[4].Width = 40;
                dataGridView1.Columns[5].Width = 40;
                dataGridView1.Columns[6].Width = 70;
                dataGridView1.Columns[7].Width = 42;
                dataGridView1.Columns[8].Width = 40;
                dataGridView1.Columns[9].Width = 90;
                dataGridView1.Columns["DelBtn"].DisplayIndex = 9;

                //氏名昇順でソートする
                dataGridView1.Sort(dataGridView1.Columns[2], ListSortDirection.Ascending);
            }
        }

        //検索ボタン
        private void Button1_Click_1(object sender, EventArgs e)
        {
            {
                //テキストボックスの文字取得
                string prefecture = AddressSrc.Text;
                string name = NameSrc.Text;

                //未入力で検索押下時
                if (prefecture == "" && name == "")
                {
                    MessageBox.Show("氏名または都道府県を入力してください");
                }
                else if (prefecture != "")  //都道府県検索
                {
                    //MySQLへ接続
                    MySqlConnection conn = new MySqlConnection();
                    conn.ConnectionString = Properties.Settings.Default.address_noteConnectionString;
                    MySqlCommand sqlCommand = new MySqlCommand(@"SELECT **SEQ_NO**, NAME1, NAME2, ZIP1, ZIP2, ADDRESS1, ADDRESS2, ADDRESS3, TEL FROM mst_address WHERE ADDRESS1 = @ADDRESS1 AND DEL_FLG = 0", conn);

                    //データを保存するテーブル作成
                    DataTable dt = new DataTable();

                    try
                    {
                        //DBとの接続オープン
                        conn.Open();

                        sqlCommand.Parameters.Add(new MySqlParameter(@"ADDRESS1", prefecture));

                        MySqlDataReader reader = sqlCommand.ExecuteReader();

                        dt.Load(reader);

                        dataGridView1.DataSource = dt;

                        reader.Close();
                    }
                    catch
                    {
                        MessageBox.Show("検索エラーです");
                    }
                    finally
                    {
                        conn.Close();
                    }
                }
                else if (name != "")    //氏名検索
                {
                    //MySQLへ接続
                    MySqlConnection conn = new MySqlConnection();
                    conn.ConnectionString = Properties.Settings.Default.address_noteConnectionString;
                    MySqlCommand sqlCommand = new MySqlCommand(@"SELECT **SEQ_NO**, NAME1, NAME2, ZIP1, ZIP2, ADDRESS1, ADDRESS2, ADDRESS3, TEL FROM mst_address WHERE NAME1 LIKE @NAME1 AND DEL_FLG = 0", conn);

                    //データを保存するテーブル作成
                    DataTable dt = new DataTable();

                    try
                    {
                        //DBとの接続オープン
                        conn.Open();

                        sqlCommand.Parameters.Add(new MySqlParameter(@"NAME1", @"%" + name + @"%"));

                        MySqlDataReader reader = sqlCommand.ExecuteReader();

                        dt.Load(reader);

                        dataGridView1.DataSource = dt;

                        reader.Close();
                    }
                    catch
                    {
                        MessageBox.Show("検索エラーです");
                    }
                    finally
                    {
                        conn.Close();
                    }
                }
            }            
        }
        

        // 終了ボタン
        private void Button2_Click(object sender, EventArgs e)
        {
            this.Close();
        }

        //削除ボタン
        private void DataGridView1_CellContentClick(object sender, DataGridViewCellEventArgs e)
        {
            // 削除ボタンの列かどうかを確認 
            if (e.ColumnIndex == this.dataGridView1.Columns["DelBtn"].Index)
            {
                if (DialogResult.Yes == MessageBox.Show("削除してもよろしいですか？", "削除",
                    MessageBoxButtons.YesNo, MessageBoxIcon.Question))
                {
                    // 削除 
                    //this.dataGridView1.Rows.RemoveAt(e.RowIndex);

                    MySqlConnection conn = new MySqlConnection();
                    conn.ConnectionString = Properties.Settings.Default.address_noteConnectionString;
                    conn.Open();
                    MySqlCommand sqlCommand = new MySqlCommand("UPDATE mst_address SET DEL_FLG = 1** WHERE SEQ_NO = dataGridView1.CurrentRow.Cells[1].Value**", conn);
                    DataTable dt = new DataTable();
                    MySqlDataReader reader = sqlCommand.ExecuteReader();
                    dt.Load(reader);
                    dataGridView1.DataSource = dt;
                    reader.Close();

                    MessageBox.Show("削除完了しました。");

                }
                else
                {
                    this.Close();
                }
            }
        }
    }
}
 
 

Comments

[退会済みユーザー]

主キーはないのですか？ テーブルの構造はどうなっているのですか？

[Orlofsky]

質問に CREATE TABLE を https://teratail.com/help/question-tips#questionTips3-7 の [コード] で載せては？

[ICCI]

遅くなりましたが、CREATE TABLEを追記せさせて頂きました。よろしくお願い致します。

Answer 1

mst_addressテーブルの仕様が分からないので
NAME1,NAME2,ZIP1,ZIP2,ADDRESS1,ADDRESS2,ADDRESS3,TEL
のうちキーとなるものがあるのであれば、それを指定すればいいのではないですか？
もしキーがないのでしたら、テーブル仕様としてキーを追加した方がいいと思います。

追記

それであれば、SEQ_NOも読み込み、DataGridView上では非表示にしてください。
そして、WHERE句にSEQ_NOを条件として指定するようにしてください。

Comments

[ICCI]

ありがとうございます。SEQ_NOを読み込み、WHERE句に設定するとWHERE句の二行下のMySqlDataReader reader = sqlCommand.ExecuteReader();というところでエラーがひっかかり、"You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '[1].Value' at line 1"と出てしまいます。
dataGridView1.CurrentRow.Cells[1].ValueでSEQ_NOが取得できることは確認済なのですが、
WHERE SEQ_NO = dataGridView1.CurrentRow.Cells[1].Valueとして指定することは出来ないのでしょうか？
参考までに修正後ソースコードも追記させて頂きました。

[YAmaGNZ]

文字列と変数の違いを理解しましょう。
氏名での検索が出来ているのですから、分かるはずです。
また、こちら
https://www.moonmile.net/blog/archives/1298
が参考になるかと思います。

[ICCI]

ありがとうございます！参考にさせて頂き、無事に完成させることができました！

Answer 2

UPDATE mst_address SET DEL_FLG = 1 WHERE SEQ_NO = dataGridView1.CurrentRow.Cells[1].Value

そのような UPDATE 文は文法的に間違っているのでダメです。クエリをパラメータ化してそれの Value に代入するようにしないと。

それより、WEHRE 句なして単純に UPDATE すればよさそうですけど。

まず、初期画面は以下の SELECT クエリで DataTabe に読み込んで、それを DataGridView にバインドして表示するのですよね？

SELECT SEQ_NO, NAME1,NAME2,ZIP1,ZIP2,ADDRESS1,ADDRESS2,ADDRESS3,TEL FROM mst_address WHERE DEL_FLG = 0

そして、ユーザーが DataGridView の画面を見て、削除したいレコードの DEL_FLG 列のチェックボックスにチェックマークを入れるのですよね？

そうであれば、ユーザーが編集完了後［削除］ボタンをクリックしたとき、そのイベントハンドラ DataGridView1_CellContentClick でユーザーが編集したすべての行を UPDATE すれば良いはずです。

そのような機能は DataSet/DataTable + DataAdapter に備わっているので、それを利用すれば容易にできると思います。

以下の記事の「非接続型のデータ更新」のセクションの図１と図２を見てください。見るのはとりあえず図だけでいいです。興味が湧いたら説明も読んでください。

DB 設計者のための明解 ADO.NET 第 1 回
https://docs.microsoft.com/ja-jp/previous-versions/cc482903(v=msdn.10)

そのような構造にすれば、ユーザーが DataGridView を操作・編集した結果は上の記事の図１にあるように DataSet / DataTable に反映されます。もちろん DEL_FLG にチェックを入れた結果もです。

編集作業終了後、ユーザーが［削除］ボタンをクリックすることにより DataAdapter に Update メソッドをかければ、上の記事の図２のように DataAdapter によって SQL が自動的に発行され、SQL Server のテーブルが更新されます。

DB が SQL Server の場合ですが（MySQL の場合も Visual Studio に同等な機能を追加できます）、以下のチュートリアル、

10 行でズバリ !! 非接続型のデータ アクセス (ADO.NET) (C#)
https://code.msdn.microsoft.com/windowsdesktop/10-ADONET-C-cbfe7688

のように Visual Studio のデータソース構成ウィザードを利用して型付 DataSet + TableAdapter を作って、それを利用してアプリを作ると、

DataGridview ⇔ BindingSource/BindingNavigator ⇔　DataSet/DataTable ⇔ DataAdapter ⇔ DB

と言う構造、すなわち以下のページの図のような構造のアプリが、ほとんど自分でコードを書くこと無しに作れます。

Windows フォーム アプリケーションでのデータへの接続
https://docs.microsoft.com/ja-jp/previous-versions/wxt2cwcc(v=vs.120)

上に紹介したチュートリアルは 2 つのテーブルを階層更新するために少々複雑になっていますが、単一テーブルですともっと簡単で、操作に慣れると以下のようなアプリが 5 分もかからず作れるはずです。

今回のケースですと、削除は、DataGridView に DEL_FLG 列があるはずなのでそれにユーザーがチェックを入れると、内部の DataTable の当該 DEL_FLG 列が健康されその行には図１でいう Modified マークがつきます。その後、フロッピーディスクのアイコンをクリックすれば DB のテーブルに編集結果が反映されます。

自力で一からコードを書く場合のような種々の問題に悩むことはなくなり、開発工数は激減するはずです。保守工数も減るはずです。

お試しください。

【追伸】

MySQL で Visual Studio の TableAdapter 構成ウィザードを利用する場合、Visual Studio のウィザードが使う MySql.Data.dll のバージョン 6.9.8 と、GAC の MySql.Data.dll のバージョン 6.9.9 に不一致があって、それが原因でウィザードが期待通り動かないということがありましたので注意してください。

対処方法はバージョンを一致察せることです。詳しくは以下の記事を見てください。

MySQL での TableAdapter 構成ウィザード不具合
http://surferonwww.info/BlogEngine/post/2018/02/10/tableadapter-configuration-wizard-does-not-work-properly-for-mysql.aspx

上記は MySQL Installer 5.7.17 の話で、それ以外でどうなっているか（直ったのか、変わってないのか、さらに悪くなっているのか等）は不明です。

Comments

[ICCI]

ご丁寧に教えてくださり、ありがとうございます！お陰様で無事に完成させることができました！

Answer 3

テーブルにユーザーIDみたいなものはありますか？ないなら一意となるIDを追加した方がいいと思います？
ユーザーID列を一覧に追加して、表示したくないなら非表示にして、削除するときにその列を参照すればいいのではないのでしょうか