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

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

ただいまの
回答率

90.04%

Entity frameworkの更新処理について

解決済

回答 2

投稿

  • 評価
  • クリップ 1
  • VIEW 1,577

elvis

score 27

質問

現在、フォームでDBの値を更新できるアプリを開発しています。

そこで質問ですが、下記のようにDbContextをメンバーにおいて、
共用することは、正しい書き方なのでしょうか。

それとも、各イベントやメソッドの中で、using ステートメント等を用いて、
その度に使い捨てたほうが良いのでしょうか。

接続を専有してそうで不安になり、質問させていただきました。
どうぞ宜しくお願いします。

環境

◎.net Frame Work 4.61
◎C#
◎Entity framework 6.2
◎SQLite3
◎visual studio Community 2017
◎windows7 64bit

現在のソースコード(共用するパターン)

namespace Test
{
    public partial class MainWindow : Window
    {
        EntityDbContext Context = new EntityDbContext();

        public MainWindow()
        {
            InitializeComponent();

            //全体にバインド
            this.DataContext = Context.MT_Users.Where(x => x.UserID == 235).Single();
        }

        /// <summary>
        /// 更新ボタンクリック
        /// </summary>
        /// <param name="sender"></param>
        /// <param name="e"></param>
        private void ButtonUpdate_Click(object sender, RoutedEventArgs e)
        {
            //実行
            Context.SaveChanges();
        }

        /// <summary>
        /// 削除ボタンクリック
        /// </summary>
        /// <param name="sender"></param>
        /// <param name="e"></param>
        private void ButtonDelete_Click(object sender, RoutedEventArgs e)
        {
            var deleteTarget = new MT_Users { UserID = 235 };
            Context.MT_Users.Attach(deleteTarget);
            Context.MT_Users.Remove(deleteTarget);

            //実行
            Context.SaveChanges();
        }
    }
}

仕掛中のコード(毎回接続するパターン)

namespace Test
{
    public partial class MainWindow : Window
    {
        public MainWindow()
        {
            InitializeComponent();

            using (var context = new EntityDbContext())
            {
                //バインド
                this.DataContext = context.User.Where(x => x.UserID == 235).Single();
            }
        }

        /// <summary>
        /// 更新ボタンクリック
        /// </summary>
        /// <param name="sender"></param>
        /// <param name="e"></param>
        private void ButtonUpdate_Click(object sender, RoutedEventArgs e)
        {
       //フォームで変更された値をSaveChanges()で更新したいのですが、
       //どう書けば良いか、わかりません。
        }

        /// <summary>
        /// 削除ボタンクリック
        /// </summary>
        /// <param name="sender"></param>
        /// <param name="e"></param>
        private void ButtonDelete_Click(object sender, RoutedEventArgs e)
        {
            using (var context = new EntityDbContext())
            {
               var deleteTarget = new MT_Users { UserID = 235 };
               context.MT_Users.Attach(deleteTarget);
               context.MT_Users.Remove(deleteTarget);

             //実行
               context.SaveChanges();
            }
        }
    }
}
  • 気になる質問をクリップする

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

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

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

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

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

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 2

checkベストアンサー

+2

こんにちは。

実はEntityFrameworkはクエリを実行してデータの実体を取得するか、SaveChangesなどを実行するタイミングで接続を開くため、「接続を占有」という点ではどちらの記述でも問題ないかと思います。
個人的には、DBという「リソース」の有効区間を明示化できている後者の書き方の方が好みです。が、DbContextをDIで放り込む場合などではサービスクラス単位で1つのDbContextを共有するのが一般的なため、前者のような書き方も広く利用されています。

以下は主観ですが、前者のように、クラス内に「リソース」(一般的に、IDisposableを実装するオブジェクトのこと)を配置する場合、そのクラス自身も「リソース」であると看做すことができるため、クラス自身にもIDisposableを実装し、Disposeメソッド内で自分が持つリソースの破棄するように記述するという設計が理想的です。そのため、作成->破棄のサイクルを回すことができないMainWindowのようなオブジェクト内にリソースを配置するのはあまり好ましくないと考えています。なので、今回のケースでは後者の「毎回接続するパターン」をベースに考えることを個人的にオススメします。

あとは、「フォームで変更された値をSaveChanges()で更新したい」というコメントがありますが、それは単純にButtonUpdate_Clickメソッド内で必要なフォームの値を全てかき集めて、それを書き込めばいいだけではないでしょうか?

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

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

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

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/02/27 14:56

    了解しました。後はポリシーの問題になるのでここでは差し控えます。

    キャンセル

  • 2019/02/27 15:09

    こちらこそどうもです。また何か気がかりなことがありましたらそのときはよろしくお願いします。

    キャンセル

  • 2019/03/10 23:19

    皆さま、様々なご意見をありがとうございます。
    私は未熟者ゆえ半分くらいしか内容を理解できていませんが、利用するケースによって適切な使い分けが必要だということはわかった気がします。
    本当はこんなに奥が深いのに、かいつまんで解りやすく教えてくださったtamotoさんに心より感謝申し上げます。ありがとうございました。

    キャンセル

0

こんにちは!
ほぼほぼtamoto様がご説明されているので蛇足感がありますが。
私の場合も後者の毎回接続/切断の記述を好みます。
理由は発行されたクエリおよび取得データがEntityFramework側でキャッシュされ
同じクエリが発行されたときにキャッシュのデータを返すため都合が悪かったためです。
AsNoTracking句で回避することもできますが、1つのContextを使いまわすよりも
DBへアクセスする処理をテーブル単位などの粒度でサービスクラスを作ってレイヤーを分けて
それぞれでその都度Contextの生成、破棄を行うようにしたほうが機能もまとまってわかりやすいです。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

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

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

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/03/10 22:58

    ご返信が遅くなり申し訳ございません。
    キャッシュデータを返すことや、AsNoTracking句など、初心者の私には未知の領域ですが、ひとつひとつ調べて勉強していきたいと思います。
    ありがとうございました。

    キャンセル

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

  • ただいまの回答率 90.04%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

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