Error en FormClosing

[eloy_ameneiros]

Hola a todos,

el problema que tengo es que cuando ejecuto el método de cerrar el formulario me recorre el if dos veces cuando tendría que hacerlo una vez. Dejo el código que tengo por si alguien me puede decir que es lo que falla:

private void Principal_FormClosing(object sender, FormClosingEventArgs e)
{
//Comprueba si hay registros nuevos para añadir y los añade, sino actualiza
DialogResult dr = MessageBox.Show("¿Quiere guardar los datos y salir?", "Guardar Datos", MessageBoxButtons.OKCancel);

if (dr == DialogResult.OK)
{
if (txt_campaña.Text != "")
{
if (cmb_categoria.Text != "")
{
if (cmb_turnos.Text != "")
{
this.añadir();
Application.Exit();
}
}
}
else
{
this.actualizar();
Application.Exit();
}
}
else if (dr == DialogResult.Cancel)
{
Application.Exit();
}
}

[Ka0stj]

Porque no haces la comparación de que si no estan vacíos en un solo if?
Trata con algo asi:

Código c#:

DialogResult dr = MessageBox.Show("¿Quiere guardar los datos y salir?", "Guardar Datos", MessageBoxButtons.OKCancel);
 
            if (dr == DialogResult.OK)
            {
                if (txt_campania.Text != "" && cmb_categoria.Text != "" && cmb_turnos.Text != "")
                {
          
                    this.añadir();
                    Application.Exit();
                }
          
                else
                {
                    this.actualizar();
                    Application.Exit();
                }
            }
            else if (dr == DialogResult.Cancel)
            {
            Application.Exit();
            }
            }
 

[eloy_ameneiros]

El problema es que me salta el DialogResult dos veces antes de cerrar la aplicación y no logro corregirlo.

Código:

private void Principal_FormClosing(object sender, FormClosingEventArgs e)
{
//Antes de cerrar nos pregunta si queremos guardar y en base a eso actualizamos o insertamos
DialogResult dr = MessageBox.Show("¿Quiere guardar los datos antes de salir?", "Guardar Datos", MessageBoxButtons.YesNo);
if (dr == DialogResult.Yes)
{
if (txt_campaña.Text != "" && cmb_categoria.Text != "" && cmb_turnos.Text != "")
{
this.añadir();
}
else
{
this.actualizar();
}
Application.Exit();
}
else if (dr == DialogResult.No)
{
Application.Exit();
}
}

[Bazookao]

el problema es que tienes esta instruccion

Código vb:

Application.Exit();
 

entonces a la hora de que tu aplicacion te muestre por primera vez el cuadro de dialogo y ejecute esta linea te volvera a preguntar

Solo debes quitarla suerte

[eloy_ameneiros]

Solucionado, gracias.