Discussion :

[terminator59140]

J'ai un problème sous VC#2008! Je suis entrain de développez une application et je voudrais que lorsque je clique sur un bouton par exemple, il ne m'affiche qu'une seul fois le contenu de la fenêtre.

Voila le code pour créer la fenêtre

Code :

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

1
2
3
4
5
        private void paramètreToolStripMenuItem_Click(object sender, EventArgs e)
        {
            MKParametre MKActiveParametre = new MKParametre();
            MKActiveParametre.Show();
        }

Si vous pouvez m'aider, je suis nouveau dans ce langage! Merci d'avance pour votre aide.

[binoo]

Tu peux en dire plus sur ton problème. Le code (que tu as fourni) est celui utilisé pour afficher une fenêtre / boite de dialogue.

[Graffito]

MKActiveParametre étant une variable locale de la procedure, une forme sera créée à chaque appel.
Il suffit de déclarer la variable au niveau de la forme principale

Code :

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

MKParametre MKActiveParametre = null

et de la créer une seule fois dans la procedure :

Code :

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

if (MKActiveParametre != null)  MKActiveParametre= new MKParametre();

[terminator59140]

Tu peux en dire plus sur ton problème. Le code (que tu as fourni) est celui utilisé pour afficher une fenêtre / boite de dialogue.

Bonsoir Binoo! Ben c'est pour afficher une fenêtre que j'ai faite et que que je voudrais afficher une seule fois et qu'elle n'apparaît pas lorsque je re-clique sur ce bouton! Sauf si je referme la fenêtre.

[terminator59140]

MKActiveParametre étant une variable locale de la procedure, une forme sera créée à chaque appel.
Il suffit de déclarer la variable au niveau de la forme principale

Code :

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

MKParametre MKActiveParametre = null

et de la créer une seule fois dans la procedure :

Code :

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

if (MKActiveParametre != null)  MKActiveParametre= new MKParametre();

Bonsoir Graffito!J'ai essayée ta méthode et il me met une erreur de type:

Erreur 1 Une variable locale nommée 'MKActiveParametre' ne peut pas être déclarée dans cette portée, car elle modifierait la signification de 'MKActiveParametre', déjà utilisée dans une portée 'parent ou en cours' pour désigner autre chose


et voici le code que j'ai modifié avec l'erreur en rouge:

Code :

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

1
2
3
4
5
6
7
8
9
        private void paramètreToolStripMenuItem_Click(object sender, EventArgs e)
        {
            MKParametre MKActiveParametre = null;
            if (MKActiveParametre != null)
            {
                MKParametre MKActiveParametre = new MKParametre();
                MKActiveParametre.Show();
            }
        }

[Graffito]

MKParametre MKActiveParametre = new MKParametre();

Il ne faut pas redéclarer MKActiveParametre dans la procédure :

Code :

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

MKActiveParametre = new MKParametre();

et surtout déclarer : private MKParametre MKActiveParametre =null ; en dehors de la procedure.
ce qui donne:

Code :

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

1
2
3
4
5
6
7
private  MKParametre MKActiveParametre = null;
 
private void paramètreToolStripMenuItem_Click(object sender, EventArgs e)
{
   if (MKActiveParametre != null)  { MKActiveParametre = new MKParametre(); }
   MKActiveParametre.Show();
}

[terminator59140]

Merci pour cet exemple Graffito! Je suis encore un débutant! Le problème est partiellement résolu! Car là il s'agit d'une histoire de pile des appels! J'essaie de regarder sur l'aide en ligne de microsoft, il me met cet exemple:

"Dim CustomerTable As DataTable " doit être réécrit comme suit "Dim CustomerTable As New DataTable."

Là je suis perdu!

[binoo]

Il y a eu une petite erreur dans le code fourni , il faut mettre :

Code :

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

if (MKActiveParametre == null)

En fait tu regardes si ton objet MKActiveParametre a été instancié, si ce n'est pas le cas il est null et donc tu instancies celui-ci par :

Code :

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

MKActiveParametre = new MKParametre();

[fally]

Moi je ferais ceci :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
public object IsAlreadyOnScreen(Type t)
{
 foreach(Form f in this.MdiChildren)
  {
  if (f.GetType() == t)
   {
   return f;
   }
  }
 return null;
}

et avant d'ouvrir une fenetre, je fais :

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
private void paramètreToolStripMenuItem_Click(object sender, EventArgs e)
{
try
{
Form f = (Form)IsAlreadyOnScreen(typeof(MKActiveParametre));
f.BringToFront();
return; 
}
catch
{
}
MKAParametre MKActiveParametre= new MKParametre();
MKActiveParametre.MdiParent = this; // ta fenetre principale doit etre MdiContainer
MKActiveParametre.Show();
}

[terminator59140]

Il y a eu une petite erreur dans le code fourni , il faut mettre :

Code :

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

if (MKActiveParametre == null)

En fait tu regardes si ton objet MKActiveParametre a été instancié, si ce n'est pas le cas il est null et donc tu instancies celui-ci par :

Code :

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

MKActiveParametre = new MKParametre();

Merci Binoo! C'était bien ça! Il fallait mettre " == null "

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
        private MKParametre MKActiveParametre = null;

        private void paramètreToolStripMenuItem_Click(object sender, EventArgs e)
        {
            if (MKActiveParametre == null)
            {
                MKActiveParametre = new MKParametre();
            }
            MKActiveParametre.Owner = this;
            MKActiveParametre.Show();
           
        }

Mais une fois que je quitte la fenêtre, pour après la réouvrir il me met :

[terminator59140]

Salut fally! Je suis un débutant dans C# et visual Studio! Donc je ne sais pas ou trop mettre le code que tu m'as fourni:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
public object IsAlreadyOnScreen(Type t)
{
 foreach(Form f in this.MdiChildren)
  {
  if (f.GetType() == t)
   {
   return f;
   }
  }
 return null;
}

Donc je l'ai mis au-dessus de ceci:

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
private MKParametre MKActiveParametre = null;

private void paramètreToolStripMenuItem_Click(object sender, EventArgs e)
{
try
{
Form f = (Form)IsAlreadyOnScreen(typeof(MKActiveParametre));
f.BringToFront();
return; 
}
catch
{
}
MKParametre MKActiveParametre= new MKParametre();
MKActiveParametre.MdiParent = this; // ta fenetre principale doit etre MdiContainer
MKActiveParametre.Show();
}

Et lorsque je compile, il m'indique qu'il y a une erreur de la part de typeof:

Erreur 1 'MKActiveParametre' est un 'champ' mais est utilisé comme un 'type'

[fally]

rempalce

Code :

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

Form f = (Form)IsAlreadyOnScreen(typeof(MKActiveParametre));

par

Code :

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

Form f = (Form)IsAlreadyOnScreen(typeof(MKParametre));

erreur de ma part

[terminator59140]

rempalce

Code :

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

Form f = (Form)IsAlreadyOnScreen(typeof(MKActiveParametre));

par

Code :

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

Form f = (Form)IsAlreadyOnScreen(typeof(MKParametre));

erreur de ma part

Merci Fally ! Ceci fonctionne très bien! Je quitte la fenêtre, elle réapparait automatiquement!

Mais avant que je mette résolu! Je voulais savoir pourquoi la fenêtre devient gris foncée avec un genre de style 3D enfoncée???

[fally]

Merci Fally ! Ceci fonctionne très bien! Je quitte la fenêtre, elle réapparait automatiquement!

Mais avant que je mette résolu! Je voulais savoir pourquoi la fenêtre devient gris foncée avec un genre de style 3D enfoncée???

j'ai pas bien compris!

[terminator59140]

j'ai pas bien compris!

Ben mon application était tout a fait normal! Ben dans la Form dans l'onglet Propriété j'avais mis un BackColor "Sky Blue"! Puis comme ça ne fonctionnait pas et qu'il fallait définir la fenêtre principale comme une MDI (ce que tu m'avais mis en commentaire "//"). Donc dans l'onglet propriété de la Form, je suis allait sur Style de fenêtre et j'ai mis à true IsMdiContainer.

J'espère t'avoir éclairé un peu!

[fally]

Ben mon application était tout a fait normal! Ben dans la Form dans l'onglet Propriété j'avais mis un BackColor "Sky Blue"! Puis comme ça ne fonctionnait pas et qu'il fallait définir la fenêtre principale comme une MDI (ce que tu m'avais mis en commentaire "//"). Donc dans l'onglet propriété de la Form, je suis allait sur Style de fenêtre et j'ai mis à true IsMdiContainer.

J'espère t'avoir éclairé un peu!

J'ai remarqué que quand une fenetre est MdiContainer, on ne peut plus modifier la couleur d'arriere plan....j'ai meme eu a poster un sujet a propos mais pas de reponse depuis
tu vas devoir faire avec alors

[terminator59140]

J'ai remarqué que quand une fenetre est MdiContainer, on ne peut plus modifier la couleur d'arriere plan....j'ai meme eu a poster un sujet a propos mais pas de reponse depuis
tu vas devoir faire avec alors

Je vais allais voir sur msdn pour voir ce que signifie MdiContainer et voir si on peux faire autrement! Car c'est monotone la couleur. Merci pour ton aide...

[fally]

Tiens moi au courant!

[SaumonAgile]

Moi je ferais ceci :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
public object IsAlreadyOnScreen(Type t)
{
 foreach(Form f in this.MdiChildren)
  {
  if (f.GetType() == t)
   {
   return f;
   }
  }
 return null;
}

C'est pas mal. Tu devrais utiliser des noms de variable explicites, et tu peux remplacer le test par :

Code :

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

if (f is t)

et avant d'ouvrir une fenetre, je fais :

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
private void paramètreToolStripMenuItem_Click(object sender, EventArgs e)
{
try
{
Form f = (Form)IsAlreadyOnScreen(typeof(MKActiveParametre));
f.BringToFront();
return; 
}
catch
{
}
MKAParametre MKActiveParametre= new MKParametre();
MKActiveParametre.MdiParent = this; // ta fenetre principale doit etre MdiContainer
MKActiveParametre.Show();
}

Par contre ça, c'est vraiment sale. Les exceptions ne servent pas à ça. Au lieux de mettre un try catch, il faut simplement tester si IsAlreadyOnScreen retourne null : si oui on affiche la fenêtre, sinon crée une nouvelle instance qu'on affiche.

[fally]

Absolument ok avec toi mais bon un code pondu rapido est très souvent pas optimal...