三层架构纠错

   最近，运用三层架构的理念，对机房收费系统重新进行了设计。虽然对三层的感念做过一些总结，但是还是存在一些认识上的差错，在这儿做一下错误解剖。

   以下是U层登录窗体中的代码，主要任务完成用户验证，登录系统主界面：

Private Sub Button1_Click(sender As Object, e As EventArgs) Handles cmdLogin.Click Dim MyUIB As New ManagerB.UserInfoBll Dim userInput As New ManagerE.UserEntity Dim MyDate As New ManagerD.DateDAL Dim currentDate As String = MyDate.MyDate '错误1 Dim currentTime As String = MyDate.MyTime If txtUserID.Text = "" Then MessageBox.Show("密码不能为空，请输入！") End If If txtUserPWD.Text = "" Then MessageBox.Show("用户名不能为空，请输入！") End If userInput.password = Trim(txtUserPWD.Text) userInput.UserID = Trim(txtUserID.Text) If MyUIB.Login(userInput) Then If MyUIB.IsLogin(userInput.userID) Then '错误2 MessageBox.Show("改用户已经登录，不能重复登陆！！") Else Dim MyORB As New ManagerB.OndutyRecordBll MyORB.AddLoginRecord(userInput.userID, currentDate, currentTime) frmMain.Show() Me.Hide() '将登陆记录放进数据库 End If Else MessageBox.Show("用户名或密码错误，请核实后再进行操作！") End If

   错误1：直接引用D层；

   错误2：U层代替B层进行逻辑处理

接着是登录的时序图，刚开始这个功能的时序图做了两个：

    对于仅登录这一个功能，在B层建立了两个类,在D层建立了两个类。D层建立两个类还算说的过去，这是根据两张表的处理，建立了两个类，然后就顺着表的处理建立了B层的两个类。错误也就随之出现了。然后再顺着，U层的设计也就跟着出了问题。

   U层要做的是触发某一个功能的开始，之后所有的该功能的实现，应该放在B层的逻辑处理层去完成，结果大部分的逻辑处理都是在U曾完成的，而且如果要创建这个B层的类，也应该是根据这个逻辑处理来创建，而不是针对表的处理去创建。

   之所以把U层和B层的功能划分的不清，是因为，刚开始的时候，先完成的代码，然后再去画的UML图。结果创建的类和图无组织无纪律，显得有些散乱。散乱还是次要的，关键是一点不符合面向对象程序设计的要求-封装。

看修改后理想的结果；

        If MyLoginBll.Login(userInput, newRecord) Then
            frmMain.Show()
            Me.Hide()
        Else
            MessageBox.Show("用户名或密码错误，请核实后再进行操作！")
        End If

   然后是该登录过程修改后的时序图：

   修改之后，登录所有的过程处理都真正交由B层去处理了。如果对我上面的叙述不狗明白的话，可以套用以前的比喻，我原先的代码是，服务员干了厨师应该做的事儿，现在又让他们各司其职了，没有越俎代庖了。

   有错误就改正，让系统在不断修改中完善，One bird in hand is better than double-bird in the bush，继续完善，重构这个系统！





转载于:https://www.cnblogs.com/lucky7/p/3768650.html