c# – 面向对象的n层设计.我抽象的太多了吗？还是不够？

我正在构建我的第一个企业级解决方案(至少我试图使其成为企业级).我正在尝试遵循最佳实践设计模式,但我开始担心我可能在抽象方面走得太远.

我正在尝试将我的asp.net webforms(在C#中)应用程序构建为n层应用程序.我使用XSD强类型数据集创建了一个数据访问层,该数据集与sql服务器后端连接.我通过我以1：1为基础创建的一些业务层对象访问DAL到数据集中的数据表(例如,数据集中用户数据表的UsersBLL类).我正在BLL中进行检查,以确保传递给DAL的数据遵循应用程序的业务规则.这一切都很好.我遇到困难的地方是将BLL连接到表示层的点.例如,我的UsersBLL类主要处理整个数据表,因为它与DAL连接.我现在应该创建一个单独的“用户”(Singular)类来映射单个用户的属性,而不是多个用户吗？这样我就不必在表示层中搜索数据表了,因为我可以使用在User类中创建的属性.或者以某种方式尝试在UsersBLL中处理这个更好？

很抱歉,如果这听起来有点复杂……以下是UsersBLL的代码：

using System;
using System.Data;
using PedChallenge.DAL.PedDataSetTableAdapters;

[System.ComponentModel.DataObject]
public class UsersBLL
{
    private UsersTableAdapter _UsersAdapter = null;
    protected UsersTableAdapter Adapter
    {
        get
        {
            if (_UsersAdapter == null)
                _UsersAdapter = new UsersTableAdapter();

            return _UsersAdapter;
        }
    }


    [System.ComponentModel.DataObjectMethodAttribute
        (System.ComponentModel.DataObjectMethodType.Select,true)]
    public PedChallenge.DAL.PedDataSet.UsersDataTable GetUsers()
    {
        return Adapter.GetUsers();
    }

    [System.ComponentModel.DataObjectMethodAttribute
        (System.ComponentModel.DataObjectMethodType.Select,false)]
    public PedChallenge.DAL.PedDataSet.UsersDataTable GetUserByUserID(int userID)
    {
        return Adapter.GetUserByUserID(userID);
    }

    [System.ComponentModel.DataObjectMethodAttribute
        (System.ComponentModel.DataObjectMethodType.Select,false)]
    public PedChallenge.DAL.PedDataSet.UsersDataTable GetUsersByTeamID(int teamID)
    {
        return Adapter.GetUsersByTeamID(teamID);
    }


    [System.ComponentModel.DataObjectMethodAttribute
        (System.ComponentModel.DataObjectMethodType.Select,false)]
    public PedChallenge.DAL.PedDataSet.UsersDataTable GetUsersByEmail(string Email)
    {
        return Adapter.GetUserByEmail(Email);
    }


    [System.ComponentModel.DataObjectMethodAttribute
        (System.ComponentModel.DataObjectMethodType.Insert,true)]
    public bool AddUser(int? teamID,string FirstName,string LastName,string Email,string Role,int LocationID)
    {
        // Create a new UsersRow instance
        PedChallenge.DAL.PedDataSet.UsersDataTable Users = new PedChallenge.DAL.PedDataSet.UsersDataTable();
        PedChallenge.DAL.PedDataSet.UsersRow user = Users.NewUsersRow();

        if (UserExists(Users,Email) == true)
            return false;


        if (teamID == null) user.SetTeamIDNull();
        else user.TeamID = teamID.Value;
        user.FirstName = FirstName;
        user.LastName = LastName;
        user.Email = Email;
        user.Role = Role;
        user.LocationID = LocationID;

        // Add the new user
        Users.AddUsersRow(user);
        int rowsAffected = Adapter.Update(Users);

        // Return true if precisely one row was inserted,// otherwise false
        return rowsAffected == 1;
    }

    [System.ComponentModel.DataObjectMethodAttribute
        (System.ComponentModel.DataObjectMethodType.Update,true)]
    public bool UpdateUser(int userID,int? teamID,int LocationID)
    {
        PedChallenge.DAL.PedDataSet.UsersDataTable Users = Adapter.GetUserByUserID(userID);
        if (Users.Count == 0)
            // no matching record found,return false
            return false;

        PedChallenge.DAL.PedDataSet.UsersRow user = Users[0];

        if (teamID == null) user.SetTeamIDNull();
        else user.TeamID = teamID.Value;
        user.FirstName = FirstName;
        user.LastName = LastName;
        user.Email = Email;
        user.Role = Role;
        user.LocationID = LocationID;

        // Update the product record
        int rowsAffected = Adapter.Update(user);

        // Return true if precisely one row was updated,// otherwise false
        return rowsAffected == 1;
    }

    [System.ComponentModel.DataObjectMethodAttribute
        (System.ComponentModel.DataObjectMethodType.Delete,true)]
    public bool DeleteUser(int userID)
    {
        int rowsAffected = Adapter.Delete(userID);

        // Return true if precisely one row was deleted,// otherwise false
        return rowsAffected == 1;
    }

    private bool UserExists(PedChallenge.DAL.PedDataSet.UsersDataTable users,string email)
    {
        // Check if user email already exists
        foreach (PedChallenge.DAL.PedDataSet.UsersRow userRow in users)
        {
            if (userRow.Email == email)
                return true;
        }
        return false;
    }
}

一些指导正确的方向将非常感谢!!

谢谢大家！

马克斯

解决方法

您正在尝试的这种分层通常涉及从DataTable方法转向使用(大致)数据库中每一行的实例.换句话说,DAL将返回单个用户或用户集合,具体取决于您调用的静态Load方法.这意味着所有采用一堆参数来表示用户的方法都会接受用户DTO.

用户的DAL看起来像这样：

public static class UserDal
{
    public static User Load(int id) { }

    public static User Save(User user) } { }

    public static IEnumerable<User> LoadByDiv(int divId) { }
}

>它是静态的,因为它没有状态.(可以说,它可能有一个数据库连接作为其状态,但那是在大多数情况下并不是一个好主意连接池删除任何效益.其他人可能会争辩单身模式.)>它在用户级别运行DTO类,不是DataTable或任何其他特定于数据库的抽象.也许实现使用了数据库,也许它使用LINQ：来电者不需要知道任何一种方式.注意它是如何返回IEnumerable的而不是承诺任何特别的收藏品.>它只关注数据访问,不是商业规则.因此,它应该只能从企业内部调用处理用户的逻辑类.这样一个类可以决定访问级别如果有的话,允许呼叫者拥有.> DTO代表数据传输对象,它通常相当于一个只包含的类公共财产.它可能很脏属性时自动设置的标志改变了.可能有一种方法可以明确表达设置脏标志,但没有公开的方式来清除它.此外,ID通常是只读的(因此它只能填补反序列化).> DTO故意不包含业务试图确保正确性的逻辑;代替,相应的业务逻辑类是什么从语境上强制执行规则.商业逻辑变化,所以如果DTO或DAL负担它,违反了单一责任原则会导致诸如此类的灾难能够反序列化一个对象,因为它价值不再被视为合法.>表示层可以实例化用户对象,填写并询问业务逻辑层请在DAL中调用Save方法.如果BLL选择这样做,它将填写ID和清除脏旗.使用此ID,BLL可以然后通过调用来检索持久化实例DAL的ID-Load-ID方法.> DAL始终具有Save方法和Load-by-ID方法,但它可能有基于查询的加载方法,例如上面的LoadByDiv示例.它需要提供无论BLL需要什么方法来提高效率操作.>就DAL而言,DAL的实施是一个秘密关于BLL及以上.如果支持是数据库,通常会有存储过程对应于各种DAL方法,但这是一个实现细节.以同样的方式,也是如此一种缓存.

c# – 面向对象的n层设计.我抽象的太多了吗？还是不够？

解决方法

相关文章