Question

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

我正在尝试将我的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, string FirstName, string LastName,
        string Email, string Role, 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;
    }
}

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

全部谢谢！

最高

Answer 1

你正在尝试的那种分层通常涉及从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加载方法。
DAL始终具有Save方法和Load-by-ID 方法，但它可能有基于查询的加载方法，例如上面的LoadByDiv示例。它需要提供无论BLL需要什么方法来提高效率操作
DAL的实施是一个秘密关于BLL及以上。如果支持是数据库，通常会有存储过程对应于各种DAL方法，但这是一个实现细节。以同样的方式，任何一个一种缓存。

Answer 2

为了方便您的设计，您绝对不希望撤回整个数据表并在表示层中搜索它们。数据库的优点在于它被编入索引以便于快速查询行级数据（即通过索引标识符获取行）。

您的DAL应该公开GetUserByUserID（int userID）之类的方法。然后，您应该通过BLL公开该方法，强制执行任何所需的业务逻辑。

此外，我会清除类型数据集并考虑一个ORM工具，如实体框架。

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

2 个答案: