Discussion :

[pemco]

salut les éminents développeurs,j'ai un souci avec un code.
Je me permets de publier celui-ci sur le site afin de bénéficier de vos largesses.
Voici mon code:
Classe de connection:

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
 
public static class ConnectionClass
    {
        private static SqlConnection conn;
        private static SqlCommand command;
 
        static ConnectionClass()
        {
            string ConnectionString = ConfigurationManager.ConnectionStrings["GestEmployeConnectionString"].ToString();
            conn = new SqlConnection(ConnectionString);
            command = new SqlCommand("", conn);
        }       
 
        public static void AjoutDepartemnt(Departement departement)
        {
            string query = string.Format("INSERT INTO DEPT VALUES ('{0}',@DNAME,'{2}'",
            departement.Code, departement.Nom, departement.Loc);
            command.CommandText = query;
            command.Parameters.Add(new SqlParameter("DNAME", departement.Nom));
 
            try
            {
                conn.Open();
                command.ExecuteNonQuery();
            }
            finally
            {
                conn.Close();
                command.Parameters.Clear();
            }
 
        }
 
    }

Classe departement:

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
public class Departement
    {
 
        public string Code { get; set; }
        public string Nom { get; set; }
        public string Loc { get; set; }
 
 
        public Departement(string code, string nom, string loc)
        {
            Code = code;
            Nom = nom;
            Loc = loc;
 
        }
 
    }

Classe pour réinitialiser les champs:

Code :

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

1
2
3
4
5
6
7
8
private void ClearTextFields()
        {
            TxtCode.Text = "";            
            TxtNom.Text = "";
            TxtLocalité.Text = "";
 
 
        }

Bouton enregister:

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
 
protected void Btn_Enregistrer_Click(object sender, EventArgs e)
        {
            try
            {
                string code = TxtCode.Text;
                string nom = TxtNom.Text;
                string loc = TxtLocalité.Text;
 
                Departement departement = new Departement(code, nom, loc);
                ConnectionClass.AjoutDepartemnt(departement);
 
                MessageBox.Show("les données ont été enregistreées avec succès!");
                ClearTextFields();
            }
            catch (Exception)
            {
 
                MessageBox.Show("Impossible d'enregistrer les données!");
            }
        }

Je n'arrive pas enregister les données la base de donnés.Prière de me donner un coup de main.
voici le message qui s'affiche:Impossible d'enregistrer les données!

[roro06]

Bonjour

1/ Au minimum, j'aurais remplacé

Code :

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

1
2
3
4
 catch (Exception)
             {
                   MessageBox.Show("Impossible d'enregistrer les données!");   
          }

par :

Code :

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

1
2
3
4
 catch (Exception ex)
             {
                   MessageBox.Show(ex.Message);  
           }

J'aurais quand même mis :

Code :

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

1
2
3
4
5
 
catch (Exception ex)
             {
                    EventLog.WriteEntry(sSource, ex.Message, EventLogEntryType.Error);
             }

Et en fin de compte, pour le débuggage, j'aurais même enlevé le bloc de code du try..catch, histoire d'avoir un vrai message d'erreur

2/ Es tu sûr d'être en ASP? L'utilisation de MessageBox en ASP me semble ... bizarre

3/

string.Format("INSERT INTO DEPT VALUES ('{0}',@DNAME,'{2}'", ...

ça, ça va planter. mets {1}. Tu as fait un micmac entre les paramètres de ta requête et ton String.Format()

[pemco]

en mettant 1 à la place de @DNAME
ça ne change rien.

Code :

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

1
2
 
string.Format("INSERT INTO DEPT VALUES ('{0}',@DNAME,'{2}'", ...

[roro06]

Soit tu fais un String.Format :

Code :

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

string sql=string.Format("INSERT INTO DEPT VALUES ('{0}', '{1}', '{2}')", code, nom, loc);

soit une requête paramétrée (c'est mieux)

Code :

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

1
2
3
4
5
6
7
8
sql=
"INSERT INTO DEPT VALUES (@code, @nom, @loc)"

...
command.Parameters.AddWithValue("code", code);
command.Parameters.AddWithValue("nom", nom);
command.Parameters.AddWithValue("loc", loc);

Mais les deux ensembles,

[Air P-E]

Beaucoup d'informations t'ont été remontées déjà pour ton problème. Je vais donc simplement te donner un conseil, une préférence. Après chacun ses goûts et sa façon de faire.

Personnellement je ne suis pas fan des requêtes écrites dans le code, je préfère utiliser des procédures stockées ce qui permet d'avoir un code et des requêtes bien séparées. De plus ça te permet éventuellement de modifier tes requêtes sans avoir à modifier/recompiler ton code.

[Hlama]

D'abord tu utilise quel est SGBD?
Parce que ta requête me semble bizzare
"INSERT INTO DEPT VALUES (@code, @nom, @loc)"

[chamamo]

D'abord tu utilise quel est SGBD?
Parce que ta requête me semble bizzare
"INSERT INTO DEPT VALUES (@code, @nom, @loc)"

La requête reste correcte (dans le cas ou la table contient que les 3 champs code, nom et loc)

Personnellement je ne suis pas fan des requêtes écrites dans le code, je préfère utiliser des procédures stockées ce qui permet d'avoir un code et des requêtes bien séparées. De plus ça te permet éventuellement de modifier tes requêtes sans avoir à modifier/recompiler ton code.

Chacun ses goûts tu as raison, moi je ne suis pas du même avis surtout dans le cas ou on a une application qui utilises plusieurs SGBD, et je préfère ne pas mettre du tout le code fonctionnel dans les procédures stockées ça me permet de rester indépendant du support du stockage (SGBD, XML, Service Web...)