Discussion :

[eldrad95]

RE bonjour (et désolé d'être un boulet ).

J'ai une winform(requeteur) qui en appelle une autre(visu). (création d'un rapport).

Si je ferme la visu et que je la relance depuis requeteur j'ai une erreur au moment d'accéder à un des objet contenu dans visu comme si la fenetre existait dejà.

quand je fais une exécution pas à pas je remarque que mon objet visu existe toujours bel et bien au moment de la seconde requête.
je pensait que lorsque l'on utilise un close sur une winform l'objet en question était détruit (ou au moins remis à null).
est ce normal ? ou est ce que me suis planté ?

[jacky01]

Bonjour,

En effet lorsque tu fait un from.close() l'objet est détruit.
http://msdn.microsoft.com/fr-fr/libr...orm.close.aspx

Ce qui m'interpèle c'est que tu dit que tu veux accéder à ta form hors si tu la close (donc détruit) il faudra donc la reconstruire fatalement et j'ai pas l'impression que c'est cela que tu fait.

Donc tu as deux solutions :

- soit tu recréé une instance de ta form
- soit tu fait simplement un form.Hide() de ta visu , puis form.Show() quand tu veux la ré-afficher en modifiant les valeurs des controls avant si besoin.

[eldrad95]

normalement si voila l'extrait de mon code :

Code :

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

1
2
3
4
5
6
7
8
9
10
 if (visu == null)
                {
 
                    visu = new visualisation_rapport(rptviewer);
                    visu.Show();
                }
                else
                {
                    visu.rechargement_doc(rptviewer);
                }

visu est la form appelé.
visu.chargement_doc permet d'uniquement recharger le document contenu dans le viewer (active report) de la form sans avoir à tout relancer.
Normalement avec ce code quand je ferme ma form visu elle redevient nul et est recréée la fois d'après ? non ?

ps : après vérif, mon objet visu existe toujours bel et bien une fois this.Close() lancé.

[jacky01]

Peux-tu envoyer le message d'erreur ?

[eldrad95]

voici l'erreur :

La référence d'objet n'est pas définie à une instance d'un objet.

Elle se produit sur viewer qui est l'object qui affiche les rapport (object en parametre).

j'ai reussi à m'en sortir je pense pas que ça soit super propre par contre

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
17
18
19
20
 if (visu == null)
                {
 
                    visu = new visualisation_rapport(rptviewer);
                    visu.Show();
 
                }
                else
                {
                    if (visu.IsDisposed == true)
                    {
                        visu = null;
                        visu = new visualisation_rapport(rptviewer);
                        visu.Show();
                    }
                    else
                    {
                        visu.rechargement_doc(rptviewer);
                    }
                }

j'ai rajouté la vérif du visu.IsDisposed qui me permet de remettre l'object à "null" si il a été fermé.

[lcfseth]

Pourquoi ne pas appeler visu en tant que dialogbox?

Le code est le même à chaque fois :

Code :

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

1
2
3
 
visu = new visualisation_rapport(rptviewer);
 visu.ShowDailog();

Si tu souhaite conserver la possibilité de switcher de l'une à l'autre pourquoi ne pas utiliser la fonction hide. Tu recupere l'evenemnt closing de la fenetre visu et tu cache la fenetre plutot que de recréer une nouvelle instance à chaque fois.

Code :

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

1
2
3
4
5
6
 
private void Visu_Closing(object sender, System.ComponentModel.CancelEventArgs e)
{
   Visu.Hide(); //Cache la fenetre
   e.Cancel = true; //Evite la fermeture de la fenetre.
}