[C#] tester Fenetre déja ouverte?

**codez** · 05/04/2006, 18h26

Bonjour,

Dans une Application MDI j'ai une menu qui permet d'ouvrir un certain nombre de fenetre filles de la fenetre MDI parente.

Pour l'ouverture d'une des fenetre filles j'utilise le code suivant sur l'evenement click des elements du menu.

Exemple :

Code :

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

1
2
3
4
5
6
7
8
 
        private void articlesToolStripMenuItem_Click(object sender, EventArgs e)
        {
            frm_articles FRM_ARTICLES = new frm_articles();
            FRM_ARTICLES.MdiParent = frm_main.ActiveForm;
            FRM_ARTICLES.Show();
 
        }

L'ouverture marche correctement, par contre si je clique une deuxieme fois sur le meme element du menu cela m'ouvre une deuxieme fenetre identique. Vous me direz c'est normal vu que je fais a chaque fois

Code :

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

1
2
 
frm_articles FRM_ARTICLES = new frm_articles();

Ma question est donc comment faire en sorte que lorsque la fenetre est déja ouverte on ne puisse pas l'ouvrir une deuxieme fois.

Merci d'avance.

**pas05** · 05/04/2006, 18h45

moi j'ai fais ceci ca fonctionne mais je pense qu'il ya mieux

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
 
 
             try
            {
                Form fr = this.ActiveMdiChild;
 
                if (fr != null)
 
                    fr.Close();
 
 
                FrmArticles frmArticle = new FrmArticles(dsjac);
 
 
                frmArticle.MdiParent = this;
 
                frmArticle.Show();
 
            }

**Thomas Lebrun** · 05/04/2006, 19h04

Plus simple:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
private frm_articles FRM_ARTICLES = null;
 
private void articlesToolStripMenuItem_Click(object sender, EventArgs e)
        {
            if ( FRM_ARTICLES == null)
           {
                FRM_ARTICLES = new frm_articles();
                FRM_ARTICLES.MdiParent = frm_main.ActiveForm;
                FRM_ARTICLES.Show();
            }
        }

**codez** · 05/04/2006, 19h09

Merci Morpheus, ca marche par contre une fois que j'ai fermer ma fenetre fille, impossible de la réouvrir ...

**pas05** · 05/04/2006, 19h32

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
 
 
private frm_articles FRM_ARTICLES = null;
 
private void articlesToolStripMenuItem_Click(object sender, EventArgs e)
        {
            if ( FRM_ARTICLES == null)
           {
                FRM_ARTICLES_= new frm_articles();
                FRM_ARTICLES.MdiParent = frm_main.ActiveForm;
               FRM_ARTICLES..Disposed += new EventHandler(FRM_ARTICLES._Disposed);
                FRM_ARTICLES.Show();
            }
        } 
void FRM_ARTICLES_Disposed(object sender, EventArgs e)
        {
            //FRM_ARTICLES.Dispose();//
 
            FRM_ARTICLES= null;
        }

**Merlin** · 05/04/2006, 19h40

vos méthodes marchent mais j'avoue que je n'aime pas cette technique qui repose sur des variables (une par fenêtre MDI fille). Je trouve ça peu évolutif et lourd puisque chaque ouverture de fille doit tester "sa" variable. Avec déjà un risque de bug sur lequel notre ami est tombé de plein pied du premier coup : l'oublie de remise à null de la variable...

En réalité, un moyenplus propre et beaucoup plus générique consiste à créer une méthode "OuvrirFille(system.Type classeFille)" qui :
1/ balaye toutes les fenêtres filles déjà ouverte et teste leur classe vis à vis du paramètre passé,
2/ qui instancie la classe passée en paramètre

avec cette méthode, ça marche à tous les coups, c'est propre, évolutif, et si ça marche une fois ça marche toujours.

un code du genre du suivant par exemple (en supposant que la classe recherchée soit MyChildForm) :

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
21
22
foreach(Form f in this.MdiChildren)
     {
          if(f is MyChildForm)
          {
               // found it
               childForm = (MyChildForm) f;
               break;
          }
     }
 
     if( childForm != null)
     {
          childForm.Show();
          childForm.Focus();
     }
     else
     {
          childForm = new MyChildForm();
          childForm.MdiParent = this;
          childForm.Show();
          childForm.Focus();
     }

**NicolasG** · 05/04/2006, 22h19

Envoyé par Merlin

vos méthodes marchent mais j'avoue que je n'aime pas cette technique qui repose sur des variables (une par fenêtre MDI fille).

Le problème, c'est juste que si on ouvre une fenêtre, c'est le plus souvent pour récupérer des entrées utilisateur.
Pour ça, c'est plus pratique d'avoir une variable non?

**Merlin** · 05/04/2006, 23h34

Envoyé par NicolasG

Le problème, c'est juste que si on ouvre une fenêtre, c'est le plus souvent pour récupérer des entrées utilisateur.
Pour ça, c'est plus pratique d'avoir une variable non?

C'est pas forcément la règle, une fenêtre traite en général ses saisies, sauf certains dialogues (genre saisie d'un login dans une petite fenêtre).

Mais dans tous les cas créer un couplage entre les fiches est une mauvaise idée, et quand on le fait on doit y réfléchir à deux fois.

De même, entre la méthode que j'ai proposée et l'autre qui a été débattue dans ce thread, la première fonctionne dans tous les cas sans avoir à connaître toutes les classes des fenêtres filles, l'autre réclame la copie d'un code identique pour chaque classe. Cette dernière créé un coupable fort entre la méthode qui créée les filles et ces dernières alors que la première met en place à l'inverse un découpable fort. Elle est donc préférable.
Plus les acteurs sont indépendants moins on risque d'avoir du code spaghetti et plus il est facile de le faire évoluer (maintenance corrective et évolutive).
Dans l'absolu il faut toujours viser quelque chose qui se rapproche du modèle MVC, on est rarement déçu :-)

**NicolasG** · 06/04/2006, 08h59

Je suis entièrement d'accord avec ta position mais si tu regardes attentivement le code proposé par pas05, tu peux remarquer que le handler de dispose qui remet à null la variable de fenêtre fille fait partie de la fenêtre mère.
L'implémentation de la fenêtre fille ne dépends donc pas du mécanisme de remise à zéro de cette variable.
Le critère d'indépendance est donc bien rempli.

**NiamorH** · 06/04/2006, 09h57

moi j'ai toujours utilisé la méthode de Merlin qui me semble la plus logique.
en VB.Net pour ceux qui veulent :

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
    Private Sub MNUDossiers_Click(ByVal sender As System.Object, ByVal e As System.EventArgs) Handles MNUDossiers.Click
        Dim Frm As Form
 
        For Each Frm In Me.MdiChildren
            If TypeOf Frm Is frmAccueilDossier Then
                If Frm.WindowState = FormWindowState.Minimized Then
                    Frm.WindowState = FormWindowState.Normal
                End If
                Frm.Activate()
                Exit Sub
            End If
        Next
 
        Dim f As New frmAccueilDossier
 
        '    f.Initialiser(oMesDonnees)
        f.MdiParent = Me
        f.Show()
    End Sub

la ligne commentaire me permet d'initialiser ma fenetre (datagrid etc.) avec mon dataset

**codez** · 07/04/2006, 01h11

Ok merci a tous, je vais essayer toutes vos propositions et choisir celle qui me convient le mieux.

**Cazaux-Moutou-Philippe** · 27/04/2006, 13h30

Ayant eu le meme soucis, voici la solution qui m a été proposée et qui me conviens

a++

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
21
22
23
24
        private void mListeFilms_ItemClick(object sender, DevExpress.XtraBars.ItemClickEventArgs e)
        {
            Boolean bOpen = false;
            foreach (Form f in Application.OpenForms)
            {
                if (f.Name == "FFilmsListe")
                {
                  // MessageBox.Show("oui");
                    bOpen = false;
                }
                else
                {
                    bOpen = true;
                }
            }
 
            if (bOpen == true)
            {
                FFilmsListe aFFilmsListe = new FFilmsListe(this);
                // Affichage de la fenetre
                aFFilmsListe.Show();
            }
 
        }

**davcha** · 28/04/2006, 00h47

Et pourquoi pas ça ?

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
21
22
23
sealed class frm_articles : Form {
  private static frm_articles myInstance = null;
 
  private frm_articles(){
    InitializeComponent();
  }
 
  public static void Display(Form parent){
    if(myInstance == null){
      myInstance = new frm_articles();
      myInstance.MdiParent = parent.ActiveForm;
      myInstance.Show();
    }
  }
  private void frm_articles_FormClosed(object sender,FormClosedEventArgs e){
    myInstance = null;
  }
}
 
private void articlesToolStripMenuItem_Click(object sender, EventArgs e)
{
  frm_articles.Display(frm_main);
}