リポジトリを使用した作業単位パターンへの依存性注入

Question

thisと同様にリポジトリをラップする作業単位クラスを作成したいとします。

私が抱えている問題は、例の汎用リポジトリをIRepositoryインターフェイスに置き換えて依存性注入を実装しようとしていることです。リンクされた記事の中で、彼らはゲッターを使ってリポジトリがインスタンス化されているかどうかをチェックし、リポジトリがインスタンス化されていないかどうかをチェックします。

public GenericRepository<Department> DepartmentRepository 
{ 
    get 
    { 
     if (this.departmentRepository == null) 
     { 
      this.departmentRepository = new GenericRepository<Department>(context); 
     } 
     return departmentRepository; 
    } 
} 

強く結合している。

これを回避するには2つの方法があります。

1. コンストラクタインジェクションを使用します。
2. セッター注入を使用します。

1の問題は、すべてのリポジトリを注入すると、その特定の作業単位でそれらを使用しなくても、各リポジトリをインスタンス化する必要があるということです。したがって、そうするオーバーヘッドが発生します。私は、データベース全体の作業単位クラスを使用することを想像していたので、これは多くの不必要なインスタンス化と巨大なコンストラクタにつながります。

2の問題は、ヌル参照例外を設定して終了することを忘れやすいことです。

このシナリオでは、ベストプラクティスはありますか？そして私が逃した他のオプションはありますか？

私は依存性注入に取り掛かり、トピックで見つけることができるすべてのリサーチを行っていますが、何かキーが見つからない可能性があります。

Answer 1

これにアプローチする方法は、コンテナの注入を介してRepositoryを作成するためのUnitOfWorkが責任を負うことはありませし、代わりにUnitOfWorkは、インスタンス作成時にその存在を知っていることを保証するために、それを各Repositoryの責任を作ることです。

これは

• は、あなたのUnitOfWorkあなたが

（ anti-patternであることを多くの人に考えられる）サービスロケータを使用していないそれぞれの新しい Repository

のために変更する必要がないことを保証します

これは最高のいくつかのコードを用いて実証された - 私は例がこの問題を回避基づいてSimpleInjectorを使用します。

public interface IRepository 
{ 
    void Submit(); 
} 
public interface IRepository<T> :IRepository where T : class { } 
public abstract class GenericRepository<T> : IRepository<T> where T : class { } 

とUnitOfWork

public interface IUnitOfWork 
{ 
    void Register(IRepository repository); 
    void Commit(); 
} 

は各RepositoryがUnitOfWorkに自身を登録する必要があり、これはそれがあることを確認するために、抽象親クラスGenericRepositoryを変更することで行うことができますRepository抽象化で開始行わ：

public abstract class GenericRepository<T> : IRepository<T> where T : class 
{ 
    public GenericRepository(IUnitOfWork unitOfWork) 
    { 
     unitOfWork.Register(this); 
    } 
} 

からの

各実Repository継承：

public class Department { } 
public class Student { } 

public class DepartmentRepository : GenericRepository<Department> 
{ 
    public DepartmentRepository(IUnitOfWork unitOfWork): base(unitOfWork) { } 
} 

public class StudentRepository : GenericRepository<Student> 
{ 
    public StudentRepository(IUnitOfWork unitOfWork) : base(unitOfWork) { } 
} 

はUnitOfWorkの物理的な実装に追加すると、すべてのセットです：

public class UnitOfWork : IUnitOfWork 
{ 
    private readonly Dictionary<string, IRepository> _repositories; 
    public UnitOfWork() 
    { 
     _repositories = new Dictionary<string, IRepository>(); 
    } 

    public void Register(IRepository repository) 
    { 
     _repositories.Add(repository.GetType().Name, repository); 
    } 

    public void Commit() 
    { 
     _repositories.ToList().ForEach(x => x.Value.Submit()); 
    } 
} 

コンテナの登録が自動的にすべてピックアップするように設定することができます定義されたインスタンスIRepositoryを生涯有効範囲に登録して、トランザクションの存続期間中存続するようにします。

public static class BootStrapper 
{ 
    public static void Configure(Container container) 
    { 
     var lifetimeScope = new LifetimeScopeLifestyle(); 

     container.Register<IUnitOfWork, UnitOfWork>(lifetimeScope); 

     container.RegisterManyForOpenGeneric(
      typeof(IRepository<>), 
      lifetimeScope, 
      typeof(IRepository<>).Assembly); 
    } 
} 

DIを中心に構築されたこれらの抽象化とアーキテクチャでは、UnitOfWorkがあり、すべてのサービス呼び出しでインスタンス化されたすべてのRepositoryがあり、すべてのリポジトリが定義されているコンパイル時間の検証があります。あなたのコードはopen for extension but closed for modificationです。

はすべてこれをテストするために - これらのクラスを追加

public class SomeActivity 
{ 
    public SomeActivity(IRepository<Department> departments) { } 
} 

public class MainActivity 
{ 
    private readonly IUnitOfWork _unitOfWork; 
    public MainActivity(IUnitOfWork unitOfWork, SomeActivity activity) 
    { 
     _unitOfWork = unitOfWork; 
    } 

    public void test() 
    { 
     _unitOfWork.Commit(); 
    } 
} 

コードの行に対してブレークポイントを置きBootStrapper.Configure()

//register the test classes 
container.Register<SomeActivity>(); 
container.Register<MainActivity>(); 

にこれらの行を追加します。

_repositories.ToList().ForEach(x => x.Value.Submit()); 

そして、最後に、このConsoleテストコードを実行してください：

class Program 
{ 
    static void Main(string[] args) 
    { 
     Container container = new Container(); 
     BootStrapper.Configure(container); 
     container.Verify(); 
     using (container.BeginLifetimeScope()) 
     { 
      MainActivity entryPoint = container.GetInstance<MainActivity>(); 
      entryPoint.test(); 
     } 
    } 
} 

コード停止はブレークポイントで実行され、アクティブなインスタンスはIRepositoryであり、Submit()はデータベースへの変更を待っています。

あなたは、トランザクションなどを処理するためにUnitOfWorkを装飾することができます。私は強力に譲歩します。この時点でNetJunkieは、これら2つの記事hereとhereを読むことをお勧めします。

Answer 2

リポジトリインスタンスを挿入する代わりに、それらのインスタンスの作成を担当する単一のファクトリオブジェクトを挿入します。あなたのゲッターはその工場を使用します。