Discussion :

[SQLpro]

Bonjour,

Je dispose d'une main form qui a une variable de nom serverName déclarée de manière globale :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
using System;
...
using System.Windows.Forms;

namespace WFA_sql
{
    public partial class FRM_retrosert : Form
    {
        string serverName;
        public FRM_retrosert()
        {
            InitializeComponent();
...

j'ai ouvert une boîte de dialogue depuis un main form, par ce code :

Code :

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

1
2
3
4
5
6
7
        private void TB_ServerName_DoubleClick(object sender, EventArgs e)
        {
            Cursor.Current = Cursors.WaitCursor;
            WFD_list Frm_list = new WFD_list();
            Frm_list.ShowDialog();
            Cursor.Current = Cursors.IBeam;
        }

J'aimerais pouvoir placer une valeur dans la variable serverName depuis la boîte de dialogue, venant de l'item sur lequel est positionné le focus dans la ListBox de la boite de dialogue.

Pour cela j'avais envisagé le code suivant :

Code :

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

1
2
3
4
5
private void BTN_ok_Click(object sender, EventArgs e)
        {
            FRM_retrosert.serverName = LB_servers.Items[LB_servers.SelectedIndex];
 
        }

Bien évidemment la boite de dialogue ne "voit" pas la main form...

Bref, je suis à la recherche d'une solution qui permet de remonter une valeur.

D'avance merci

A +

[SQLpro]

Bon, je me répond à moi même, j'ai fait l'inverse :
1) déclaration d'une chaine publique dans la boite de dialogue :

Code :

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

1
2
3
    public partial class WFD_list : Form
    {
        public string serverNameSet;

2) assignation de cette chaine dans le clic du bouton OK :

Code :

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

1
2
3
4
5
6
        private void BTN_ok_Click(object sender, EventArgs e)
        {
            serverNameSet = LB_servers.Items[LB_servers.SelectedIndex].ToString();
            MessageBox.Show(serverNameSet);
            this.Close();
         }

3) reprise de cette info après l'appel de la boite de dialogue :

Code :

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

1
2
3
4
5
6
7
8
9
private void TB_ServerName_DoubleClick(object sender, EventArgs e)
        {
            Cursor.Current = Cursors.WaitCursor;
            WFD_list Frm_list = new WFD_list();
            Frm_list.ShowDialog();
            serverName = Frm_list.serverNameSet;
            TB_ServerName.Text = serverName;
            Cursor.Current = Cursors.IBeam;
        }

C'est peut être pas très joli ?

A +

[Graffito]

C'est peut être pas très joli ?

Y'a pas à en rougir .

Mais le plus propre serait de créér une procédure dédiée retournant le nom du serveur dans la form WFD_list.

Code :

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

1
2
3
4
5
6
internal DialogResult MyShowDialog(out string ServerName)
{
   DialogResult Result=ShowDialog() ;
   ServerName= serverNameSet ;
   return Result ;
}

ou

Code :

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

1
2
3
4
internal string MyShowDialog()
{
   return ShowDialog()==DialogResult.OK ? serverNameSet : "" ;
}

Sur un autre point qui n'a rien à voir, je te conseillerai de gérer systématiquement les curseurs par 2 procédures static : PushCursor(Cursor) et PopCursor(). Ceci, afin de ne pas avoir de soucis quand on fait des appels imbriqués à des procédures modifiant le curseur.

[tomlev]

C'est peut être pas très joli ?

C'est la façon "standard" de faire ce genre de chose. Les dialogues prédéfinis genre OpenFileDialog/SaveFileDialog fonctionnent d'ailleurs suivant ce principe.

Par contre, utilise une propriété plutôt qu'un champ... Les champs publics sont généralement considérés comme une mauvaise pratique, parce qu'ils permettent de modifier l'état interne de l'objet sans que tu puisses le contrôler

[SQLpro]

OK pour la propriété... Je ferais mieux la prochaine fois !

Merci

A +