それはASP.NET MVC＆EFアプリの正しいアーキテクチャですか？

Question

I持って、次のリポジトリクラス：それは正しいアーキテクチャ

public class TestController : Controller 
{ 
    private Model1Repository _repository; 

    public TestController() 
    { 
     _repository = new Model1Repository(); 
    } 
    public ActionResult Tagedit() 
    { 
     return View(_repository.GetDataFromDB1()); 
    } 

    public ActionResult Avatar() 
    { 
     return View(_repository.GetDataFromDB2()); 
    } 
} 

です：コントローラ次

public class Model1Repository 
{ 
    private NEOGOV_IdeasEntities _dataContext; 
    public Model1Repository() 
    { 
     _dataContext = new NEOGOV_IdeasEntities(); 
    } 
    public IdeaType IdeaType(int ID) 
    { 
     var q = from i in _dataContext.IdeaTypes where i.ID == ID select i; 
     return q.FirstOrDefault(); 
    } 
    ... rest methods.... 
} 

？か否か？私の場合、_repository変数は各リクエストごとに作成されます。

Answer 1

個人的には、コンポーザーではなくコントローラのアクションでリポジトリをインスタンス化します。すべてのコントローラのアクションがリポジトリを必要とすると肯定的であれば十分ですが、コントローラの操作はUIの操作に関係しているため、コントローラメソッドを必要としないコントローラメソッドを持つことはまれではありません。

しかし、それは実際には終日のスタイルのことです（おそらく、多少のパフォーマンスのオーバーヘッドがあります）。あなたがそのようにしたいならば、気軽に！

Answer 2

アーキテクチャ全体では答えられませんが、1つだけのリポジトリが存在しますモデル1リポジトリオブジェクトは1つではなく、TestController用にインスタンス化されています要求（HTTP要求）。