c# - 如何在 C# 的消息框中显示 3 层应用程序中 SQL 异常(唯一标识符)的错误消息？

Question

我正在使用 C# 和 SQL 设计一个 Windows 窗体 3 层应用程序。我设计了使用存储过程与数据库连接的用户注册页面。我已经把文本框从用户那里获取输入。我为我的用户名 列使用了唯一约束。因为我使用的是 3 层应用程序架构，所以我将我的 register_user 函数放在 BLL 中并从 UI 中的按钮调用它。当我在文本框中重复输入 user name 时，它会抛出唯一约束的异常(如预期的那样)，但我希望此消息应显示在 Windows 框中并且我的应用程序不应停止工作.我曾尝试在 register_user 函数中使用 try catch，但没有用。同样，我也尝试在我的 UI(注册按钮)中使用 try catch，但再次失败。我在这里发布我的代码:谢谢

//BLL

    public void register_user(string First_Name,string Last_Name,string User_Name,string Password,string Date_Of_Birth,string Security_Question,string Security_Answer)
    {
        try
        {
            DAL obj = new DAL();
            obj.OpenConnection();
            obj.LoadSpParameters("UR", First_Name, Last_Name, User_Name, Password, Date_Of_Birth, Security_Question, Security_Answer);
            obj.ExecuteQuery();
            obj.UnLoadSpParameters();
        }
        catch (SqlException ex)
        {
            MessageBox.Show(ex.ToString());
        }
        finally
        {
            DAL obj2 = new DAL();
            obj2.CloseConnection();
        }   
       } 

//界面

    private void register_button_Click(object sender, EventArgs e)
    {

        if (first_name_text.Text == "" || last_name_text.Text == "" || user_name_text.Text == "" || password_text.Text == "" || confirm_password_text.Text == "" || answer_txt.Text == "")
        {

            alert1_label.Show();
            error1_label.Show();
            error1_label.Text = "You cannot left mandatory fields empty";
        }

        if (password_text.Text.Length <= 8)
        {

            alert1_label.Show();
            error1_label.Show();
            error1_label.Text = "Password must be greater than 8 characters";
            password_text.Clear();
            confirm_password_text.Clear();
            return;


        }
        if (first_name_text.Text.Any(Char.IsDigit) || first_name_text.Text.Any(Char.IsPunctuation) || first_name_text.Text.Any(Char.IsSeparator) || first_name_text.Text.Any(Char.IsSymbol))
        {
            alert1_label.Show();
            error1_label.Show();
            error1_label.Text = "Numbers and Special Characters are not allowed in first name";
            first_name_text.Clear();
            return;
        }
        if (last_name_text.Text.Any(Char.IsDigit) || last_name_text.Text.Any(Char.IsPunctuation) || last_name_text.Text.Any(Char.IsSeparator) || last_name_text.Text.Any(Char.IsSymbol))
        {
            alert1_label.Show();
            error1_label.Show();
            error1_label.Text = "Numbers and Special Characters are not allowed in last name";
            last_name_text.Clear();
            return;
        }
        if (!user_name_text.Text.Any(Char.IsLetter))
        {
            alert1_label.Show();
            error1_label.Show();
            error1_label.Text = "User Name must contain atleast one alphabet, and must be atleast 4 characters long.";
            user_name_text.Clear();
            return;
        }
        if (user_name_text.Text.Length <= 3)
        {
            alert1_label.Show();
            error1_label.Show();
            error1_label.Text = "User Name must contain atleast one alphabet, and must be atleast 4 characters long.";
            user_name_text.Clear();
            return;
        }

        else
        {
           try
           {
            error1_label.Hide();
            alert1_label.Hide();
            BLL obj = new BLL();
            obj.register_user(first_name_text.Text, last_name_text.Text, user_name_text.Text, password_text.Text, date_of_birth_text.Text, security_question_text.SelectedItem.ToString(), answer_txt.Text);

                MessageBox.Show("Registration Successful");
                UP frm = new UP();
                frm.Text = "Welcome" + " " + first_name_text.Text;
                this.Dispose();
                frm.Show();
            }
            catch(SqlException ex)
            {
                MessageBox.Show(ex.Message);
            }
        }
    }

Answer 1

您的 BLL 中的一个问题是关闭未打开的连接。在您的 register_user 方法中，您有一个 finally block ，它创建一个 new DAL() 对象，然后您调用 CloseConnection。

在没有看到该实现的情况下，我怀疑您想要关闭在 try block 中创建的 DAL 对象。如果是这种情况，一种可能的修复方法是将 obj 变量提升到 try block 之外，以便它在 try 和 finally block 中可用。

请记住，变量在 C# 中具有 block 作用域。

public void register_user(string First_Name,string Last_Name,string User_Name,string Password,string Date_Of_Birth,string Security_Question,string Security_Answer)
{ 
    DAL obj = null; // initialize to a value
    try
    {
        obj = new DAL();
        obj.OpenConnection();
        obj.LoadSpParameters("UR", First_Name, Last_Name, User_Name, Password, Date_Of_Birth, Security_Question, Security_Answer);
        obj.ExecuteQuery();
        obj.UnLoadSpParameters();
    }
    catch (SqlException ex)
    {
        MessageBox.Show(ex.ToString());
    }
    finally
    {
        if (obj!=null)  // check for null in case the constructor did throw an exception
        {
          obj = new DAL();
          obj.CloseConnection();
        } 
        else
        {
            Debug.Assert(obj!=null, "DAL not intialized");
        }
    }   
   } 

超出此答案的范围，您必须重新审视捕获异常的原因和时间。作为一般规则，如果您知道如何处理异常，您只会捕获异常。在您当前的代码示例中，BLL 和 UI 层都处理异常。如果处理得当，BLl 层不应该负责向用户显示错误。它应该将错误传递给调用者，在这种情况下，UI 层和调用者可以决定要做什么。

如果您在上面的示例中采纳了该建议，则应删除 MessageBox.Show 并可能用日志记录和/或错误集合代替。