Entity FrameworkでDataGridViewにデータバインディングしたときRowsAddedが複数回実行される

Question

いつもお世話になっております。
EntityFrameworkを使用してDataGridViewに表示するアプリを作成しています
タイトルのとおりイベントが複数回実行されてしまいます。

開発環境は以下の通りです
VS2017
NETFramework,Version=v4.5.21
EntityFramework,version="6.2.0"

    public partial class Form2 : Form
    {
        private SeikouDbContext _db = new SeikouDbContext();

        public Form2()
        {
            InitializeComponent();
        }

        private void Form2_Load(object sender, EventArgs e)
        {
            _db.SupplierStaffs.Load();
            dgv.DataSource = _db.SupplierStaffs.Local.ToBindingList();
        }

        private void dgv_RowsAdded(object sender, DataGridViewRowsAddedEventArgs e)
        {
            Debug.WriteLine($"e.RowIndex = {e.RowIndex}, e.RowCount = {e.RowCount}");
        }
    }

「DataGridViewの設定は追加を有効にする」はチェックを外しています
それ以外のプロパティはデフォルトのままです。

---

結果


1行しか追加されていないのに
Debug.WriteLine($"e.RowIndex = {e.RowIndex}, e.RowCount = {e.RowCount}");が
2回実行されています
期待しているのは、もちろん1行しか追加されていないので1回だけ実行されることです

本来は「DataGridViewの設定は追加を有効にする」はチェックをつけるのでこれで実行すると

Debug.WriteLine($"e.RowIndex = {e.RowIndex}, e.RowCount = {e.RowCount}");が
7回実行されています

これは仕様なのでしょうか？それともなにかプロパティを変更しないといけないのでしょうか？

Comments

[退会済みユーザー]

DataGrid と DataGridView は違うものです。DataGrid のタグは外してください。あと、使用言語は C# のようですが、であれば C# のタグもつけることをお勧めします。

[kikiinu]

はい。ありがとうございます。変更しました

Answer 1

回答ではないですが、ためしにvs2010のVb.netで、Datatableでも行ってみましたが下記の様にkikiinuさんと
同様の結果になりました。
「DataGridViewの設定は追加を有効にする」はチェックを外した場合、2回出力
「DataGridViewの設定は追加を有効にする」はチェックをした場合、7回出力

なので、今回の現象はEntity Frameworkが原因ではなさそうです。

また、※Aと※Bを入れ替えて実行すると下記の様になりました
「DataGridViewの設定は追加を有効にする」はチェックを外した場合、1回出力
「DataGridViewの設定は追加を有効にする」はチェックをした場合、6回出力

なので、DataGridView1.DataSource = dtのタイミングで複数回RowsAddedイベントが実行されている様です。

Public Class Form1

    Private dt As New DataTable

    Private Sub Form1_Load(sender As Object, e As System.EventArgs) Handles Me.Load
        dt.Columns.Add("aaa")
        dt.Columns.Add("bbb")

        dt.Rows.Add("111", "222")　'※A

        DataGridView1.DataSource = dt '※B

    End Sub

    Private Sub DataGridView1_RowsAdded(sender As Object, e As System.Windows.Forms.DataGridViewRowsAddedEventArgs) Handles DataGridView1.RowsAdded

        Debug.WriteLine(String.Format("e.RowIndex = {0}, e.RowCount = {1}", e.RowIndex, e.RowCount))
    End Sub

   
    Public Sub New()

        ' この呼び出しはデザイナーで必要です。
        InitializeComponent()

        ' InitializeComponent() 呼び出しの後で初期化を追加します。
      
    End Sub
End Class

Comments

[kikiinu]

回答ありがとうございます。

年末の忙しい時期にわざわざ試して頂いてありがとうございます。
Entity Framework関係なかったんですね…。

別の方法で対応したいと思います。
ありがとうございました。

Answer 2

質問に対する直接の回答ではなくて別の手段の提案ですが・・・

Entity Framework + Linq to Entity は DataGridView との親和性はあまり高くないです。

DataGridView を UI として、データベースのテーブルのレコードの一覧表示・編集・更新操作を行うなら、旧来の ADO.NET の DataSet / DataTable と DataAdapter を使うことをお勧めします。

一度 Visual Studio のウィザードを使って定番の構成のアプリを作ってみてはいかがでしょう？

質問者さんの使っている DB でウィザードが使えるかどうか分かりませんが（SQL Server はもちろん、MySQL や Oracle でも使えるはずですが）、使えなければ、多分 VS2017 をインストールするときに一緒にインストールされた LocalDB を使って作ってみて、ウィザードが自動生成するコードを見れば今後の開発の参考になると思います。

DB が SQL Server の場合ですが、以下のチュートリアル、

チュートリアル : データベースへのデータの保存 (単一テーブル)
https://msdn.microsoft.com/ja-jp/library/0f92s97z(v=vs.120).aspx

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

・・・のように Visual Studio のデータソース構成ウィザードを利用して型付 DataSet + TableAdapter を作って、それを利用してアプリを作ると、以下のページの図のような構造のアプリが、ほとんど自分でコードを書くこと無しに作れます。

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

操作に慣れると 10 分もかからず作れるはずです。今回のような問題に悩むことはなくなり、開発工数は激減するはずです。保守工数も減るはずです。お試しください。

ところで、質問者さんが立てた先のスレッド https://teratail.com/questions/105000 が放置状態ですが、クローズするまできちんとフォローしていただくようお願いします。

Comments

[kikiinu]

回答ありがとうございます。
最初は従来の開発でやろうとしたんですが、便利なフレームワークがあるのに気づいて
それからEntity Frameworkでやることに決めました。
全然コードを書かなくて済むので感動していたのですが、やっぱりかゆいところに手が届かない…。
自社システムで期限はないのでゆっくりEntity Frameworkでやっていきたいと思います。

気になるのはどういう手順でRowsAddedイベントが呼ばれているか知りたいのですが
これは調べることはかのうなんでしょうか？

[退会済みユーザー]

かゆいところってどこですか？　自分的には SQL クエリを書かない Entity Framework の方がその傾向が強いと思うのですが・・・　あんまりこの話を引っ張っても何だとは思うのですが、教えていただけると幸いです。

[kikiinu]

返事ありがとうございます。
かゆいところ。というか惜しいと言うか。
DataGridViewがあるフォームを継承して違うフォームを作ろうとしたら
継承先のフォームがエラーでデザインモードが開かない。
なら、いっそDataGridViewがあるフォームは継承できないようにしてほしい。
そのためにまたネットで調べてあげくのはてに違う方法で対処して・・・。