Discussion :

[the java lover]

Bonjour,

J'ai défini une méthode publique dans un form (Form1).
J'ouvre une popup (Form2) depuis ce form.

Lorsque je clique sur un bouton de Form2, je souhaite invoquer la méthode publique définie dans Form1.

private void button1_Click(object sender, EventArgs e)
{
Form1.xxxxx ???
this.Close();
}

La complession ne me permet pas d'accéder à cette méthode. Je ne sais pas comment faire...

Merci

[Thomas Lebrun]

Ta méthode dans Form1 doit être marquée comme étant static

[Mose]

Niveau architectural, c'est très très mal ce que tu veux faire.

Ensuite :
* Si tu veux appeler une méthode d'instance, il faut faire l'appel depuis une instance, pas depuis la classe.
* Si tu veux appeler une méthode de classe (comme tu 'las fait), il faut déclarer ta méthode comme une méthode de classe (voir la réponse de Thomas)

[caviar]

Bonjour,

Je débute en C#,

Mose, suite à ta réponse, je me pose la question suivante :

tu indiques de passer par une instance de la classe, cela veut t'il dire que dans form2, tu ajoutes le code suivant :

private void BtnRetourForm1_Click(object sender, System.EventArgs e)
{
Form1 maForm = new Form1();
this.Hide();
maForm.Owner = this;
maForm.ShowDialog();
}

Mais alors, à chaque fois que l'on va switcher entre la form1 et la Form2,
il faut recréer une nouvelle instance de la class Form1 ?

Est ce vraiment "clean" de procéder de cette manière ? (n'y a t'il pas de saturation mémoire au bout de n switch ? )

Est ce que Le garbage Collector gère en auto la destruction de l'instance à chaque switch ?

caviar

[seb8327]

Bonjour,

Il y a un autre moyen de faire :
1 - Ajouter un constructeur dans la Form2 avec un paramètre de type Form
2 - Ajouter dans la Classe Form2 une donnée membre de Type Form qui sera initialisée par le constructeur créer précedement.
3 - Dans la création de la Form2 qui est faite dans la Form1 il faut utiliser le constructeur créer précedement en passant en argument This.

Exemple de code :
//-----------------------
//Dans la Classe Form2
//-----------------------

// Constructeur
public Form2(Form formAppelante)
{
InitializeComponent();
referenceFormAppelante = formAppelante;
}

// Donnée membre
private Form referenceFormAppelante = null;

// Méthode liée au Click
private void button1_Click(object sender, EventArgs e)
{
referenceFormAppelante.MethodeTruc();
this.Close();
}

//-----------------------
//Dans la Classe Form1
//-----------------------

public void Methode1()
{
Form2 f = new Form2(this);
f.ShowDialog();
}
De cette façon tu garde une référence sur la Form appelante et donc tu peux faire appel à des méthodes d'instances.

[the java lover]

Par rapport à la réponse de Mose, qu'est ce qui n'est pas bien ?

J'ai fais passer l'instance de mon Form dans le constructeur. Et cela fonctionne.

Pour le reste il me semblait propre de faire effectuer un traitement par le composant concerné (mettre à jour des champs) lui-même.

Merci à tous

[Mose]

[B]Mais alors, à chaque fois que l'on va switcher entre la form1 et la Form2, il faut recréer une nouvelle instance de la class Form1 ?

Ah non, je t'ai jamais dit d'instancier une nouvelle form.
En revanche, tu peux passer la référence de ta Form1 à ta Form2.

Bon, ceci dit c'est pas très très bien non plus.
Normallement, il faut voir une appli comme une hiérarchie : la Form de base peut connaître tous ses sous-contrôles, et les sous-Form et les sous-contrôles doivent revenir vers elle via des évènements.
Ca te permet de conserver une certaines indépendance entre tes contrôles et d'éviter de faire du code "spaghetti".

Par rapport à la réponse de Mose, qu'est ce qui n'est pas bien ?
Pour le reste il me semblait propre de faire effectuer un traitement par le composant concerné (mettre à jour des champs) lui-même.

Que le composant agisse sur les données, OUI.
Qu'il agisse directement sur le contrôle qui le contient, NON.
Exemple :
* Form1 a un label "Nom"
* Pour modifier le nom, tu fais apparaitre une boite de dialogue Form2 "saisissez le nom".
* Deux choix :
1 - (mal) : Form2 modifie le label contenant le nom dans Form1
2 - (bien) :
* Dans Form2 tu créé une propriété Nom qui retourne le nom qu'a entré l'utilisateur (un simple return _txtNom.Text;)
* Form1 détecte que Form2 se ferme quand la méthode ShowDialog() se termine
* Form1 récupère alors le nom entré dans Form2 grâce à la propriété Nom.

[the java lover]

OK nickel et merci pour ces précieux conseils...

Le cas que tu décris est exactement celui que j'ai codé...
Comment la form appelante détecte que la form appelée se ferme ?
Pour mon cas, j'ai passé l'appelante ds le constructeur de l'appelée... et j'ai invoqué la méthode sur l'appelante...

Enfin, pourrais-tu m'indiquer un bon cours (pdf) ou on explique comment bien structurer une application ? Un bon cours C#...

Merci beaucoup.

[The_badger_man]

Comment la form appelante détecte que la form appelée se ferme ?
Pour mon cas, j'ai passé l'appelante ds le constructeur de l'appelée... et j'ai invoqué la méthode sur l'appelante...

dans le cas d'un showdialog(), la form 1 est bloquée tant que la form 2 n'est pas fermée. Donc le code en dessous de showdialog() dans form1 sera executé forcement quand form2 sera fermée.