Discussion :

[melleb]

Bonjour,

j'ai créé une application WinForm
et je voudrais savoir s'il est possible d'annuler son lancement lorsqu'il est en cours
et si oui, comment?...

J'ai essayé un truc mais ça plante...

Dans le main, j'ai ça :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

Application.Run(new MonAppli());

et dans le constructeur de l'appli,
je teste la config, et si elle n'est pas ok, je voudrais annuler le lancement...
j'ai essayé ça :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
public MonAppli()
{
    // des trucs...
 
    if (configNOK)
        this.Close();
 
    // des trucs...
}

ça plante dans le main au niveau du Application.Run
avec une erreur 'Impossible d'accéder à un objet supprimé'...

Quelqu'un aurait-il une solution?...

Merci

[fally]

a la place de ton this.Close() essaie Application.Exit()

[theMonz31]

salut

Application.Exit() ou simplement :

Tester ta "configuration" avant de lancer ton run()

The Monz, Toulouse

[melleb]

Bonjour,

le Application.Exit() ne marche pas mieux...

finalement, j'ai laissé le this.Close() dans le constructeur
et dans le main, j'ai fait ça :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
MonAppli appli = new MonAppli();
if (!appli.IsDisposed)
    Application.Run(appli);

et là, ça marche...

[SirJulio]

Salut,

Application.Exit n'aura pas marché etant donné que l'application n'a pas encore etait "runné" au moment de l'appel.

Pour revenir sur le close(), appeler une methode qui va disposer des ressources en plein milieu d'un contructeur, ca sent le paté. Pour faire ca bien, pour ma part, j'exposerai une propriété IsValid sur ma form que j'assignerai dans le constructeur. Si le flag est assigné, tu ne lances pas le Run() et l'application se terminera d'elle meme.

[dev01]

Tester ta "configuration" avant de lancer ton run()

est la seule méthode fiable et correct.

pour rappel, si tu veux faire ça il faut aller dans le fichier program.cs et coder cela dans la méthode Main (enfin quand je dit coder c'est appeler la méthode de vérification et lancer ou non l'affichage de la form )

[theMonz31]

salut

est la seule méthode fiable et correct.

Merci beaucoup, ca me fait chaud au coeur de lire cela !!!

Me reste plus qu'a devenir MVP un jour

The Monz, Toulouse

[melleb]

Pour revenir sur le close(), appeler une methode qui va disposer des ressources en plein milieu d'un contructeur, ca sent le paté. Pour faire ca bien, pour ma part, j'exposerai une propriété IsValid sur ma form que j'assignerai dans le constructeur. Si le flag est assigné, tu ne lances pas le Run() et l'application se terminera d'elle meme.

en effet,
ça revient au même que ce que j'ai fait,
mais c'est sûrement plus propre

[SirJulio]

C'est surtout que d'une maniere generale, c'est dangeureux :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
class MaForm : Form
{
DummyClass s;
    public MaForm()
    {
        //..
        this.Close();
        //..
        s = new DummyClass();
    }
 
    protected override void Dispose()
    {
        s.DoSomeStuff(); // <= NullRefException !
    }
}

L'exemple est trivial, mais c'est pour exposer le probleme. On pourrait faire cela en placant le Close() en fin de contructeur pour eviter des problemes de champs non initialisés, mais au final en terme de design ca revient à tester l'etat post construction. De plus le fait d'appeler un close() dans le constructeur te laissera un fin de construction une form invalide (InvalidOperation) ce qui pourrait etre suprenant si ce n'est pas documenté.

Bref, pour toutes ces raisons, et pour repeter ce qui a été indiquer au dessus, il vaut mieux tester l'etat, et laisser le constructeur faire son boulot. =)