为什么Form#FormClosing事件在c#.net中执行2次

我在C#.net中有一个windows窗体应用程序，我想在通过X按钮关闭窗体时进行备份，因此我编写了以下代码来执行此操作。但当我运行该程序时，我注意到此事件执行了2次，请帮我解决此问题

private void MenuFrm_FormClosing(object sender, FormClosingEventArgs e)
        {
            //  DialogResult dialogResult = MessageBox.Show("ایا مایل به گرفتن نسخه پشتیبان می باشید", "هشدار", MessageBoxButtons.YesNo);
            //  if (dialogResult == DialogResult.Yes && !closefrm)
            try
            {
                DialogResult dialogResult = MessageBox.Show("آیا مایل به خروج از نرم افزار میباشید؟", "خروج", MessageBoxButtons.YesNo);
                if (dialogResult == DialogResult.Yes)
                {
                    SaveFileDialog f = new SaveFileDialog();
                    f.InitialDirectory = "D:\\";
                    f.Title = "HoghooghDastmozdBackup";
                    if (Directory.Exists("E:\\MobtakeranSoftBackup\\"))
                    {
                        f.FileName = "E:\\MobtakeranSoftBackup\\" + getPersianDate() + ".BAK";
                        f.FilterIndex = 1;
                        f.OverwritePrompt = true;
                        f.Filter = @"SQL Backup files (*.BAK) |*.BAK|All files(*.*) |*.*";
                        SqlConnection sqlconn = new SqlConnection(DBsetting.Connstring);
                        SqlCommand sqlcmd = new SqlCommand("BACKUP DATABASE HoghooghDastmozd TO  DISK =@n", sqlconn);
                        sqlcmd.Parameters.AddWithValue("@n", f.FileName);
                        sqlconn.Open();
                        sqlcmd.ExecuteNonQuery();
                        sqlconn.Close();
                        Application.Exit();
                    }
                    else
                    {
                        Directory.CreateDirectory("E:\\MobtakeranSoftBackup\\");
                        f.FileName = "E:\\MobtakeranSoftBackup\\" + getPersianDate() + ".BAK";
                        f.FilterIndex = 1;
                        f.OverwritePrompt = true;
                        f.Filter = @"SQL Backup files (*.BAK) |*.BAK|All files(*.*) |*.*";
                        SqlConnection sqlconn = new SqlConnection(DBsetting.Connstring);
                        SqlCommand sqlcmd = new SqlCommand("BACKUP DATABASE HoghooghDastmozd TO  DISK =@n", sqlconn);
                        sqlcmd.Parameters.AddWithValue("@n", f.FileName);
                        sqlconn.Open();
                        sqlcmd.ExecuteNonQuery();
                        sqlconn.Close();
                        Application.Exit();
                    }
                }
                else
                {
                    e.Cancel = true;
                }

            }
            catch (Exception ex)
            {
                MessageBox.Show(ex.Message);
                Application.Exit();
            }


        }

---

您正在调用

Application.Exit（）

。这是在窗体完全关闭之前完成的。。。它还开着。因此，将触发一个新的

Close

操作，该操作涉及第二个

Form_Closing

事件。当您能够处理第二个事件时，第一个事件已经完成，表单已经完全关闭，因此它就停止在那里（您最终会触发两次事件，而不是更多）

为此，您可以通过向类添加一个布尔值来进行调整，默认值为false，在方法顶部检查它是否为false，并在方法末尾将其设置为true。但这只是修补了真正的问题。。。你在form_闭幕式上做得太多了。

Application.Exit（）

调用属于

Form\u Closed

，通常只触发一次

---

在这里，我想指出以下代码：

sqlconn.Open();
sqlcmd.ExecuteNonQuery();
sqlconn.Close(); 

这是一种糟糕的做法。对SqlConnection对象的

.Close（）

调用应该总是在

最后
块中（最简单的方法是通过
使用
块）。在这种情况下，您可能没问题，因为应用程序无论如何都要退出，但以这种方式执行查询并不是一个好习惯，这让人怀疑程序中的其他地方是否也犯了同样的错误。我还想知道你为什么在这里重复这么多代码。整个部分可以简化为：

Directory.CreateDirectory("E:\\MobtakeranSoftBackup\\"); //it's good to just call this, even if the directory already exists.
f.FileName = System.IO.Path.Combine(@"E:\MobtakeranSoftBackup", getPersianDate() + ".BAK");
f.FilterIndex = 1;
f.OverwritePrompt = true;
f.Filter = @"SQL Backup files (*.BAK) |*.BAK|All files(*.*) |*.*";
using (var sqlconn = new SqlConnection(DBsetting.Connstring))
using (var sqlcmd = new SqlCommand("BACKUP DATABASE HoghooghDastmozd TO  DISK =@n", sqlconn))
{
     sqlcmd.Parameters.AddWithValue("@n", f.FileName);
      sqlconn.Open();
      sqlcmd.ExecuteNonQuery();
}
Application.Exit();                  

您正在调用
Application.Exit（）
。这是在窗体完全关闭之前完成的。。。它还开着。因此，将触发一个新的
Close
操作，该操作涉及第二个
Form_Closing
事件。当您能够处理第二个事件时，第一个事件已经完成，表单已经完全关闭，因此它就停止在那里（您最终会触发两次事件，而不是更多）

为此，您可以通过向类添加一个布尔值来进行调整，默认值为false，在方法顶部检查它是否为false，并在方法末尾将其设置为true。但这只是修补了真正的问题。。。你在form_闭幕式上做得太多了。
Application.Exit（）
调用属于
Form\u Closed
，通常只触发一次


在这里，我想指出以下代码：

sqlconn.Open();
sqlcmd.ExecuteNonQuery();
sqlconn.Close(); 

这是一种糟糕的做法。对SqlConnection对象的
.Close（）
调用应该总是在
最后
块中（最简单的方法是通过
使用
块）。在这种情况下，您可能没问题，因为应用程序无论如何都要退出，但以这种方式执行查询并不是一个好习惯，这让人怀疑程序中的其他地方是否也犯了同样的错误。我还想知道你为什么在这里重复这么多代码。整个部分可以简化为：

Directory.CreateDirectory("E:\\MobtakeranSoftBackup\\"); //it's good to just call this, even if the directory already exists.
f.FileName = System.IO.Path.Combine(@"E:\MobtakeranSoftBackup", getPersianDate() + ".BAK");
f.FilterIndex = 1;
f.OverwritePrompt = true;
f.Filter = @"SQL Backup files (*.BAK) |*.BAK|All files(*.*) |*.*";
using (var sqlconn = new SqlConnection(DBsetting.Connstring))
using (var sqlcmd = new SqlCommand("BACKUP DATABASE HoghooghDastmozd TO  DISK =@n", sqlconn))
{
     sqlcmd.Parameters.AddWithValue("@n", f.FileName);
      sqlconn.Open();
      sqlcmd.ExecuteNonQuery();
}
Application.Exit();                  

确保没有连接两个事件处理程序-在项目中搜索“MenuFrm_FormClosing”。您可能还需要检查FormClosingEventArgs的CloseReason，因为这也会在Windows关闭时触发。请确保您没有连接两个事件处理程序-在项目中搜索“MenuFrm\u FormClosing”。您可能还需要检查FormClosingEventArgs的CloseReason，因为这也会在Windows关闭时触发。Joel您的代码工作不正常。它的工作方式与我的代码类似。它在关闭应用程序时运行两次。只有水平线上方的部分解决了您的问题。其余部分则指出了代码中一些明显的错误做法。Joel您的代码运行得不好。它的工作方式与我的代码类似。它在关闭应用程序时运行两次。只有水平线上方的部分解决了您的问题。其他人则指出了代码中一些明显的错误做法。