サービスパターンエンティティフレームワークasp.net mvc

Question

私のasp.net mvcエンティティフレームワーク6プロジェクトで単体テストを行いたいので、自分のコードを見せなくてはなりません。私は自分のコントローラを自分の状況にさらして、おそらくそれを分けておくべきだと決めました。私は最初にリポジトリ/ UoWパターンを見ていましたが、多くを読んだ後、DIを使って単純なサービスパターンを決定しました（コントローラにはGetProducts()、FindProduct()などのサービスが注入されています）。

私の質問はこのパターンでの変更の追跡に関するものです。多くのことは、コントローラ方式で行われますが、これらのようにすべて作られたメソッドを分離した後、私はちょうどSaveChanges()を呼び出します前に：あなたはSaveChanges()が2回呼び出されていることがわかり上記で

public void AddRequest(Request request) 
    { 
     using (PricedNotesContext ctxt = new PricedNotesContext()) 
     { 
      ctxt.Requests.Add(request); 
      ctxt.SaveChanges(); 
     } 
    } 

public void DeleteRequestData(BaseRequestData reqData) 
    { 
     using (PricedNotesContext ctxt = new PricedNotesContext()) 
     { 
      ctxt.RequestData.Remove(reqData); 
      ctxt.SaveChanges(); 

     } 
    } 

を。しかし、私はそれが一度呼び出されるだけです。両方の操作が正常に完了した場合、私の変更だけが持続するようにしたい。お勧めの解決方法は、SaveChanges()とDispose()というメソッドを持っているだけで、明らかにトランザクションを管理するコントローラにはSaveChanges()とDispose()が公開されていますか？

ありがとうございました！

Answer 1

サービスでは、コンシューマがコンテキストの保存に気を付けなくて済むような機能を提供する必要があります。実際には、文脈についてはわかりません。なぜあなたは、単にような何かはありません。

public class EfNotesService : INotesService 
{ 
     public ExecuteSomeBusinessOperation(input parameters here) 
     { 
      // Validate input parameters 

      using (PricedNotesContext ctxt = new PricedNotesContext()) 
      { 
       ctxt.Requests.Add(...); 
       ctxt.RequestData.Remove(...); 

       // other logic 

       ctxt.SaveChanges(); 
      } 
     } 
} 

あるいはさらに良いと、あなたはまた、要求の開始時にコンテキストを注入し、リクエストの終了時に、それを処分する依存性注入を使用することができます。このコンテキストは、その後EfNotesServiceのコンストラクタに注入され、そしてあなたはちょうどusingステートメントせずに使用することができます。このよう

public class EfNotesService : INotesService 
{ 
     private readonly PricedNotesContext _ctxt; 

     public EfNotesService(PricedNotesContext ctxt) 
     { 
      _ctxt = ctxt; 
     } 
     public ExecuteSomeBusinessOperation(input parameters here) 
     { 
      // Validate input parameters 

      _ctxt .Requests.Add(...); 
      _ctxt .RequestData.Remove(...); 

      // other logic 

      _ctxt .SaveChanges();    
     } 
} 

、同じ文脈では、複数のサービスにまたがることができ、あなたが心配する必要はありませんこれらのサービスにコンテキストを作成して廃棄する。

さらに、サービス操作では、サービスのすべてを行うのではなく、ビジネスロジックを実行するためにビジネスコンポーネントまたはドメインレイヤーを使用することもできます。

グローバル例外ハンドラを追加して、例外を正常に処理します。

Answer 2

ここで私が思いついたのです。

public class EfNotesService : INotesService 
{ 
    private readonly PricedNotesContext _ctxt = new PricedNotesContext(); 

    public IEnumerable<Request> GetAllRequests() 
    { 
     return _ctxt.Requests.ToList(); 
    } 

    public void SaveRequest(Request request) 
    { 
     _ctxt.Entry(request).State = EntityState.Modified; 
    } 

    public void DeleteRequestData(BaseRequestData reqData) 
    { 
     _ctxt.RequestData.Remove(reqData); 
    } 
    // ... 
    // Other methods... 
    // ... 

    // Transaction control 
    public void SaveChanges() 
    { 
     _ctxt.SaveChanges(); 
    } 

    public void Dispose() 
    { 
     _ctxt.Dispose(); 
    } 
} 

これはUoWやリポジトリではありませんが、単純であり、うまく機能します。さらに、私はDbContextのすべての「ストア」のメリットを維持します。