Comment appeler une methode dans une autre Form

**moonshine972** · 23/03/2011, 11h08

Bonjour,
J'écris une application qui me permet d'écrire dans une base Mysql
j'ai une form1 qui et une form2
la form1 est une fenetre d'identification qui me permet de me connecter a la base, une fois la connexion vérifiré j'accede a la form2
dans la form2 j'ai des champs ou je peux ecrire des données sur la base
mais voila mon problème j'obtiens un message d'erreur suivant quand je valide les champs

L'exception System.InvalidOperationException n'a pas été gérée
Message=Connection must be valid and open.

Voici le code de la form1:

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
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
namespace WpfApplication1
{
    /// <summary>
    /// Logique d'interaction pour MainWindow.xaml
    /// </summary>
     public partial class MainWindow : Window
    {
        MySqlConnection wConnexion = new MySqlConnection();
        public MainWindow()
        {
            InitializeComponent();
        }
 
        private void button1_Click(object sender, RoutedEventArgs e)
        {
            // Connexion à la BDD
 
            wConnexion.ConnectionString = "Database=test;Data Source=localhost;User Id="+textBox1.Text +";Password="+ passwordBox1.Password;
 
            // Tenter la connexion sinon afficher un message d’erreur
            try
            {
                wConnexion.Open();
            }
            catch (Exception ex)
            {
                MessageBox.Show("Impossible de se connecter au serveur!");
                MessageBox.Show(ex.Message);
            }
 
            wConnexion.Close();
 
            Window1 myfen = new Window1();
            myfen.Show();
 
 
            this.Close();         
        }
 
 
        private void button2_Click(object sender, RoutedEventArgs e)
        {
            this.Close();
        }
 
        private void passwordBox1_PasswordChanged_1(object sender, RoutedEventArgs e)
        {
 
        }
    }
}

et Voici le code de la form2

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
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
namespace WpfApplication1
{
    /// <summary>
    /// Logique d'interaction pour Window1.xaml
    /// </summary>
     public partial class Window1 : Window
    {
        MySqlConnection wConnexion = new MySqlConnection();
 
         public Window1()
        {
            InitializeComponent();
        }
        private void textBox1_TextChanged(object sender, TextChangedEventArgs e)
        {
 
        }
 
        private void button1_Click_1(object sender, RoutedEventArgs e)
        {
 
            using (wConnexion)
            {
                // Instancier l’objet Command
                using (MySqlCommand wCommand = wConnexion.CreateCommand())
                {
                    // Définition de la requête
                    wCommand.CommandText = "INSERT INTO `hibis_t`(`nom` ,`prenom` ,`message`) VALUES (" + textBox1.Text + ", "+ textBox2.Text +"," + textBox3.Text + ")";
                    //wCommand.CommandText = "UPDATE INTO `hibis_t`(`nom` ,`prenom` ,`message`) VALUES (" + textBox1.Text + ", " + textBox2.Text + "," + textBox3.Text + ")";
 
 
                    // Exécution de la requête
                   // try
                    //{
                        wCommand.ExecuteNonQuery();
                    //}
                    //catch (InvalidOperationException)
                    //{
                      //  MessageBox.Show("Erreur impossible d'écrire dans la table!");
                    //}
 
                    wConnexion.Close();
                }
            }
        }     
 
    }
}

comme vous pouvez le voir je suis obligé de remettre la methode MySqlConnection wConnexion = new MySqlConnection(); une deuxieme fois alors que la connexion a la base est deja active
comment puis je utiliser la methode de la form1 dans la form2 svp?

Merci

**Gurdil le nain** · 23/03/2011, 11h16

Premièrement

Ensuite, dans ta mainWindow, tu fermes la connexion avant même d'appeler la Window1...forcément tu n'auras plus de connexion ouverte.

Pour avoir une même connexion entre les différents formulaires, soit tu fais un singleton, soit tu passes ta connexion en paramètre aux formulaires qui en ont besoin.
Personnellement, je trouve le singleton plus propre.

**moonshine972** · 23/03/2011, 11h28

Désolé je savais pas comment faire pour le code merci

je suis débutant dans la programmation
je ne sais pas ce que cest que le singleton

**Gurdil le nain** · 23/03/2011, 11h34

Si tu es débutant en programmation, une info pour toi :

Sinon, voici un lien qui explique relativement simplement comment créé un singleton : http://webman.developpez.com/article...rns/singleton/

**moonshine972** · 23/03/2011, 11h59

Je vais essayer
Merci

**moonshine972** · 23/03/2011, 16h56

J'ai passé mon aprem dessus mais je n'y arrive pas

sa fait 3 jours que je taff sur cette parti je commence a perdre espoir

**Gurdil le nain** · 23/03/2011, 17h03

Tu n'arrives pas à quoi ? Créer ton singleton ?

**moonshine972** · 23/03/2011, 17h15

oui j'y arrive pas

**StringBuilder** · 23/03/2011, 17h23

Salut !

C'est peut-être mon expérience de développeur Web qui parle, mais j'ai une petite suggestion...

Plutôt que d'utiliser des Singletons, ou une connexion "explicitement" persistante, pourquoi ne pas utiliser la puissance de .NET et des objets de connexion aux données ?

En effet, en .NET, quand un objet est détruit au sens fonctionnel du terme (scope terminé) il reste en réalité tout à fait vivant physiquement, tant que le garbage collector ne lui fait pas sa fête.

Quand on sait ça, on en sait déjà pas mal. Idem pour les cnx.Close()... Ouais ouais, on a demandé à fermer la connexion... En réalité, l'objet .NET est libéré, et la connexion physique à la base est remise en pool...

Donc quand on fait X fois de suite cnx.Open() et cnx.Close(), y compris avec des objets différents, dans des scopes différents, "physiquement", il ne se passe rien : une unique connexion reste ouverte tout ce temps vers le serveur.

A partir de là, colles ta chaîne de connexion en ressource de ton application.

Et ouvre/ferme ta connexion comme tu as fait dans ton Form1 chaque fois que tu en as besoin. Tu n'as pas à te soucier de savoir si tu as laissé la connexion ouverte dans une autre form ou quoi que ce soit. Imagine que cnx.Open() ça fait un "begin transaction" et cnx.Close() ça fait un "commit".

Ca ne touche rien à ta connexion réelle (donc aux perfs) de l'ouvrir et de la fermer, et surtout, tu vas te heurter à tout un tas de problèmes d'accès (dead locks, timeout etc.) si tu commences à vouloir gérer les ouvertures/fermetures de façon fonctionnelle (j'ouvres dans la form1 et je réutilise dans la form2).
Ouvre et ferme dès que tu n'en a plus immédiatement besoin.

J'utilise par exemple dans une application des UserControl qui chargent des listes de données avec un cnx.Open() et cnx.Close() dans chaque user contrôle. Dans certaines forms de mon application, j'ai plusieur centaines de ces UserControls... ça met moins de 1 seconde à s'afficher. Je répète donc, pas de lien direct entre le cnx.Open() et le cnx.Close() et les ouvertures effectives de sessions dans ta base.

**DarkHerumor** · 23/03/2011, 17h37

Il y a plusieurs chose à voir dans ton code.

Déjà la façon dont tu ouvre ta commande, le fait de faire quelque part:

Code :

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

MySqlConnection wConnexion = new MySqlConnection();

Puis plus loin:

Code :

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

using (wConnexion) {...}

Ce n'est pas bon (bien que je ne connaisse pas la lib MySql), en .Net on procède plutôt comme suis:

Code :

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

1
2
3
4
5
6
7
8
9
using (SqlConnection sqlConnection = new SqlConnection(_strChaineDeConnectionSQL))
{
	SqlCommand sqlCommand = new SqlCommand(strNomProcedure, sqlConnection);
	sqlCommand.CommandType = CommandType.StoredProcedure;
	AttacherParametres(sqlCommand, objParametres);
 
	sqlConnection.Open();
	return sqlCommand.ExecuteNonQuery();
}

(Pense bien sur à utiliser des MySql[...], pour tes classes)

Ensuite, un singleton n'est pas nécessaire, mais la séparation de ta partie "Accès à la base de données", du reste du code est presque indispensable. Tout simplement pour clarifier ton code, et isolé les différentes parties qui n'ont pas besoin d'être aussi fortement lié.

Je conseil de réaliser une classe, static pour avoir simple.
Dans cette classe, tu rassemble tes méthode d'accès à la base, un p'tit exemple:

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
static class DataProvider
{
	private static readonly string _connectionString;
 
	//Constructeur "static", nouveauté C# 4 me semble
	static DataProvider()
	{
		//CONFIGURATION => si tu as un fichier de configuration pour les connection strings (ce qui est mieux), sinon mets directement ta ConnectionString
		_connectionString = ConfigurationManager.ConnectionStrings["MySql.ConnectionString"].ConnectionString;
	}
 
	public static int ExecuteNonQuery(string query)
	{
		using (MySqlConnection connection = new MySqlConnection(_connectionString))
		{
			MySqlCommand command = new MySqlCommand(query, connection);
 
			connection.Open();
			return command.ExecuteNonQuery();
		}
	}
}

En procédant ainsi, tu peux appeler de n'importe quelle Form de ton programm:

Code :

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

DataProvider.ExecuteNonQuery("Insert ...");

Pour aller plus loin, je te conseil également de ne pas laisser de SQL dans les forms, fait des méthodes dans le DataProvider qui englobe le tout. Par exemple une méthode "Insert{truc}" qui prend en paramètres ce qui va bien:

Code :

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

1
2
3
4
5
 
public static int InsererTruc(string nom, string prenom, string message)
{
	//...
}

Pour plus d'info sur les paramètres et les commandes, je te conseil d'aller voir par là.

En espérant t'avoir aidé

**StringBuilder** · 23/03/2011, 17h46

Je chipotte, mais moi je ferais :

Code :

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

1
2
3
4
5
6
7
8
9
 
		using (MySqlConnection connection = new MySqlConnection(_connectionString))
		{
			MySqlCommand command = new MySqlCommand(query, connection);
 
			connection.Open();
			return command.ExecuteNonQuery();
			connection.Close();
		}

(avec un .Close() explicite)

**DarkHerumor** · 23/03/2011, 17h59

Après le return, il sert pas a grand chose :p

Le Dispose() de la Connection le fait d'elle même; donc j'ai prit l'habitude de plus le mettre. Mais en effet, pour la lecture et la compréhension c'est une bonne habitude.

**StringBuilder** · 23/03/2011, 21h34

ah ouais j'avais pas vu le return

**moonshine972** · 24/03/2011, 09h05

Merci pour votre aide je test ca de suite

**Gurdil le nain** · 24/03/2011, 09h24

Ouais enfin, la classe static et le singleton au final ça revient à n'avoir qu'une seule instance de connexion...

**DarkHerumor** · 24/03/2011, 09h52

A ma connaissance il est préférable d'avoir la requête dans un bloc using autour de la connexion, mais je serai plus le justifier...

**kheironn** · 24/03/2011, 10h12

Envoyé par Gurdil le nain

Ouais enfin, la classe static et le singleton au final ça revient à n'avoir qu'une seule instance de connexion...

static ? y'a pas d'instance (il me semble)

Envoyé par DarkHerumor

A ma connaissance il est préférable d'avoir la requête dans un bloc using autour de la connexion, mais je serai plus le justifier...

Le using sert aux flemmards qui ne veulent pas s'encombrer de la gestion des connexion, etc. ça ouvre, exécute, ferme et libère...

ce n'est, au final, que de la facillitation syntaxique, car un développeur qui ferait son code "à la main" ferai la même chose (s'il n'oublie rien)...

**DarkHerumor** · 24/03/2011, 10h40

Je comprend pas les gens qui traite de flemmard quand on utilise la simplification qu'offre un langage pour passer plus de temps sur de vrai problèmes :p

Entre quelqu'un qui utilise, les outils fournit par un langage pour faire une chose. Et un autre qui pour faire la même chose, le fait lui même à la main. Lequel à le plus de risque de bug?

Après là ou j'ai un doute, c'est sur les accès concurrent, si chaque méthode ouvre sa connexion, là c'est sur, elle sont chacune isolée. Mais si tu n'a qu'une connexion, et que tu as des requête concurrentes (pas sur la table ou autre, mais juste sur l’exécution des commandes) ?