这个 4 层架构好吗？（异常处理对我来说很重要:)

Question

我的应用程序中有以下层：

• 实体
• 数据库（带有实体引用）
• 业务（带有数据库和实体引用）
• 用户界面（带有业务和实体引用）

  ---

这是我的代码示例：

• 数据库层中的 UserDAL 类：

---

public class UsersDal
{
    databaseDataContext db;
    public UsersDal()
    {
        try
        {
            db = new databaseDataContext(ConnectToDatabase.GetConnectionString());
        }
        catch (Exception ex)
        {
            throw ex;
        }
    }
    public List<User> GetAllUsers()
    {
        try
        {
            return (from u in db.Users select u).ToList();
        }
        catch (Exception ex)
        {
            throw ex;
        }
    }

等等...

---

在业务层中的 UserBLL 类中，我这样写：

public class UsersBll
{
    UsersDal user;
    public UsersBll()
    {
        try
        {
            user = new UsersDal();
        }
        catch(Exception ex)
        {
            throw new ProjectException(Errors.CannotCreateObject, ex);
        }
    }
    public List<User> GetAllUsers()
    {
        try
        {
            return user.GetAllUsers();
        }
        catch(Exception ex)
        {
            throw new ProjectException(Errors.CannotReadData, ex);
        }
    }

---

在 UI 中，我写：

    private void GetUsers()
    {
        try
        {
            UsersBll u = new UsersBll();
            datagrid.DataSource = u.GetAllUsers();
        }
        catch(ProjectException ex)
        {
            MessageBox(ex.UserMessage);// and also can show ex.InnerException.Message for more info
        }
    }

好吧，我使用一个 ProjectException 命名类来产生一个错误，其中包含我创建的 BLL 消息和操作系统自动操作的异常消息。 我还创建了一个可能错误的枚举和一个字典 以下是有关它的一些详细信息：

namespace Entities
{
    public enum Errors
    {
        CannotCreateObject,
        CannotReadData,
        CannotAdd,
        CannotEdit,
        CannotDelete,...
    }

[global::System.Serializable]
public class ProjectException : Exception
{
    public ProjectException(Errors er, Exception ex)
        : base(errors[er], ex)
    {
        currentEx = er;//er is Errors enum
    }
    static ProjectException()
    {
        errors = new Dictionary<Errors, string>();
        errors.Add(Errors.CannotCreateObject, "the application cannot connect to database!");
        errors.Add(Errors.CannotReadData, "the application cannot read data from database"); //...
    }
    public string UserMessage
    {
        get
        {
            try
            {
                return errors[currentEx];
            }
            catch
            {
                return "Unknown error!";
            }
        }
    }

---

这好吗？ 它对我来说很好。 你有什么想法？

Answer 1

在catch (ex)中执行throw ex;几乎总是错误的。要么直接抛出；要么抛出新的whateverException("someMessage", ex);。决定使用前一种形式还是后者通常取决于您是否跨越应用程序层。如果从 DatabaseWrapper 类型派生的 AcmeServerDatabaseWrapper 在抛出 AcmeDatabaseTableNotFoundException 时正在执行查询，则它应该捕获该异常并将其作为 DatabaseWrapperTableNotFoundException（如果存在此类类型）或 DatabaseWrapperOperationFailedException 重新抛出。具有从 DatabaseWrapper 派生的对象的客户端代码应该能够知道该对象将抛出什么类型的异常，而不必知道它是什么类型的对象。任何从数据库层逃逸而没有被包装的异常都是客户端代码不太可能能够合理处理的异常，但可能会错误地处理（认为它发生在实际发生的上下文之外的上下文中）。