[C#] Kan main form niet sluiten

[pjvandesande]

Het probleem
In me Windows applicatie maak ik gebruik van één main form. Vanuit dit main form worden er meerderen forms geladen en getoont.

Zodra ik de form FrmAddAction laad en open kan ik me applicatie niet meer afsluiten door rechtsboven op kruisje te drukken of op welke manier dan ook.
Dit gebeurt alleen als er in het Load event van FrmAddAction een MessageBox word getoont en ik in de event method het form ook gelijk weer sluit.

De code ziet er als volgt uit:

C#:

private void FrmAddAction_Load(object sender, System.EventArgs e)
{
    _isDirty = false;
    Text += _organisatie.ToString();

    _factory = Factorys.GetOrganisatieFactory();

    ContactPersoonCollection contactPersonen = _factory.GetContactPersonenFromOrganisatie(_organisatie);
    
    if(contactPersonen.Count == 0)
    {
        MessageBox.Show(this, "De geselecteerde organisatie heeft geen contact persoon. Er is "+
            "minimaal 1 contact persoon nodig om een actie te kunnen toevoegen.", Application.ProductName,
            MessageBoxButtons.OK, MessageBoxIcon.Warning);

        Close();
        return;
    }


    foreach(ContactPersoon contactPersoon in contactPersonen)
    {
        cboContactPersoon.Items.Add( contactPersoon );
    }
}

Omgeving
WinForms
.NET Framework 1.1
Windows XP Pro

Maar wat heb je dan al geprobeerd?
Alles in de ctor gooien, dit werkt prima, maar ik wil het niet in me ctor hebben. Want dit is weggelecht voor het Load event.

Ook de method Dispose aanroepen voor het sluiten leverd ook geen positief resultaat op. Verder heb ik nog wat met de code geschoven etc, maar niets wil helpen.
Ook www.deja.com helpt deze keer niet evenals de /14-search.

[whoami]

Ik vind het zowiezo ranzig dat je in een load event al je form wilt gaan sluiten.

Eigenlijk zou je eerst moeten checken of je het form wel moet openen; zoja: open het form, zonee, open het form niet.

[pjvandesande]

whoami schreef op dinsdag 22 februari 2005 @ 13:47:
Ik vind het zowiezo ranzig dat je in een load event al je form wilt gaan sluiten.

Eigenlijk zou je eerst moeten checken of je het form wel moet openen; zoja: open het form, zonee, open het form niet.

Met een extra method (CanOpen) of hou realiseer jij dit normaal?

Ik vind de beredenering goed, ik dacht juist altijd dat je dit soort dingen in je Load event moest stoppen.

[ Voor 15% gewijzigd door pjvandesande op 22-02-2005 13:55 ]

[OZ-Gump]

Als ik jou was zou ik de controle die je nu in je load doet even doen in de procedure die het form aanroept: In het geval dat je nu het form sluit, laadt je het dan niet eens. Dit lijkt me een stuk netter, omdat je dan je form alleen laadt als je aan de juiste voorwaarden voldoet.

[dotnetter]

Zoek het eens in deze richting, het venster word pas gesloten als de messagebox weg is..

C#:

if(contactPersonen.Count == 0)
{
        DialogResult result = MessageBox.Show(this, "De geselecteerde organisatie heeft geen contact persoon. Er is " +
        "minimaal 1 contact persoon nodig om een actie te kunnen toevoegen.", Application.ProductName, MessageBoxButtons.OK, MessageBoxIcon.Warning);
        
        while (result == DialogResult.None)
        {
            // Doe niks...  
        }
        Close();
        return; 
}

Getest en werkt, ik denk dat je het in deze richting moet zoeken, omdat een Messagebox een DialogBox is die toegang tot andere schermen blokkeert..

[ Voor 11% gewijzigd door dotnetter op 22-02-2005 18:05 ]

[Verwijderd]

[quote]dotnetter schreef op dinsdag 22 februari 2005 @ 17:51:

C#:

        while (result == DialogResult.None)
        {
            // Doe niks...  
        }
        Close();
        return; 

en ondertussen maar afvragen waarom je computer enorm traag wordt, terwijl de messagebox getoond wordt...........

Maar goed, de richting is er, alleen die while is overbodig, want er is geen while-conditie, maar een constante

[ Voor 10% gewijzigd door Verwijderd op 22-02-2005 18:55 ]

[dotnetter]

en ondertussen maar afvragen waarom je computer enorm traag wordt, terwijl de messagebox getoond wordt...........

-Voor de goede orde (sorry ik ben koppig ) .. 98% System idle time (in mijn test iig XP 2600+).....

Maar goed, de richting is er, alleen die while is overbodig, want er is geen while-conditie, maar een constante

Volgens mij is er wel een while conditie aangezien de MessageBox Asynchroon van de rest van de applicatie loopt, je zult dus moeten wachten tot de 'controlle' teruggeven word aan de form.
Net zoals je bij multithreading als je een 'join()' gebruikt om aan te geven dat er gewacht moet worden tot de thread terugkeert.

[Verwijderd]

dotnetter schreef op dinsdag 22 februari 2005 @ 19:39:
[...]


-Voor de goede orde (sorry ik ben koppig ) .. 98% System idle time (in mijn test iig XP 2600+).....


[...]

Waarom denk je dat ik het ook doorgestreept had......

Volgens mij is er wel een while conditie aangezien de MessageBox Asynchroon van de rest van de applicatie loopt, je zult dus moeten wachten tot de 'controlle' teruggeven word aan de form.
Net zoals je bij multithreading als je een 'join()' gebruikt om aan te geven dat er gewacht moet worden tot de thread terugkeert.

Stap er maar eens met debugging doorheen. Je zult zien dat je messagebox opgegooid wordt en pas als je op de OK button klikt ga je weer verder. Ook als je het logisch bekijkt. Waar had die result dan geupdate moeten worden? Had er toch minstens iets in de while moeten staan die result ging updaten.

zelfde alsof je deze code neerzet:

C#:

int i = 0;
while(i != 0)
{
//doe niets
}
// i is nu gelijk aan 0

[pjvandesande]

dotnetter schreef op dinsdag 22 februari 2005 @ 17:51:
Zoek het eens in deze richting, het venster word pas gesloten als de messagebox weg is..

C#:

if(contactPersonen.Count == 0)
{
        DialogResult result = MessageBox.Show(this, "De geselecteerde organisatie heeft geen contact persoon. Er is " +
        "minimaal 1 contact persoon nodig om een actie te kunnen toevoegen.", Application.ProductName, MessageBoxButtons.OK, MessageBoxIcon.Warning);
        
        while (result == DialogResult.None)
        {
            // Doe niks...  
        }
        Close();
        return; 
}

Getest en werkt, ik denk dat je het in deze richting moet zoeken, omdat een Messagebox een DialogBox is die toegang tot andere schermen blokkeert..

He gaaf, niets doen!

Maargoed, daar schiet je dus niets mee op. De Form.ShowDialog() method wacht al op een return van het type DialogResult. Dus de while is iets overbodig.

En natuurlijk werkt het, dat wist ik als. Alleen kan ik daarna mijn Main form niet meer op een normale manier sluiten. Alleen als ik de Message loop van de main thread exit sluit me programma. Dit is natuurlijk niet helemaal de bedoeling, maar het is ook niet de bedoeling een MessageBox te showen in een Load event dus en daarna te exit'n.

Ik heb er voor gekozen om dit maar in de ctor te doen.

[whoami]

En toch is het ugly questa.
Ik zou, zoals ik al eerder gezegd heb, en zoals OZ-gump ook al zegt, eerst checken of de form kan geopend worden.

[pjvandesande]

whoami schreef op woensdag 23 februari 2005 @ 10:12:
En toch is het ugly questa.
Ik zou, zoals ik al eerder gezegd heb, en zoals OZ-gump ook al zegt, eerst checken of de form kan geopend worden.

Ik heb het nu inderdaad niet in het load event gedaan, zelf vond ik dit ook al ransig maar dacht dat het er nou eenmaal voor bedoelt was, maar er is nu gewoon een method bij gezet Prepaire en deze returned een boolean.