我正试图找出最简洁的方法来做到这一点。

目前我有一个客户对象:

public class Customer
{
    public int Id {get;set;}
    public string name {get;set;}
    public List<Email> emailCollection {get;set}
    public Customer(int id)
    {
        this.emailCollection = getEmails(id);
    }
}

然后我的电子邮件对象也很基础。

public class Email
{
    private int index;
    public string emailAddress{get;set;}
    public int emailType{get;set;}
    public Email(...){...}
    public static List<Email> getEmails(int id)
    {
        return DataAccessLayer.getCustomerEmailsByID(id);
    }
}

DataAccessLayer 当前连接到数据库，并使用 SqlDataReader 迭代结果集并创建新的电子邮件对象并将它们添加到完成后返回的列表。

那么我可以在哪里以及如何改进它呢？

我是否应该让我的 DataAccessLayer 返回一个 DataTable 并将其留给 Email 对象来解析并将列表返回给客户？

我想“Factory”可能是错误的词，但我是否应该有另一种类型的 EmailFactory，它从 DataAccessLayer 获取 DataTable 并将 List 返回到 Email 对象？我想这听起来有些多余...

将我的 Email.getEmails(id) 作为静态方法是否正确？

我可能只是试图找到最好的“模式”并将其应用于通常是一项简单的任务，从而让自己失望。

谢谢。

跟进

我创建了一个工作示例，其中我的域/业务对象通过 ID 从现有数据库中提取客户记录。 nhibernate 中的 xml 映射文件非常整洁。在我按照教程设置 session 和存储库工厂后，提取数据库记录非常简单。

但是，我注意到性能受到了巨大影响。

我的原始方法由数据库上的存储过程组成，它由 DAL 对象调用，它将结果集解析到我的域/业务对象中。

我用 30 毫秒记录了我原来的方法来获取单个客户记录。然后我用 3000 毫秒记录了 nhibernate 方法来抓取相同的记录。

我错过了什么吗？或者使用这个 nhibernate 路由是否有很多开销？

否则我喜欢代码的整洁:

protected void Page_Load(object sender, EventArgs e)
{
    ICustomerRepository repository = new CustomerRepository();
    Customer customer = repository.GetById(id);
}

public class CustomerRepository : ICustomerRepository
        {
            public Customer GetById(string Id)
            {
                using (ISession session = NHibernateHelper.OpenSession())
                {
                    Customer customer = session
                                        .CreateCriteria(typeof(Customer))
                                        .Add(Restrictions.Eq("ID", Id))
                                        .UniqueResult<Customer>();
                    return customer;
                }
            }
        }

example I followed让我创建一个帮助程序类来帮助管理 session ，也许这就是我得到这个开销的原因？

public class NHibernateHelper
    {
        private static ISessionFactory _sessionFactory;
        private static ISessionFactory SessionFactory
        {
            get
            {
                if (_sessionFactory == null)
                {
                    Configuration cfg = new Configuration();
                    cfg.Configure();
                    cfg.AddAssembly(typeof(Customer).Assembly);
                    _sessionFactory = cfg.BuildSessionFactory();
                }
                return _sessionFactory;
            }

        }

        public static ISession OpenSession()
        {
            return SessionFactory.OpenSession();
        }
    }

对于我正在开发的应用程序，速度至关重要。最终，大量数据将在网络应用程序和数据库之间传递。如果座席需要 1/3 秒来提取客户记录，而不是 3 秒，那将是一个巨大的成功。但是，如果我正在做一些奇怪的事情，并且这是一次性的初始设置成本，那么如果性能与在数据库上执行存储过程一样好，那么这可能是值得的。

仍然愿意接受建议!

已更新。

我正在废弃我的 ORM/NHibernate 路线。我发现性能太慢，无法证明使用它是合理的。对于我们的环境来说，基本的客户查询花费的时间太长了。与亚秒级响应相比，3 秒太多了。

如果我们想要慢速查询，我们只需保留当前的实现即可。重写它的想法是大幅增加时间。

但是，在上周使用了 NHibernate 之后，它是一个很棒的工具!它只是不太符合我对这个项目的需求。

最佳答案

如果您的配置现在可以正常工作，为什么还要弄乱它呢？听起来您并没有按原样确定代码的任何特定需求或问题。

我敢肯定，一群 OO 类型的人可能会挤在一起并在这里提出各种重构建议，以便尊重正确的职责和角色，甚至有人可能会尝试在一两个设计模式中硬塞。但是您现在拥有的代码很简单，听起来没有任何问题 - 我会说留下它。

关于c# - 业务对象数据访问层的最佳 "pattern"，我们在Stack Overflow上找到一个类似的问题： https://stackoverflow.com/questions/644941/