5

私はここを含むさまざまなソースからリポジトリ作業単位のパターンを学んでいます:

http://www.asp.net/mvc/tutorials/getting-started-with-ef-using-mvc/implementing-the-repository-and-unit-of-work-patterns-in-an-asp-net- mvc-application

上記の作業単位クラスの作成へのリンクを参照すると、次のようになります。

    private GenericRepository<Department> departmentRepository;
    private GenericRepository<Course> courseRepository;

これは問題ありませんが、GenericのUnit of Workクラスを拡張し、GenericRepositoriesのコレクションをセットアップして、通過したモデルに基づいて動的に更新できるようにしたいと思いました。

最終的には、コントローラーで次のことを実行したいと思います。

public class LedgerUserController : Controller
{
    private GenericUnitOfWork unitOfWork = new GenericUnitOfWork();
    LedgerUser ledgeruser = new LedgerUser();


    public ActionResult Index()
    {
        //var ledgerusers = db.LedgerUsers.Include(l => l.Image).Include(l => l.UserType);
        var view = unitOfWork.Repository(ledgeruser).Get(l => l.LastName == "smith");
        return View(view.ToList());
    }
}

これまでの私のクラスとインターフェースは次のとおりです。

IRepository.cs

/// <summary>
/// Generic Repository for CRUD Operations and methods
/// to locate entities within your store. This is not specific to which Data Access
/// tools your are using (Direct SQL, EF, NHibernate, etc).
/// </summary>
public interface IRepository<T> where T : class
{
    //--Search Operations
    IQueryable<T> GetAll();
    IEnumerable<T> GetAllList();
    IEnumerable<T> Get(Expression<Func<T,bool>> filter);
    T GetIt(Expression<Func<T, bool>> filter);
    T GetById(object id);


    //--CRUD Operations
    void Create(T entity);
    void Update(T entity);
    void Delete(T entity);

}

GenericRepository.cs

//////Aferエンティティを検索するためのリポジトリクラス///CRUD操作//////public class GenericRepository:IRepository where TEntity:class {

    internal AccountsContext context;
    internal DbSet<TEntity> dbSet;
    internal IQueryable<TEntity> query;

    /// <summary>
    /// Default Constructor.
    /// </summary>
    /// <param name="context"></param>
    public GenericRepository(AccountsContext context)
    {
        this.context = context;
        this.dbSet = context.Set<TEntity>();
    }

    #region Methods
    #region Search Functionality
    /// <summary>
    /// Obtain the whole Entity to query if needed.
    /// </summary>
    /// <returns>IQueryable object.</returns>
    public virtual IQueryable<TEntity> GetAll()
    {
        IQueryable<TEntity> query = dbSet;
        return query;

    }

    /// <summary>
    /// Obtain the whole Entity to Enumerate throught if needed.
    /// </summary>
    /// <returns>IEnumerble object.</returns>
    public virtual IEnumerable<TEntity> GetAllList()
    {
        IQueryable<TEntity> query = dbSet;
        return query.ToList();

    }

    /// <summary>
    /// Locate an Entity by its indexed id.
    /// </summary>
    /// <param name="id"></param>
    /// <returns></returns>
    public virtual TEntity GetById(object id)
    {
        return dbSet.Find(id);
    }

    /// <summary>
    /// Gets a collection based on LINQ lambda expressions
    /// </summary>
    /// <param name="filter">Lambda Expression</param>
    /// <returns>Query</returns>
    public virtual IEnumerable<TEntity> Get(Expression<Func<TEntity, bool>> filter)
    {
        query = dbSet;

        if (filter != null)
        {
            query = query.Where(filter);
        }

        return this.query.ToList();
    }

    /// <summary>
    /// Gets one record based on a one-to-one relationship.
    /// </summary>
    /// <param name="filter">Lambda Expression.</param>
    /// <returns>One record.</returns>
    public virtual TEntity GetIt(Expression<Func<TEntity, bool>> filter)
    {
        IQueryable<TEntity> query = dbSet;
        return query.SingleOrDefault(filter);

    }


    #endregion
    #region CRUD Functionality

    /// <summary>
    /// Used to create a new entity into the database.
    /// </summary>
    /// <param name="entity">Entity to create.</param>
    public virtual void Create(TEntity entity)
    {
        dbSet.Add(entity);
    }

    /// <summary>
    /// Used to update an entity that already exists in the
    /// database.
    /// </summary>
    /// <param name="entity">Entity to update.</param>
    public virtual void Update(TEntity entity)
    {
        dbSet.Attach(entity);
        context.Entry(entity).State = EntityState.Modified;
    }

    /// <summary>
    /// Used to delete an entity from the database.
    /// </summary>
    /// <param name="entity">Entity to delete.</param>
    public virtual void Delete(TEntity entity)
    {
        if (context.Entry(entity).State == EntityState.Detached)
        {
            dbSet.Attach(entity);
        }
        dbSet.Remove(entity);
    }

    #endregion
    #endregion

}
#endregion

GenericUnitOfWork.cs

  /// <summary>
/// Unit of work class that handles multiple Repositories
/// and shares the context.
/// </summary>
public class GenericUnitOfWork : IUnitOfWork

{
    private AccountsContext context = new AccountsContext();

    Dictionary<string, GenericRepository<IRepository<IRepositoryEntity>>> repostories = null;

    /// <summary>
    /// Generic Repository method which checks the repository is available if not,
    /// it sets it up.
    /// </summary>
    /// <param name="entity">Entity</param>
    /// <returns>Repository to use.</returns>
    public  GenericRepository<IRepository<IRepositoryEntity>> Repository (IRepositoryEntity entity)
    {

            string index = entity.GetType().ToString();

            if (!repostories.ContainsKey(index))
            {

                //Reflections to create the repoositiory if it is not needed.
                Type type1 = typeof(GenericRepository<>);
                Type[] typeArgs = {typeof(IRepositoryEntity)};

                Type constructed = type1.MakeGenericType(typeArgs);
                object o = Activator.CreateInstance(constructed);

                if(o is  GenericRepository<IRepository<IRepositoryEntity>>)
                {
                    var rep = (GenericRepository<IRepository<IRepositoryEntity>>)o;
                    rep.context = this.context;
                    repostories.Add(index, rep);  
                }

            }

            return this.repostories[index];
    }

    /// <summary>
    /// Save method.
    /// </summary>
    public void Save()
    {
        context.SaveChanges();

    }

    private bool disposed = false;

    /// <summary>
    /// Dispose the conxtext when finished.
    /// </summary>
    /// <param name="disposing"></param>
    protected virtual void Dispose(bool disposing)
    {
        if (!this.disposed)
        {
            if (disposing)
            {
                context.Dispose();
            }
        }
        this.disposed = true;
    }

    public void Dispose()
    {
        Dispose(true);
        GC.SuppressFinalize(this);
    }

}

}

ラムダ式では元帳ユーザーのLastNameを解決できないため、リポジトリディクショナリとメソッドリポジトリの実装が正しくない(ジェネリック型が間違っている)ことがわかりました。

var view = unitOfWork.Repository(ledgeruser).Get(l => l.LastName == "smith");

私は問題を過剰に設計していますか、それとも、モデルオブジェクトが与えるもの(上記のLedgerUser)に基づいて設定および作成されたジェネリックリポジトリのコレクションを使用してジェネリック作業単位を作成するための優れたクリーンな方法がありますか?

4

2 に答える 2

4

いくつかのアイデア:

  1. 実装に反映してすべてを作成する代わりにUnitOfWork、IoCコンテナーを使用して、その方法でリポジトリーの作成を自動化することができます。MVCフレームワークは、実際にはこの種のアプローチのために構築されています。この目的でStructureMapを使用する簡単な説明と例は、http://www.mikesdotnetting.com/Article/117/Dependency-Injection-and-Inversion-of-Control-with-ASP.NET-MVCで入手できます。

  2. 本格的なIoCコンテナを使用したくない場合でも、ファクトリロジックとコードをインターフェイスに対して標準化することができます。これは同じ種類の役割を果たしますが、カスタムの作業単位の実装を使い続けたい場合は、より適切に機能する可能性があります。C#/ EFとリポジトリパターンに対する私の答え:複数のリポジトリを持つソリューションのどこにObjectContextを配置しますか?RepositoryProvider、作成できるクラスを投稿しましたUnitOfWork-カスタマイズ可能なファクトリを備えたスコープ付きリポジトリ。このコードは似ていますが、コード例が目的としていることを実行するためのより効果的な方法であるため、少なくともこのコードを確認することをお勧めします。理解しておくべき重要なことの1つは、その回答の例ではObjectContextをUnitOfWorkとして使用しているため、ObjectContextのオカレンスをのオカレンスに置き換えることで、この不一致を説明する必要があるということですIUnitOfWork。コードを確認した後、そのアプローチについて不明な点がある場合は、お知らせください。特定のユースケースを適応させる方法を説明します。

  3. ファクトリロジックについて何か循環しているようです。リポジトリがsを作成する場合、ファクトリを作成するためにをLedgerUser必要としないはずです。LedgerUserあなたが本当に欲しいものはのTypeようなパラメータであるように私には思えますunitOfWork.Repository(typeof(LedgerUser))。負荷をかけてジェネリック型パラメーターを作成し、unitOfWork.Repository<LedgerUser>() `を実行することで、これをより流暢にすることができます。あなたの例に基づくと、インスタンスが必要になる理由はまったくないようです。

  4. メソッドのインターフェースよりも強い型付けを好むようですRepository。多分あなたが目指しているのはもっと似ていると思います:


    public IRepository Repository()
        where T : IRepositoryEntity 
    { 
           //  your factory/cache-retrieval logic here
    }

それ以外の

public  GenericRepository<IRepository<IRepositoryEntity>> Repository (IRepositoryEntity entity)
{
      //  your factory/cache-retrieval logic here
}

次に、呼び出しがであった場合、署名に。と書かれていても、Repository<LedgerUser>メソッドはを返します。これは、私が提案した実装へのリンクがどのように機能するかとほぼ同じです。GenericRepository<LedgerUser>IRepository<LedgerUser>RepositoryProvider

于 2012-09-25T20:32:57.360 に答える
2

オブジェクトのタイプではなくインスタンスをオブジェクトに渡し、タイプごとにリポジトリを保持する理由がわかりません。

これを試して

ConcurrentDictionary<Type, object> _repositories = ...;

public GenericRepository<IRepository<TEntity>> Repository<TEntity>(IRepositoryEntity entity) where TEntity: IRepositoryEntity
{
    return (GenericRepository<IRepository<TEntity>>)_repositories.GetOrAdd(
        typeof(TEntity), 
        t => new GenericRepository<IRepository<TEntity>>(this.Context)
    );
}
于 2012-09-25T20:06:05.110 に答える