如何使用存储库模式和工厂方法模式组织我的类/接口？

Question

我正在创建一个示例应用程序来一起理解存储库和工厂方法模式，因为将在更大的项目中使用。

我想要实现的是能够使网站与不同的 ORM 工具一起工作。

例如，网站将实现 LINQ to SQL 和 Ado 实体框架工作类，然后使用工厂方法将使用这些 ORM 之一“使用配置值”来加载存储库对象中的数据。

到目前为止我得到的内容如下

interface IRepository : IDisposable
{
  IQueryable GetAll();  
}

interface ICustomer : IRepository
{
}

public class CustomerLINQRepository : ICustomer
{
    public IQueryable GetAll()
    {
       // get all implementation using linqToSql
    }
    public void Dispose()
    {
      throw;
    }
    public IRepository GetObject()
    {
        return this;
    }
}


public class CustomerADORepository : ICustomer
{
    public IQueryable GetAll()
    {
       // get all implementation using ADO
    }
    public void Dispose()
    {
      throw new NotImplementedException();
    }
    public IRepository GetObject()
    {
        return this;
    }
}


// Filling a grid with data in a page
IRepository customers = GetCustomerObject();
this.GridView1.DataSource = customers.GetAll();
this.GridView1.DataBind();
////

public IRepository GetCustomerObject()
{
    return new CustomerLINQRepository(); // this will return object based on a config value later
}

但我可以感觉到有很多设计错误希望你能帮助我找出来以获得更好的设计。

Answer 1

我的两点意见：

A.我会添加通用基础存储库类。无论类型是什么，许多存储库操作都是相同的。它可以节省您大量的打字时间。

B. 我不明白为什么您的存储库要实现 ICustomer 接口。
数据对象的接口是一个很好的实践，但我认为您的存储库不应该实现它。

C. 如果您的数据对象有一个通用的实现，我将为它们创建一个基类，并限制存储库仅适用于该类型的派生类。

我会做类似的事情：

public interface IEntity
{
     // Common to all Data Objects
}

public interface ICustomer : IEntity
{
     // Specific data for a customer
}


public interface IRepository<T, TID> : IDisposable where T : IEntity
{
     T Get(TID key);
     IList<T> GetAll();
     void Save (T entity);
     T Update (T entity);

     // Common data will be added here
}

public class Repository<T, TID> : IRepository<T, TID>
{
     // Implementation of the generic repository
}

public interface ICustomerRepository
{
     // Specific operations for the customers repository
}

public class CustomerRepository : Repository<ICustomer, int>, ICustomerRepository
{
     // Implementation of the specific customers repository
}

用法：

CustomerRepository repository = new CustomerRepository();
IList<ICustomer> customers = repository.GetAll();
// Do whatever you want with the list of customers

这就是我使用 NHibernate 实现 DAL 的方式。您可以在“NHibernate in Action”中找到该用法的一些变化。

我还建议使用某种 IoC 控制器，正如 Matt 建议的那样。

Answer 2

我不确定 GetWanted 的命名。你到底想要得到什么？名称应该更具描述性，也许是 GetCustomerRepository？

GetCustomerObject 与 GetWanted 相同吗？

抛出有什么用？

Answer 3

大部分看起来都不错。我有两点意见：

1. 我不会将您的客户存储库称为 ICustomer。我将其称为 ICustomerRepository。这更加直接，并且澄清了接口基于存储库的职责。 ICustomer 听起来像是封装 Customer 数据的 DataObject。
2. 考虑使用 IOC 容器（谷歌结构图、温莎城堡或 Unity）而不是使用工厂方法。您最终可能会得到很多工厂方法，这可能会导致混乱。相反，在代码中放置一个位置来让 IOC 连接所有存储库可能会更干净。