Discussion :

[Nadd]

Bonsoir!

Me voila lancé dans la création de ma première application Windows en C#
Etant débutant, il me reste encore beaucoup de subtilités à acquérir et c'est pour cela que je me tourne vers vous aujourd'hui

J'ai opté pour une application MDI ! J'ai donc ainsi crée, ormis la Form1, une nouvelle Form (appellé Login) qui a pour unique but de se connecter (ou plutôt d'utiliser une connexion éxistante préalablement créee dans la Form1) à une base de donnée afin de vérifier le contenu de deux textboxs Tout ceci afin de faire passer à "true" la propriété Enable de divers Menu (de Form1 bien entendu). Seule Hic, comment accéder à ma connexion SQL et aux divers composants de ma fenêtre principale, tels que mes menus, à partir de ma fenêtre Login ?

En vous remerçiant d'avance,

Nadd

[Nadd]

Je viens d'essayer de modifier une propriété enable de ma Form1 en créant (dans le code de celle-ci) une fonction public permettant de faire passer à true la propriété enable en question à partir de la Form2 :

Form2.cs

Code :

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

1
2
3
4
5
6
 
private void bLogin_Click(object sender, EventArgs e)
{
    Dollar dollar = (Dollar)this.MdiParent;
    dollar.SetEnable();
}

Form1.cs

Code :

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

1
2
3
4
5
 
public void SetEnable()
{
   déconnectionToolStripMenuItem.Enabled = true;
}

Ca marche, toutefois le code me semble un peu "primaire". J'ai surement dû passer à côté de quelque chose d'essentiel, mais quoi ?

Le problème réside, je pense, dans le fait que les composants tels que les Menus sont privées. Dois-je les rendre publiques ou existe-t-il une autre manière de faire ?

Note : Quand est-il du mot clef Internal ?

Cordialement,

Nadd!

[SaumonAgile]

Le problème avec cette approche est que tes fenêtres sont très fortement dépendantes. Pourquoi ne pas utiliser une classe séparée pour exécuter les requêtes et ainsi ne pas avoir à stocker les paramètres tel que les chaines de connexion dans les fenêtres.
Une deuxième chose, il serait bon de ne pas faire des appels dans les deux sens entre les fenêtres, par exemple form1 crée form2, mais form2 utilise des méthodes de form1. Dans ce cas, j'utiliserais soit une classe de base pour toutes les formes; soit des événements dans form2. je m'explique :
form1 instancie une form2 et garde une référence sur l'instance. form2 contient un événement statusChanged. form1 s'abonne à l'événement de form2 et quand l'événement est déclenché, exécute une action donnée.

En pratique ça donne :

Dans le chargement de form1 (ou un autre événement) :

login = new FLogin(); // login peut être un attribut de classe
FLogin.LoginSuccessful += new EventHandler(FLogin_loginSuccessful);

Dans le gestionnaire d'événement :
private void FLogin_loginSuccessful(object sender, EventArgs event)
{
login.Close();
}

m2c...

[SaumonAgile]

Le mot-clé internal sert à définir que l'attribut/propriété/méthode/etc est accessible dans l'assembly. Je m'en sers quand je développe des bibliothèques, mais je ne pense pas que ce soit une bonne solution pour ton problème.

[Nadd]

Merci pour ces réponses !

Pourquoi ne pas utiliser une classe séparée pour exécuter les requêtes et ainsi ne pas avoir à stocker les paramètres tel que les chaines de connexion dans les fenêtres.

Pouriez-vous (si possible, loin de moi l'idée de vous faire perdre votre temps) me fournir un exemple concret quant à l'utilisation d'une classe séparée accessible depuis Form1 et Form2 ?

- La création de cet objet doit-il se faire dans Main ?

Cordialement,

Nadd

[SaumonAgile]

Utilise plutot une classe statique (qui contient des méthodes déclarées avec le mot clé static) qui te permet de ne pas créer d'instance de cette classe. Tu peux aussi utiliser le design pattern singleton qui te permet de pouvoir accéder à une instance d'une classe donnée (toujours la même).

Par exemple, tu déclares une classe SqlHelper avec une méthode :

Code :

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

1
2
public static ClasseClient getClient(string ID)
{...}

et dans tes fenêtres tu appelles :

Code :

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

SqlHelper.getClient(monId);

PS : Si tu n'es pas familier avec les design patterns, je ne saurais que te conseiller de le devenir au plus vite.