Problème en code c# .NET

Version imprimable

Bonjour
J'ai créé une classe Cmateriel dont les attributs sont

Code:

1
2
3
4
5
6
7
8
 
public int ID_MAT;
        public String Description_MAT;
        public int N_serie;
        public int ID_Modele;
        public int Prix_achat;
        public int ID_Lot;
        public int ID_Etat;

ensuite j'ai créé des méthodes

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
 
public void ajouter(int id_mat, string description_mat, int N_serie, int modele_mat, int prix_mat, int id_lot, int id_etat) 
        {
            Form1 fm = new Form1();
            SqlConnection conn = new SqlConnection();
            SqlCommand cmd = new SqlCommand();
            conn.Open();
            cmd = new SqlCommand("INSERT INTO MATERIEL VALUES("+t1.text+",'"+t2.text+"',"+t3.text+","+t4.text+","+t5.text+","+t6.text+","+t7.text+")",conn);
            cmd.ExecuteNonQuery();
            conn.Close();
        }
 
public void supprimer(int id_mat, string description_mat, int N_serie, int modele_mat, int prix_mat, int id_lot, int id_etat) 
        {
            SqlConnection conn = new SqlConnection();
            SqlCommand cmd = new SqlCommand();
            conn.Open();
            cmd = new SqlCommand("DELETE FROM MATERIEL WHERE id_mat ="+ id_mat,conn);
            cmd.ExecuteNonQuery();
            conn.Close();
        }

J'ai créé un formulaire et quand je veux remplir le code des boutons Ajouter et Supprimer par

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
 
CMateriel cm = new CMateriel();
        connexion conn = new connexion();
 
 
private void ajouter_Click(object sender, EventArgs e) {
 
            cm.ajouter(textBox1.Text, textBox2.Text,textBox3.Text,textBox4.Text,textBox5.Text,textBox6.Text,textBox7.Text);
 
 
        }
 
private void supprimer_Click(object sender, EventArgs e) {
 
            cm.supprimer(textBox1.Text, textBox2.Text,textBox3.Text,textBox4.Text,textBox5.Text,textBox6.Text,textBox7.Text);
 
 
        }

ca ne fonctionne pas.Veuillez m'aider s'il vous plait.

31/05/2014, 19h40
Momoth

Bonjour,

On a besoin de plus d'information que "ça ne fonctionne pas". Tu as un message d'erreur ?

Oui j'ai des erreurs comme ici

Code:

1
2
3
4
5
6
 
//classe Form1(code du formulaire)
private void ajouter_Click(object sender, EventArgs e)
        {
            cm.ajouter(T1.Text, T2.Text, T3.Text, T4.Text, T5.Text, T6.Text, T7.Text);
        }

cm.ajouter ... est souligné en rouge

Code:

1
2
3
4
5
6
7
8
9
10
11
12
 
//classe Cmateriel
public void ajouter(int ID_MAT, String Description_MAT, int N_serie, int ID_Modele, int Prix_achat, int ID_Lot, int ID_Etat)
        {
            Form1 fm = new Form1();
            SqlConnection conn = new SqlConnection();
            SqlCommand cmd = new SqlCommand();
            conn.Open();
            cmd = new SqlCommand("INSERT INTO MATERIEL VALUES("+ T1.Text +",'" + t2.text + "'," + t3.text + "," + t4.text + "," + t5.text + "," + t6.text + "," + t7.text + ")", conn);
            cmd.ExecuteNonQuery();
            conn.Close();
        }

La meme chose pour cette partie
("+ T1.Text +",'" + t2.text + "'," + t3.text + "," + t4.text + "," + t5.text + "," + t6.text + "," + t7.text + "

02/06/2014, 09h10
Momoth

Bonjour,

Les messages d'erreurs sont affichés quand tu passes le curseur sur la ligne rouge.

Pour ta méthode ajouter_Click() tu as déjà un problème de type. Tu n'envoie que des string à ta méthode ajouter(). Le cast de string à int de ne se fait pas implicitement. Tu doit faire la conversion toi même.

Ensuite pour ta deuxième méthode, tu appelles des objets qui n'existent pas dans ce contexte. t1, t2, ... sont tes textbox situés dans Form1, ils ne sont pas accessibles directement depuis les autres classes. De plus, tu passes déjà leurs contenus en paramètre à ta méthode mais tu ne les utilises pas.

...

Salut
C'est bon j'ai trouvé l’erreur, mais je veux remplir mon bouton ajouter depuis cette méthode située dans la classe CMateriel

Code:

1
2
3
4
5
 
//La classe CMateriel : Méthode ajouter
public void Ajouter(CMateriel obj) {
            sql.update("INSERT INTO MATERIEL(DESCRIPTION_MAT, N_SERIE, ID_MODELE, PRIX_ACHAT, ID_LOT, ID_ETAT) VALUES('" + obj.DESCRIPTION_MAT + "', " + obj.N_SERIE + ", " + obj.ID_MODELE + ", " + obj.PRIX_ACHAT + ", " + obj.ID_LOT + ", " + obj.ID_ETAT + ")");
        }

Et voici la classe du Formulaire

Code:

1
2
3
4
5
6
7
 
 
private void button2_Click(object sender, EventArgs e) {
         CMateriel cm = new CMateriel();
         cm.ajouter(  //Je sais pas ce que je dois écrire ici );
 
        }

07/06/2014, 19h12
Momoth
Il faut que tu lui passe une instance de CMateriel avec toutes les propriétés que tu veux enregistrer.

Ça te donne quelque chose dans le style suivant :
Code:

1 2 3 4 5 6 cm.ajouter(new CMateriel() { N_serie = T1.text, ID_modele = T2.text, ... });
12/06/2014, 13h37
iberserk

Pensez à utiliser les sqlParameters pour l'insertion de vos données, vous vous exposez à un risque d'attaque par SQLINJECTION!