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

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

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

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

SQLite

SQLiteはリレーショナルデータベース管理システムの1つで、サーバーではなくライブラリとして使用されている。

Q&A

解決済

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

yamaguti
yamaguti

総合スコア185

C#

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

SQLite

SQLiteはリレーショナルデータベース管理システムの1つで、サーバーではなくライブラリとして使用されている。

3回答

0グッド

0クリップ

43243閲覧

投稿2017/02/18 08:59

編集2017/02/18 10:32

お世話になります。

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

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

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

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

C#

1using System; 2using System.Collections.Generic; 3using System.ComponentModel; 4using System.Data; 5using System.Drawing; 6using System.Linq; 7using System.Text; 8using System.Threading.Tasks; 9using System.Windows.Forms; 10using System.Data.SQLite; 11 12namespace WindowsFormsApplication4 13{ 14 public partial class Itiran : Form 15 { 16 public Itiran() 17 { 18 InitializeComponent(); 19 20 } 21 private void btnsintou_Enter(object sender, EventArgs e) 22 { 23 //新規登録ボタンを押下すると登録画面に遷移 24 Touroku Touroku = new Touroku(); 25 Touroku.Show(); 26 this.Visible = false; 27 } 28 private DataTable datatable = new DataTable(); 29 protected override void OnLoad(EventArgs e) 30 { 31 dataGridView1.DataSource = datatable; 32 base.OnLoad(e); 33 DataGridViewCheckBoxColumn column = new DataGridViewCheckBoxColumn(); 34 dataGridView1.Columns.Add(column); 35 } 36 //テーブルの内容を表示 37 private void buttonkensaku_Click(object sender, EventArgs e) 38 { 39 40 //検索条件がない場合 41 if (txtSiteNm.Text == "") 42 { 43 using (SQLiteConnection con = new SQLiteConnection("Data Source = C:\\Users/Desktop/WindowsFormsApplication4 /myfriend.sqlite3")) 44 45 using (SQLiteDataAdapter adapter = new SQLiteDataAdapter("select * from PASS_KANRI where DEL_FLG = 0", con)) 46 { 47 //データグリッドビューのクリア 48 datatable.Clear(); 49 50 dataGridView1.DataSource = datatable; 51 adapter.Fill(datatable); 52 //DELFLGを非表示に 53 dataGridView1.Columns[7].Visible = false; 54 55 //行の幅を自動で調整する 56 dataGridView1.AutoSizeRowsMode = DataGridViewAutoSizeRowsMode.AllCells; 57 } 58 } 59 //検索条件がある場合 60 else 61 { 62 string dbConnectionString = "Data Source = C:\\Users/Desktop/WindowsFormsApplication4 /myfriend.sqlite3"; 63 using (SQLiteConnection cn = new SQLiteConnection(dbConnectionString)) 64 { 65 cn.Open(); 66 using (SQLiteTransaction trans = cn.BeginTransaction()) 67 { 68 SQLiteCommand cmd = cn.CreateCommand(); 69 //SELECT文 70 cmd.CommandText = "SELECT * from PASS_KANRI where SiteNm = @SiteNm and DEL_FLG = 0"; 71 72 //パラメータセット 73 cmd.Parameters.Add("SiteNm", System.Data.DbType.String); 74 75 cmd.Parameters["SiteNm"].Value = txtSiteNm.Text; 76 77 //datatableのクリア 78 datatable.Clear(); 79 //datatableに検索結果をセット 80 datatable.Load(cmd.ExecuteReader()); 81 82 dataGridView1.DataSource = datatable; 83 84 dataGridView1.Columns[7].Visible = false; 85 86 trans.Commit(); 87 cn.Close(); 88 //行の幅を自動で調整する 89 dataGridView1.AutoSizeRowsMode = DataGridViewAutoSizeRowsMode.AllCells; 90 91 } 92 } 93 } 94 } 95 /// <summary> 96 /// 97 /// </summary> 98 /// <param name="sender"></param> 99 /// <param name="e"></param> 100 private void dataGridView1_Validating(object sender, CancelEventArgs e) 101 { 102 103 } 104 105 private void dataGridView1_CellClick(object sender, DataGridViewCellEventArgs e) 106 { 107 //DataGridViewの内容をコピー 108 Clipboard.SetDataObject(dataGridView1.GetClipboardContent()); 109 } 110 111 private void Itiran_Load(object sender, EventArgs e) 112 { 113 114 } 115 116 private void Itiran_KeyDown(object sender, KeyEventArgs e) 117 { 118 //Enterキーでコントロール遷移 119 if (e.KeyCode == Keys.Enter) 120 { 121 if (e.Shift) 122 { 123 ProcessTabKey(false); 124 } 125 else 126 { 127 ProcessTabKey(true); 128 } 129 } 130 } 131 132 private void Delbtn_Click(object sender, EventArgs e) 133 { 134 MessageBox.Show("削除してもよろしいですか?"); 135 foreach (DataGridViewRow r in dataGridView1.Rows) 136 { 137 //チェックボックスがチェックされているか判定 138 if ((bool)r.Cells[0].Value == true) 139 { 140 //update文 論理削除 141 string dbConnectionString = "Data Source = C:\\Users/Desktop/WindowsFormsApplication4 /myfriend.sqlite3"; 142 using (SQLiteConnection cn = new SQLiteConnection(dbConnectionString)) 143 { 144 cn.Open(); 145 using (SQLiteTransaction trans = cn.BeginTransaction()) 146 { 147 SQLiteCommand cmd = cn.CreateCommand(); 148 //SELECT文 149 cmd.CommandText = "update PASS_KANRI set DEL_FLG = 1"; 150 trans.Commit(); 151 cn.Close(); 152 } 153 } 154 } 155 } 156 } 157 } 158} 159

以下のような質問にはグッドを送りましょう

  • 質問内容が明確
  • 自分も答えを知りたい
  • 質問者以外のユーザにも役立つ

グッドが多くついた質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

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

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

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

下記のような質問は推奨されていません。

  • 間違っている
  • 質問になっていない投稿
  • スパムや攻撃的な表現を用いた投稿

適切な質問に修正を依頼しましょう。

回答3

3

ベストアンサー

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

以下、参考まで。

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

C#

1 private void Form1_Load(object sender, EventArgs e) 2 { 3 try 4 { 5 foreach (DataGridViewRow r in dataGridView1.Rows) 6 { 7 if ((bool)r.Cells[0].Value == true) 8 { 9 10 } 11 } 12 } 13 catch(Exception ee) 14 { 15 MessageBox.Show(ee.Message); 16 } 17 }

実行するとエラーが表示されます。
イメージ説明
なお、Visual Studio Community 2015 で確認しています。

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

エラー 138行目

とのことですので、

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

の部分でしょうか。

エラーが、

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

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

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

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

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

以上

投稿2017/02/18 10:40

rrryutaro

総合スコア146

Chironian, yamaguti👍を押しています

良いと思った回答にはグッドを送りましょう。
グッドが多くついた回答ほどページの上位に表示されるので、他の人が素晴らしい回答を見つけやすくなります。

下記のような回答は推奨されていません。

  • 間違っている回答
  • 質問の回答になっていない投稿
  • スパムや攻撃的な表現を用いた投稿

このような回答には修正を依頼しましょう。

回答へのコメント

yamaguti

2017/02/18 11:04

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

2

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

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

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

投稿2017/02/18 10:39

YAmaGNZ

総合スコア9461

Chironian, rrryutaro👍を押しています

良いと思った回答にはグッドを送りましょう。
グッドが多くついた回答ほどページの上位に表示されるので、他の人が素晴らしい回答を見つけやすくなります。

下記のような回答は推奨されていません。

  • 間違っている回答
  • 質問の回答になっていない投稿
  • スパムや攻撃的な表現を用いた投稿

このような回答には修正を依頼しましょう。

回答へのコメント

yamaguti

2017/02/18 11:05

お世話になります。 ご指摘ありがとうございます。 update付近の構文を見直します。
yamaguti

2017/02/18 11:39

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

2017/02/18 12:03

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

0

こんにちは。

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

投稿2017/02/18 10:23

Chironian

総合スコア23251

良いと思った回答にはグッドを送りましょう。
グッドが多くついた回答ほどページの上位に表示されるので、他の人が素晴らしい回答を見つけやすくなります。

下記のような回答は推奨されていません。

  • 間違っている回答
  • 質問の回答になっていない投稿
  • スパムや攻撃的な表現を用いた投稿

このような回答には修正を依頼しましょう。

回答へのコメント

yamaguti

2017/02/18 10:37

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

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

ただいまの回答率
86.02%

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

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

質問する

関連した質問

同じタグがついた質問を見る

C#

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

SQLite

SQLiteはリレーショナルデータベース管理システムの1つで、サーバーではなくライブラリとして使用されている。