落ちる原因が分からない（windowsフォーム）

Question

お世話になります。

パスワード管理するプログラムを実装しているのですが、
Update部分で落ちてしまい原因がわかりません。

DataGridViewのチェックをつけた、行のみ削除を行いたいです。

エラー 138行目
型 'System.NullReferenceException' のハンドルされていない例外が PassKanriSystem.exe で発生しました

追加情報:オブジェクト参照がオブジェクト インスタンスに設定されていません。

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows.Forms;
using System.Data.SQLite;

namespace WindowsFormsApplication4
{
    public partial class Itiran : Form
    {
        public Itiran()
        {
            InitializeComponent();

        }
        private void btnsintou_Enter(object sender, EventArgs e)
        {
            //新規登録ボタンを押下すると登録画面に遷移
            Touroku Touroku = new Touroku();
            Touroku.Show();
            this.Visible = false;
        }
        private DataTable datatable = new DataTable();
        protected override void OnLoad(EventArgs e)
        {
            dataGridView1.DataSource = datatable;
            base.OnLoad(e);
            DataGridViewCheckBoxColumn column = new DataGridViewCheckBoxColumn();
            dataGridView1.Columns.Add(column);
        }
        //テーブルの内容を表示
        private void buttonkensaku_Click(object sender, EventArgs e)
        {

            //検索条件がない場合
            if (txtSiteNm.Text == "")
            {
                using (SQLiteConnection con = new SQLiteConnection("Data Source = C:\\Users/Desktop/WindowsFormsApplication4 /myfriend.sqlite3"))

                using (SQLiteDataAdapter adapter = new SQLiteDataAdapter("select * from PASS_KANRI where DEL_FLG = 0", con))
                {
                    //データグリッドビューのクリア
                    datatable.Clear();

                    dataGridView1.DataSource = datatable;
                    adapter.Fill(datatable);
                    //DELFLGを非表示に
                    dataGridView1.Columns[7].Visible = false;

                    //行の幅を自動で調整する
                    dataGridView1.AutoSizeRowsMode = DataGridViewAutoSizeRowsMode.AllCells;
                }
            }
            //検索条件がある場合
            else
            {
                string dbConnectionString = "Data Source = C:\\Users/Desktop/WindowsFormsApplication4 /myfriend.sqlite3";
                using (SQLiteConnection cn = new SQLiteConnection(dbConnectionString))
                {
                    cn.Open();
                    using (SQLiteTransaction trans = cn.BeginTransaction())
                    {
                        SQLiteCommand cmd = cn.CreateCommand();
                        //SELECT文
                        cmd.CommandText = "SELECT * from PASS_KANRI where SiteNm = @SiteNm and DEL_FLG = 0";

                        //パラメータセット
                        cmd.Parameters.Add("SiteNm", System.Data.DbType.String);

                        cmd.Parameters["SiteNm"].Value = txtSiteNm.Text;

                        //datatableのクリア
                        datatable.Clear();
                        //datatableに検索結果をセット
                        datatable.Load(cmd.ExecuteReader());

                        dataGridView1.DataSource = datatable;

                        dataGridView1.Columns[7].Visible = false;

                        trans.Commit();
                        cn.Close();
                        //行の幅を自動で調整する
                        dataGridView1.AutoSizeRowsMode = DataGridViewAutoSizeRowsMode.AllCells;

                    }
                }
            }
        }
        /// <summary>
        /// 
        /// </summary>
        /// <param name="sender"></param>
        /// <param name="e"></param>
        private void dataGridView1_Validating(object sender, CancelEventArgs e)
        {

        }

        private void dataGridView1_CellClick(object sender, DataGridViewCellEventArgs e)
        {
            //DataGridViewの内容をコピー
            Clipboard.SetDataObject(dataGridView1.GetClipboardContent());
        }

        private void Itiran_Load(object sender, EventArgs e)
        {

        }

        private void Itiran_KeyDown(object sender, KeyEventArgs e)
        {
            //Enterキーでコントロール遷移
            if (e.KeyCode == Keys.Enter)
            {
                if (e.Shift)
                {
                    ProcessTabKey(false);
                }
                else
                {
                    ProcessTabKey(true);
                }
            }
        }

        private void Delbtn_Click(object sender, EventArgs e)
        {
            MessageBox.Show("削除してもよろしいですか？");
            foreach (DataGridViewRow r in dataGridView1.Rows)
            {
                //チェックボックスがチェックされているか判定
                if ((bool)r.Cells[0].Value == true)
                {
                    //update文 論理削除
                    string dbConnectionString = "Data Source = C:\\Users/Desktop/WindowsFormsApplication4 /myfriend.sqlite3";
                    using (SQLiteConnection cn = new SQLiteConnection(dbConnectionString))
                    {
                        cn.Open();
                        using (SQLiteTransaction trans = cn.BeginTransaction())
                        {
                            SQLiteCommand cmd = cn.CreateCommand();
                            //SELECT文
                            cmd.CommandText = "update PASS_KANRI set DEL_FLG = 1";
                            trans.Commit();
                            cn.Close();
                        }
                    }
                }
            }
        }
    }
}

Answer 1

おそら Value が null なのではないでしょうか。
もしそうであれば、nullチェックを加えれば解決するかと思います。

以下、参考まで。

例えば、次のようなプロジェクトを作ってみました。
フォームにデータグリッドを配置して、135～138行付近のコードを入れてみました。

        private void Form1_Load(object sender, EventArgs e)
        {
            try
            {
                foreach (DataGridViewRow r in dataGridView1.Rows)
                {
                    if ((bool)r.Cells[0].Value == true)
                    {

                    }
                }
            }
            catch(Exception ee)
            {
                MessageBox.Show(ee.Message);
            }
        }

実行するとエラーが表示されます。

なお、Visual Studio Community 2015 で確認しています。

今回起こっている部分は、

エラー 138行目

とのことですので、

if ((bool)r.Cells[0].Value == true)

の部分でしょうか。

エラーが、

型 'System.NullReferenceException' のハンドルされていない例外が PassKanriSystem.exe で発生しました

追加情報:オブジェクト参照がオブジェクト インスタンスに設定されていません

とあります。これは、典型的なエラーかと思います。
簡単に言えば、参照しているものの実体がありません。(参照先がnull)

このため、rかCells[0]かValueのいずれかが実体が無い(null)ということになります。
rはforeachで取り出しているので、nullということは無いかと思いますし、Cells[0]は列が無ければnullになりますが、ただこの場合はIndexのエラーになるかと思います。となるとValueですがプロパティなので、Valueの中身がnullだと考えられます。

この手のエラーが起こった場合、参照しているものが有効かどうかをデバッグで確認すると良いかと思います。以下のイメージは、rにカーソルを合わせてしばらくフローティングさせて、順次中身を確認していって、Valueがnullであることを確認した様子です。

以上

Comments

[yamaguti]

お世話になります。

わかりやすい解説ありがとうございました。
問題は解決いたしました。

Answer 2

r.Cells[0].Valueの値がNullになっているためにNullReferenceExceptionが発生します。
これは、DataGridViewにデータを追加するときに値が指定されていないためです。
（見た目はチェックが付いていないのでfalseだと勘違いしますがチェックを付け外しした後の値と比べれば分かります）

ですので、初期値としてfalseをセットするようにするか、判断時にnullチェックをするかすればよいかと思います。

またこの問題とは関係ありませんが、このupdate文だとデータ全部のDEL_FLGが1になりますし、
そもそもSQLの実行がされません。

Comments

[yamaguti]

お世話になります。

ご指摘ありがとうございます。
update付近の構文を見直します。

[yamaguti]

rの値を取得し、ID_NOにパラメータを持たせて処理を行おうと思ったのですが、
ID_NOは最初のうちはセルとイコールなのですが、削除がかかるとID_NOとrowがノットイコールになって
しまうため困っています。
どのようにしたほうがいいとかあれば、アドバイスいただければ幸いです。

[YAmaGNZ]

ソースを見る限り、テーブルの全項目をグリッドに表示しているのではないのですか？
r.Cells[X].Valueでキーは取得できると思いますので、それを利用すればいいだけだと思います。
SQLを構築するところで止めて、値を確認すれば分かるはずです。

Answer 3

こんにちは。

updateの周りを見る限りでは問題なさそうに見えますが、どこかでnull設定されたままの参照を使っている筈です。
foreach (DataGridViewRow r in dataGridView1.Rows)当たりにブレークを張って、ステップ実行してみてはどうでしょうか？

Comments

[yamaguti]

お世話になります。
ブレークを実行しておりますが、foreach (DataGridViewRow r in dataGridView1.Rows)で落ちます