• Najnowsze pytania
  • Bez odpowiedzi
  • Zadaj pytanie
  • Kategorie
  • Tagi
  • Zdobyte punkty
  • Ekipa ninja
  • IRC
  • FAQ
  • Regulamin
  • Książki warte uwagi

Aplikacja podwójnie pyta się o zamknięcie

VPS Starter Arubacloud
0 głosów
422 wizyt
pytanie zadane 22 września 2015 w C# przez alibaba40 Początkujący (360 p.)
Witam napisałem sobie aplikację i gdy naciskam krzyżyk wyświetla mi się messagebox, lecz po kliknięciu przycisku wyświetla się on raz jeszcze. Oto kod

 if (richTextBox1.Text != "" || saveFileDialog1.FileName != "")
            {
                if (MessageBox.Show("Czy chcesz zapisać zmiany?", "Komunikat", MessageBoxButtons.YesNoCancel) == DialogResult.Yes)
                {
                    e.Cancel = true;
                    zapiszToolStripMenuItem_Click(sender, e);
                    e.Cancel = false;
                }
             else   if (MessageBox.Show("Czy chcesz zapisać zmiany?", "Komunikat", MessageBoxButtons.YesNoCancel) == DialogResult.No)
                {
                    e.Cancel = false;
                }
                else
                {
                    e.Cancel = true;
                }
            }

 
Dodam, że kod jest w zdarzeniu: Form1_FormClosing
Co może być tego przyczyną?

1 odpowiedź

+1 głos
odpowiedź 22 września 2015 przez zmiennoimienny Mądrala (5,540 p.)

Witaj
Problem leży w tym że w przypadku gdy funkcja równa się DialogResoult.No (klikniesz krzyżyk) wykonuje się kolejny if który ponownie wykonuje tą metodę.

Tu może troche jaśniej:
 

WYWOŁANIE NR. 1:

if (MessageBox.Show("Czy chcesz zapisać zmiany?", "Komunikat", MessageBoxButtons.YesNoCancel) == DialogResult.Yes)
                {

                    //bla bla bla

                }


JEŻELI RÓWNA SIĘ DialogResult.NO (warunek nie jest spełniony) TO NASTĘPUJE WYWOŁANIE NR. 2 


             else   if (MessageBox.Show("Czy chcesz zapisać zmiany?", "Komunikat", MessageBoxButtons.YesNoCancel) == DialogResult.No)
{

bla bla bla
}


Dlatego też po kliknięciu krzyżyka wyświetla się jeszcze raz.

komentarz 22 września 2015 przez alibaba40 Początkujący (360 p.)
Czyli co z tym zrobić? Nie ważne co nacisne: Yes, No czy Cancel i tak MessageBox wyświetli się raz jeszcze, rozumiem zły warunek tak?
komentarz 22 września 2015 przez alibaba40 Początkujący (360 p.)
Rozwiązane :)
komentarz 22 września 2015 przez zmiennoimienny Mądrala (5,540 p.)

Nie jestem pewny czy to będzie działać ale możesz zrobić coś takiego:
 

DialogResult wynik = new DialogResult;
wynik = MessageBox.Show(...); 
if (wynik = DialogResult.Yes)
{
coś tam
}
else if (wynik = DialogResult.No)
{
coś tam
}


Jedyne co pisałem w C# to z wykorzystaniem Unity tak więc prosze nie bić za błędy wink

komentarz 22 września 2015 przez alibaba40 Początkujący (360 p.)
Zrobiłem tak:

switch (result)
                {
                    case DialogResult.Yes:   e.Cancel = true;
                                             zapiszToolStripMenuItem_Click(sender, e);
                                             e.Cancel = false; break;
                    case DialogResult.No: e.Cancel = false; break;
                    case DialogResult.Cancel: e.Cancel = true; break;

                }

 

w form_closing działa super, ale chce jeszcze pod guzikiem zrobić i niestety ten sam problem:

 switch (result)
            {
                case DialogResult.Yes:
                    zapiszToolStripMenuItem_Click(sender, e);
                   Application.Exit(); break;
                
                case DialogResult.Cancel: break;

                case DialogResult.No: Application.Exit(); break;

            }

 

tak zmieniłem bo e.Cancel nie działa tutaj

Podobne pytania

0 głosów
1 odpowiedź 188 wizyt
pytanie zadane 26 stycznia 2019 w C# przez Thanvis Nowicjusz (180 p.)
+1 głos
1 odpowiedź 831 wizyt
pytanie zadane 12 stycznia 2022 w C# przez Czang Kai Shrek Obywatel (1,990 p.)
0 głosów
1 odpowiedź 355 wizyt

92,452 zapytań

141,262 odpowiedzi

319,085 komentarzy

61,854 pasjonatów

Motyw:

Akcja Pajacyk

Pajacyk od wielu lat dożywia dzieci. Pomóż klikając w zielony brzuszek na stronie. Dziękujemy! ♡

Oto polecana książka warta uwagi.
Pełną listę książek znajdziesz tutaj.

Akademia Sekuraka

Akademia Sekuraka 2024 zapewnia dostęp do minimum 15 szkoleń online z bezpieczeństwa IT oraz dostęp także do materiałów z edycji Sekurak Academy z roku 2023!

Przy zakupie możecie skorzystać z kodu: pasja-akademia - użyjcie go w koszyku, a uzyskacie rabat -30% na bilety w wersji "Standard"! Więcej informacji na temat akademii 2024 znajdziecie tutaj. Dziękujemy ekipie Sekuraka za taką fajną zniżkę dla wszystkich Pasjonatów!

Akademia Sekuraka

Niedawno wystartował dodruk tej świetnej, rozchwytywanej książki (około 940 stron). Mamy dla Was kod: pasja (wpiszcie go w koszyku), dzięki któremu otrzymujemy 10% zniżki - dziękujemy zaprzyjaźnionej ekipie Sekuraka za taki bonus dla Pasjonatów! Książka to pierwszy tom z serii o ITsec, który łagodnie wprowadzi w świat bezpieczeństwa IT każdą osobę - warto, polecamy!

...