是否可以将此代码的第二部分用于存储库模式和泛型

Question

使用版本 2 获得与版本 1 相同的结果是否存在任何问题？ 或者这只是糟糕的编码。

任何想法

public class Customer
{
    public int CustomerID { get; set; }
    public string EmailAddress { get; set; }
    int Age { get; set; }
}

public interface ICustomer
{
    void AddNewCustomer(Customer Customer);
    void AddNewCustomer(string EmailAddress, int Age);
    void RemoveCustomer(Customer Customer);
}

public class BALCustomer
{
    private readonly ICustomer dalCustomer;
    public BALCustomer(ICustomer dalCustomer)
    {
        this.dalCustomer = dalCustomer;
    }

    public void Add_A_New_Customer(Customer Customer)
    {
        dalCustomer.AddNewCustomer(Customer);
    }

    public void Remove_A_Existing_Customer(Customer Customer)
    {
        dalCustomer.RemoveCustomer(Customer);
    }

}

public class CustomerDataAccess : ICustomer
{

    public void AddNewCustomer(Customer Customer)
    {
        // MAKE DB CONNECTION AND EXECUTE
        throw new NotImplementedException();
    }

    public void AddNewCustomer(string EmailAddress, int Age)
    {
        // MAKE DB CONNECTION AND EXECUTE
        throw new NotImplementedException();
    }

    public void RemoveCustomer(Customer Customer)
    {
        // MAKE DB CONNECTION AND EXECUTE
        throw new NotImplementedException();
    }


}

//          VERSION 2 

public class Customer_New : DataRespository<CustomerDataAccess>
{
    public int CustomerID { get; set; }
    public string EmailAddress { get; set; }
    public int Age { get; set; }
}

public class DataRespository<T> 
    where T:class,new()
{

    private T item = new T();
    public T Execute { get { return item; } set { item = value; } }
    public void Update()
    {
        //TO BE CODED
    }

    public void Save()
    {
        //TO BE CODED
    }

    public void Remove()
    {
        //TO BE CODED
    }
}    

class Program
{
    static void Main(string[] args)
    {
        Customer_New cus = new Customer_New()
        {
            Age = 10,
            EmailAddress = "[email protected]"
        };

        cus.Save();
        cus.Execute.RemoveCustomer(new Customer());

        // Repository Version

        Customer customer = new Customer()
        {
            EmailAddress = "[email protected]",
            CustomerID = 10
        };

        BALCustomer bal = new BALCustomer(new CustomerDataAccess());
        bal.Add_A_New_Customer(customer);
    }
}

Answer 1

你发生了很多没有多大意义的事情。

首先，属性的名称应该始终是名词（单数或复数）或“存在”动词，例如 Is* 或 Has*。这些是对象的属性，应该类似于您在回答“请描述一下您的办公桌吗？”之类的问题时所说的内容。 Execute 是一个操作，因此应该是一个方法。同样，版本 1 中的命名约定应为 PascalCased，这意味着没有下划线，并且所有单词的第一个字母应大写。这些并不是铁杆事实，但它们被认为是OOP常见的 C# 编码标准。

其次，主方法中的代码实际上并未实现泛型类中的任何内容。您的类实际上所做的唯一事情是创建 CustomerDataAccess 的实例。 Save() 方法不会执行任何操作，除非您特别能够调用 item.Save() 为了在您的设备上使用您的保存、更新、删除功能泛型类，您的 CustomerDataAccess 类必须实现泛型类所需的接口。例如：

public interface IDataAccess<T> : where T : YourBaseObject {
   public void Update(T item);
   public void Save(T item);
   public void Remove(T item);
}

public class Customer : YourBaseObject {
    public int CustomerID { get; set; }
    public string EmailAddress { get; set; }
    public int Age { get; set; }
}

public class CustomerDataAccess : 
    DataRespository<IDataAccess<Customer>> {
    public void PerformCustomerOnlyAction(Customer customer) { 
     /* do stuff */
    }
}

现在，您可以创建一个处理基本 CRUD 功能的泛型类，并且所有其他功能都可以通过 BaseRepository 属性访问。

/* e.g. T = IDataAccess<Customer>, K = Customer */
public class DataRespository<T>
   where T : IDataAccess<K>, new()
   where K : YourBaseObject, new()
{

    private T _base;
    public T BaseRepository { 
          get { 
             if(_base == null)
                 _base = Activator.CreateInstance<T>(); 
             return _base;
          } 
    }
    public void Update(K item) { /* functionality for YourBaseObject */ }
    public void Save(K item) { /* functionality for YourBaseObject */ }
    public void Remove(K item) {  /* functionality for YourBaseObject */ }
}   

class Program
{
    static void Main(string[] args)
    {
        var repository = new CustomerDataAccess();
        Customer c = new Customer {
            Age = 10,
            EmailAddress = "[email protected]"
        };
        repository.Save(c);
        // This pass-through is no longer needed, but shown as example
        // repository.BaseRepository.PerformCustomerOnlyAction(c);
        repository.PerformCustomerOnlyAction(c);
    }
}

注意 我从头开始/记忆中编写了上述代码。通用类型约束可能无法完全按照我的方式工作。

Stephen Walther 的 ASP.NET 3.5 Unleashed 有几章介绍如何创建存储库模式的设置与您在版本 2 中尝试完成的类似。他还将业务逻辑层和数据访问层之间的处理分开。尽管这本书很大（近 2000 页）并且许多代码示例都是多余的或者最好作为 CD 的一部分保留，但他对初级到中级的范围进行了相当深入的介绍。它可以在亚马逊上使用，价格约为 25 美元。

Answer 2

我认为在实现应用程序的对象模型时，您只需要问自己一些问题，就好像您正在对同事代码进行对象设计审查一样。

• 为什么CustomerAccessLayer要实现接口？是否会有多个层实现此接口。或者也许您期望实现此接口的类有任何多态行为？或者，您可能会将接口与独立模块分开，并通过任何类型的服务提供其功能？
• 为什么需要 BALCustomer 类？为什么不能直接调用 CustomerAccesLayer？而且，我已经谈到过 codesyle 了吗？ :)
• 如果 DataRepository 具有通用行为并且会提供许多 AccessLayers 抛出 Execute 属性，为什么它有自己的方法？

我认为可以继续...我希望你明白我的意思？